patchFreeciv - Patches: patch #4928, Introduce savegame3.c

 
 
Show feedback again

patch #4928: Introduce savegame3.c

Submitted by:  Marko Lindqvist <cazfi>
Submitted on:  Fri 11 Jul 2014 01:09:45 AM UTC  
 
Category: generalPriority: 5 - Normal
Status: In ProgressPrivacy: Public
Assigned to: NoneOpen/Closed: Open
Planned Release: 3.0.0

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Wed 19 Nov 2014 05:05:21 AM UTC, comment #5:

- Updated against svn

(file #22922)

Marko Lindqvist <cazfi>
Project Administrator
Wed 19 Nov 2014 04:53:16 AM UTC, comment #4:

> Is it really need? Are we already planning to make such changes
> in the savegame format that would require a new module
> "savegame3"?


This is not about one big change, but the accumulating cruft over time. At the time savegame3.c gets created, savegam2.c is only in half way through its life. It will still be needed for a long time to provide backeward compatibility to old savegames. To avoid the situation that it gets very hard to maintain in the end, we should do the cleanup to savegame3.c early enough. See the metaticket patch #4783

Marko Lindqvist <cazfi>
Project Administrator
Fri 19 Sep 2014 06:13:46 AM UTC, comment #3:

Is it really need? Are we already planning to make such changes in the savegame format that would require a new module "savegame3"?

pepeto <pepeto>
Project Member
Thu 18 Sep 2014 11:14:35 PM UTC, comment #2:

- Updated against svn

(file #22276)

Marko Lindqvist <cazfi>
Project Administrator
Sun 13 Jul 2014 02:58:25 AM UTC, comment #1:

Patch. Copy savegame2.c as savegame3.c and savegame2.h as savegame3.h before applying

(file #21397)

Marko Lindqvist <cazfi>
Project Administrator
Fri 11 Jul 2014 01:09:45 AM UTC, original submission:

This is "immediately-after-S2_6-branching" part of patch #4738. This patch needs to make the minimum to introduce savegame3.c where any new savegame format changes can take place during 3.0 development.

Marko Lindqvist <cazfi>
Project Administrator

 

(Note: upload size limit is set to 1024 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

Attached Files
file #22922:  Savegame3-141119.patch.bz2 added by cazfi (16kB - application/x-bzip)
file #21397:  Savegame3.patch.bz2 added by cazfi (15kB - application/x-bzip)

 

Digest:
   patch dependencies.

Digest:
   patch dependencies.

 

Carbon-Copy List
  • -unavailable- added by pepeto (Posted a comment)
  • -unavailable- added by cazfi (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 5 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Wed 19 Nov 2014 05:05:21 AM UTCcazfiAttached File-=>Added Savegame3-141119.patch.bz2, #22922
    Thu 18 Sep 2014 11:14:35 PM UTCcazfiAttached File-=>Added 0002-Introduce-savegame3.c.patch, #22276
    Sun 13 Jul 2014 02:58:25 AM UTCcazfiAttached File-=>Added Savegame3.patch.bz2, #21397
    Fri 11 Jul 2014 06:55:32 AM UTCcazfiDependencies-=>Depends on patch #4929
    Fri 11 Jul 2014 01:10:05 AM UTCcazfiDependencies-=>patch #4783 is dependent
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup