megaglest-source icon indicating copy to clipboard operation
megaglest-source copied to clipboard

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

Open andy5995 opened this issue 7 years ago • 6 comments

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

andy5995 avatar Mar 10 '17 22:03 andy5995

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.

tomreyn avatar Mar 10 '17 23:03 tomreyn

The map cliff_valley is also affected by this

andy5995 avatar Mar 11 '17 02:03 andy5995

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

titiger avatar Mar 11 '17 20:03 titiger

I will remove critical maps now from mod center.

titiger avatar Mar 11 '17 20:03 titiger

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 avatar Mar 11 '17 20:03 andy5995

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

andy5995 avatar Mar 11 '17 22:03 andy5995