[Freeciv-Dev] Re: Blitzkrieg patch

[David Pfitzner <dwp@xxxxxxxxxxxxxx>]

Reed Meyer <rdm@xxxxxxxxxxxxxx> wrote:

> >A problem with this patch is that it introduces incompatibility
> >for servers and clients which don't have this patch (see 
> >common/capstr.c for how to address), and I guess for old 
> >savegames (should be possible to add work-around I guess),
> >and for other modpacks.
> 
> I'm submitting this patch for hopeful inclusion into the MAINSTREAM source
> code as of some future release (for example, 1.11.6 or 1.12 or whatever).
> Thus, the problems you mention are the same as would be encountered for
> ANY OTHER patch to the mainstream source code; generally speaking, for
> example, a server running freeciv 1.11 wouldn't be able to talk to a
> client running 1.10, right?

Freeciv has a mechanism for handling semi-compatible and incompatible 
protocol changes when they occur, called capability strings.  Please 
phrase your comments in a way which shows you have read common/capstr.c 
as directed :-)  This is supposed to be explained there, although
perhaps not very well...  www.freeciv.org/contribute.html also has a 
small discussion in section VI.  The capability string system applies
even to development versions.

-- David