Changes between Version 4 and Version 9 of Ticket #2807


Ignore:
Timestamp:
Jul 12, 2016, 10:27:05 AM (8 years ago)
Author:
Itms
Comment:

Hi, and thanks for your work so far. :)

When this ticket was created, we didn't have the list of enabled mods included in the commands.txt file. Now that we have that, it should be possible to check that the list of enabled mods is the same as in the commands file.

Currently the check is done in the visual replay menu but not when using the command line. I think a mismatch in the mod lists should trigger a warning instead of an error, so people using the command line are still allowed to do what they want.

This would be a more complex task. Would you be up for it? You can come to our IRC channel if you need more information.

Also, for next time, please take a look at our new way of SubmittingPatches: we're making changes to our review process.

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #2807

    • Property Keywords patch added
    • Property Status newassigned
    • Property Milestone BacklogAlpha 21
    • Property Cc leper removed
    • Property Owner set to Arno Hemelhof
    • Property Summary Display a nice error message when trying to run a replay without mods[PATCH] Check the list of enabled mods when running a replay from the command line
  • Ticket #2807 – Description

    v4 v9  
    11Right now the replay mode fails when no mod is specified, we should check that and inform the user about what is happening.
    22
    3 Maybe this is part of a more general task, which is including mod data in commands.txt and other logging stuff.
     3~~Maybe this is part of a more general task, which is including mod data in commands.txt and other logging stuff.
     4
     5We can use the list of mods in the commands.txt to make sure everything is fine.