Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add 'libmodplug' to RHEL prereqs in docs #515

Merged
merged 1 commit into from
Sep 1, 2020
Merged

Add 'libmodplug' to RHEL prereqs in docs #515

merged 1 commit into from
Sep 1, 2020

Conversation

yeslayla
Copy link
Contributor

@yeslayla yeslayla commented Sep 1, 2020

Error

Version: 0.9.0
OS: Linux (Fedora Silverblue)

I was getting an exception on run due to MIX_INIT_MOD from the following code being passed into Mix_Init:

mix_call(Mix_Init, MIX_INIT_FLAC | MIX_INIT_MOD | MIX_INIT_MP3 | MIX_INIT_OGG)

Looking into this, I found that this is due to the SDL2_mixer binary build for Fedora is missing the MOD support for whatever reason.

Full error (when missing package) for reference:

Traceback (most recent call last):
  File "main.py", line 12, in <module>
    ppb.run(setup=setup)
  File "/var/home/vetra/Projects/build-ppb-action/test_project/.venv/lib/python3.7/site-packages/ppb/__init__.py", line 122, in run
    with make_engine(setup, starting_scene=starting_scene, title=title, **engine_opts) as eng:
  File "/var/home/vetra/Projects/build-ppb-action/test_project/.venv/lib/python3.7/site-packages/ppb/engine.py", line 107, in __enter__
    self.start_systems()
  File "/var/home/vetra/Projects/build-ppb-action/test_project/.venv/lib/python3.7/site-packages/ppb/engine.py", line 124, in start_systems
    self.exit_stack.enter_context(system)
  File "/usr/lib64/python3.7/contextlib.py", line 427, in enter_context
    result = _cm_type.__enter__(cm)
  File "/var/home/vetra/Projects/build-ppb-action/test_project/.venv/lib/python3.7/site-packages/ppb/systems/sound.py", line 88, in __enter__
    mix_call(Mix_Init, MIX_INIT_FLAC | MIX_INIT_MOD | MIX_INIT_MP3 | MIX_INIT_OGG)
  File "/var/home/vetra/Projects/build-ppb-action/test_project/.venv/lib/python3.7/site-packages/ppb/systems/_sdl_utils.py", line 94, in mix_call
    raise SdlMixerError(f"Error calling {func.__name__}: {err.decode('utf-8')}")
ppb.systems._sdl_utils.SdlMixerError: Error calling Mix_Init: MOD support not available

Solution

To solve this error I installed the libmodplug package.

Code Changes

This PR just updates the docs to include the libmodplug package for Fedora, Centos, and RHEL.

@yeslayla yeslayla requested a review from a team as a code owner September 1, 2020 02:19
@binny-the-binary-bear
Copy link

Thank you for contributing to PursuedPyBear! Don't forget to add yourself CONTRIBUTORS.md.

Copy link
Member

@AstraLuma AstraLuma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bors r+

bors bot added a commit that referenced this pull request Sep 1, 2020
515: Add 'libmodplug' to RHEL prereqs in docs r=AstraLuma a=josephbmanley

## Error

Version: `0.9.0`
OS: `Linux (Fedora Silverblue)`

I was getting an exception on run due to `MIX_INIT_MOD` from the following code being passed into `Mix_Init`:
https://github.com/ppb/pursuedpybear/blob/f34fed3cd34b9b5da2629bbc0e4de29c17619b56/ppb/systems/sound.py#L88

Looking into this, I found that this is due to the `SDL2_mixer` binary build for Fedora is missing the `MOD` support for whatever reason. 

Full error (when missing package) for reference:
```
Traceback (most recent call last):
  File "main.py", line 12, in <module>
    ppb.run(setup=setup)
  File "/var/home/vetra/Projects/build-ppb-action/test_project/.venv/lib/python3.7/site-packages/ppb/__init__.py", line 122, in run
    with make_engine(setup, starting_scene=starting_scene, title=title, **engine_opts) as eng:
  File "/var/home/vetra/Projects/build-ppb-action/test_project/.venv/lib/python3.7/site-packages/ppb/engine.py", line 107, in __enter__
    self.start_systems()
  File "/var/home/vetra/Projects/build-ppb-action/test_project/.venv/lib/python3.7/site-packages/ppb/engine.py", line 124, in start_systems
    self.exit_stack.enter_context(system)
  File "/usr/lib64/python3.7/contextlib.py", line 427, in enter_context
    result = _cm_type.__enter__(cm)
  File "/var/home/vetra/Projects/build-ppb-action/test_project/.venv/lib/python3.7/site-packages/ppb/systems/sound.py", line 88, in __enter__
    mix_call(Mix_Init, MIX_INIT_FLAC | MIX_INIT_MOD | MIX_INIT_MP3 | MIX_INIT_OGG)
  File "/var/home/vetra/Projects/build-ppb-action/test_project/.venv/lib/python3.7/site-packages/ppb/systems/_sdl_utils.py", line 94, in mix_call
    raise SdlMixerError(f"Error calling {func.__name__}: {err.decode('utf-8')}")
ppb.systems._sdl_utils.SdlMixerError: Error calling Mix_Init: MOD support not available
```

## Solution

To solve this error I installed the `libmodplug` package.

## Code Changes

This PR just updates the docs to include the `libmodplug` package for Fedora, Centos, and RHEL.

Co-authored-by: Joseph Manley <j@cloudsumu.com>
@bors
Copy link
Contributor

bors bot commented Sep 1, 2020

Build failed:

@pathunstrom
Copy link
Collaborator

bors retry

bors bot added a commit that referenced this pull request Sep 1, 2020
515: Add 'libmodplug' to RHEL prereqs in docs r=AstraLuma a=josephbmanley

## Error

Version: `0.9.0`
OS: `Linux (Fedora Silverblue)`

I was getting an exception on run due to `MIX_INIT_MOD` from the following code being passed into `Mix_Init`:
https://github.com/ppb/pursuedpybear/blob/f34fed3cd34b9b5da2629bbc0e4de29c17619b56/ppb/systems/sound.py#L88

Looking into this, I found that this is due to the `SDL2_mixer` binary build for Fedora is missing the `MOD` support for whatever reason. 

Full error (when missing package) for reference:
```
Traceback (most recent call last):
  File "main.py", line 12, in <module>
    ppb.run(setup=setup)
  File "/var/home/vetra/Projects/build-ppb-action/test_project/.venv/lib/python3.7/site-packages/ppb/__init__.py", line 122, in run
    with make_engine(setup, starting_scene=starting_scene, title=title, **engine_opts) as eng:
  File "/var/home/vetra/Projects/build-ppb-action/test_project/.venv/lib/python3.7/site-packages/ppb/engine.py", line 107, in __enter__
    self.start_systems()
  File "/var/home/vetra/Projects/build-ppb-action/test_project/.venv/lib/python3.7/site-packages/ppb/engine.py", line 124, in start_systems
    self.exit_stack.enter_context(system)
  File "/usr/lib64/python3.7/contextlib.py", line 427, in enter_context
    result = _cm_type.__enter__(cm)
  File "/var/home/vetra/Projects/build-ppb-action/test_project/.venv/lib/python3.7/site-packages/ppb/systems/sound.py", line 88, in __enter__
    mix_call(Mix_Init, MIX_INIT_FLAC | MIX_INIT_MOD | MIX_INIT_MP3 | MIX_INIT_OGG)
  File "/var/home/vetra/Projects/build-ppb-action/test_project/.venv/lib/python3.7/site-packages/ppb/systems/_sdl_utils.py", line 94, in mix_call
    raise SdlMixerError(f"Error calling {func.__name__}: {err.decode('utf-8')}")
ppb.systems._sdl_utils.SdlMixerError: Error calling Mix_Init: MOD support not available
```

## Solution

To solve this error I installed the `libmodplug` package.

## Code Changes

This PR just updates the docs to include the `libmodplug` package for Fedora, Centos, and RHEL.

Co-authored-by: Joseph Manley <j@cloudsumu.com>
@bors
Copy link
Contributor

bors bot commented Sep 1, 2020

Build failed:

@pathunstrom
Copy link
Collaborator

bors retry

bors bot added a commit that referenced this pull request Sep 1, 2020
515: Add 'libmodplug' to RHEL prereqs in docs r=AstraLuma a=josephbmanley

## Error

Version: `0.9.0`
OS: `Linux (Fedora Silverblue)`

I was getting an exception on run due to `MIX_INIT_MOD` from the following code being passed into `Mix_Init`:
https://github.com/ppb/pursuedpybear/blob/f34fed3cd34b9b5da2629bbc0e4de29c17619b56/ppb/systems/sound.py#L88

Looking into this, I found that this is due to the `SDL2_mixer` binary build for Fedora is missing the `MOD` support for whatever reason. 

Full error (when missing package) for reference:
```
Traceback (most recent call last):
  File "main.py", line 12, in <module>
    ppb.run(setup=setup)
  File "/var/home/vetra/Projects/build-ppb-action/test_project/.venv/lib/python3.7/site-packages/ppb/__init__.py", line 122, in run
    with make_engine(setup, starting_scene=starting_scene, title=title, **engine_opts) as eng:
  File "/var/home/vetra/Projects/build-ppb-action/test_project/.venv/lib/python3.7/site-packages/ppb/engine.py", line 107, in __enter__
    self.start_systems()
  File "/var/home/vetra/Projects/build-ppb-action/test_project/.venv/lib/python3.7/site-packages/ppb/engine.py", line 124, in start_systems
    self.exit_stack.enter_context(system)
  File "/usr/lib64/python3.7/contextlib.py", line 427, in enter_context
    result = _cm_type.__enter__(cm)
  File "/var/home/vetra/Projects/build-ppb-action/test_project/.venv/lib/python3.7/site-packages/ppb/systems/sound.py", line 88, in __enter__
    mix_call(Mix_Init, MIX_INIT_FLAC | MIX_INIT_MOD | MIX_INIT_MP3 | MIX_INIT_OGG)
  File "/var/home/vetra/Projects/build-ppb-action/test_project/.venv/lib/python3.7/site-packages/ppb/systems/_sdl_utils.py", line 94, in mix_call
    raise SdlMixerError(f"Error calling {func.__name__}: {err.decode('utf-8')}")
ppb.systems._sdl_utils.SdlMixerError: Error calling Mix_Init: MOD support not available
```

## Solution

To solve this error I installed the `libmodplug` package.

## Code Changes

This PR just updates the docs to include the `libmodplug` package for Fedora, Centos, and RHEL.

Co-authored-by: Joseph Manley <j@cloudsumu.com>
@bors
Copy link
Contributor

bors bot commented Sep 1, 2020

Build failed:

@AstraLuma
Copy link
Member

bors retry

@bors bors bot merged commit f38fd84 into ppb:canon Sep 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants