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

fix for uqm optional 3DO music and speech #3484

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

atfrase
Copy link

@atfrase atfrase commented Feb 17, 2022

As reported in #3483, the uqm.sh install script currently puts all content archives in the installed .../content/packages/ directory, which seems eminently reasonable. However, despite the upstream project website and INSTALL guide having no obvious documentation to this effect, it is not correct: in fact only the uqm-*-content.uqm file goes here; uqm-*-voice.uqm and uqm-*-3domusic.uqm have to go in the .../content/addons/ directory or the game won't find them.

This patch corrects the path when installing uqm from source, but the pre-compiled binary archive still contains all three files in the packages/ directory and will have to be similarly updated.

I might also suggest adding the "remix" music packs to the installation, which are found at http://sc2.sourceforge.net/downloads.php under "The official Ur-Quan Masters remix packs"; they also have to go under .../content/addons/. I haven't added that to this PR since those files would also have to be mirrored in the RetroPie archives first.

@cmitu
Copy link
Contributor

cmitu commented Feb 21, 2022

Indeed, according to the Wiki, the proper location for addons is content/addons.
The same page mentions the addons are added with the --addon parameter - is the parameter needed for the 2 addons to be enabled/usable ? If so, we might add them to the launch command.

@atfrase
Copy link
Author

atfrase commented Feb 21, 2022 via email

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.

None yet

2 participants