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 Community Created Mods and Fix Replay Filename Issue #153

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

dnl-nash
Copy link

No description provided.

… War mods, along with Tang Campaign

Removed nonfunctional crazymod directory
…y not the best way to do this, but I'm still learning python, so please excuse any formatting/other issues
Added documentation for Crazy Mod
@soundmud
Copy link
Owner

Thanks for your pull request. I cannot include it as is, but I'll do my best to take it into account. Don't hesitate to send your pull requests or issues more "atomically". No need to group them.

First, about the mods, I am trying to isolate them from the code and have them in packages.

Please try this experimental feature if you want:

  1. edit cfg/package_servers.txt
  2. remove the ";" from the second line
  3. start SoundRTS
  4. wait until the available packages are downloaded or updated (it should say "OK" every time a package is downloaded)
  5. restart SoundRTS if the mods are not available in the options menu

To undo the change:

  1. restore the ";" in the txt file
  2. remove the zip files from user/downloaded/packages

That's all for now. More later.

@soundmud
Copy link
Owner

I will include a variant of your patch about replay files. I removed some redundancy and replaced the () which are not a tuple (it lacks a comma when there is one item) with a [] which doesn't need a comma. It was working anyway but it spelled the name instead of reading the name.

I might include the notice about pip and Linux later.

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