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

If map is included with distribution, but still in mod center, MG may crash at game mods menu #161

Open
andy5995 opened this issue Mar 10, 2017 · 6 comments

Comments

@andy5995
Copy link
Contributor

@andy5995 andy5995 commented Mar 10, 2017

Was meteor added to the 3.13.0 distribution?

For me and @tomreyn, MG crashes when we go into "game mods", click the meteor map, then return to the main menu.

I believe this is because MG is looking for the map in ~/.megaglest/maps
If it doesn't find it there, it crashes.

Now it's included in the distribution maps directory, but still apparently in the mod center.

The crash will not happen if a copy of meteor.mgm is placed in ~/.megaglest/maps

@tomreyn
Copy link
Member

@tomreyn tomreyn commented Mar 10, 2017

Indeed, this map appears to be included in the distribution now, and thus needs to be removed off the mod center for this release (only?).

The mod center database contains a field for minimum game version (by which mods will start to show up on mod center), but this issue suggests we may also need to have a maximum version (for when we add files previously available on the mod center into the distribution).

Additionally, we should add safeguards to handle this situation (a mod is included into distribution but still listed on mod db) gracefully in the future.

For now, I suggest we remove this mod off the mod DB, which fixes it for the current release, and makes the mods unavailable in past releases (where it was not part of the distribution).

We also need to check whether this also affects other mods.

@andy5995
Copy link
Contributor Author

@andy5995 andy5995 commented Mar 11, 2017

The map cliff_valley is also affected by this

@titiger
Copy link
Member

@titiger titiger commented Mar 11, 2017

Terminal output including stacktrace:
http://pastebin.com/raw/CuiDS22P

@titiger
Copy link
Member

@titiger titiger commented Mar 11, 2017

I will remove critical maps now from mod center.

@andy5995
Copy link
Contributor Author

@andy5995 andy5995 commented Mar 11, 2017

It actually seems the 2 maps mentioned were included since 3.12.0 and not recently added to 3.13.

https://github.com/MegaGlest/megaglest-data/tree/3.12.0/maps

@andy5995
Copy link
Contributor Author

@andy5995 andy5995 commented Mar 11, 2017

"prevention" was the 3rd and last map. @titiger removed all 3.

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

No branches or pull requests

3 participants