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

Better mod naming. #1062

Closed
wants to merge 1 commit into from
Closed

Better mod naming. #1062

wants to merge 1 commit into from

Conversation

kaeza
Copy link
Contributor

@kaeza kaeza commented Dec 21, 2013

This patch does the following:

  • Allows mod authors to provide a mod.conf file, to specify the mod name directly (and can later be used for other information about mods).
  • Strips everything after the first dash. This makes it possible to have the -master suffix added by GitHub to the directory name.

This should avoid the need to rename the mod folders in most cases (with the first point, in all cases).

This patch does the following:
- Allows mod authors to provide a `mod_info.txt' file, to
  specify the mod name directly (and can later be used for
  other information about mods).
- Strips everything after the first dash. This makes it
  possible to have the `-master' suffix added by GitHub to the
  directory name.

This should avoid the need to rename the mod folders in
most cases (with the first point, in all cases).
@sapier
Copy link
Contributor

sapier commented Dec 23, 2013

There's already a file read for information in mod tab maybe we should use a single file for all of those things?
You have to fix mainmenu mod download code too. If you don't add support for this file it'll keep guessing the name as it does now.

@ShadowNinja
Copy link
Member

The second point should be removed because some archives are named author-mod-01234abc or similar.

description.txt should become a (multi-line) field in mod.conf. That can be done seperately though.

@rubenwardy
Copy link
Member

👍

You should also update everywhere else to use mod.conf instead of description.txt, keeping backwards compatibility of course.

@nerzhul
Copy link
Member

nerzhul commented Feb 21, 2015

Someone can review this ?

@nerzhul
Copy link
Member

nerzhul commented Feb 21, 2015

As discussed with @ShadowNinja on IRC. This issue is partially solved.

Commit c5defe7

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants