On Tue, 15 Feb 2011, Marko Lindqvist wrote:
I'm going through patches freeciv.net has applied on top of freeciv.
Goal is to make freeciv -> freeciv.net diff smaller where possible.

General notes: Copy of freeciv has been pushed to freeciv.net git in
quite a messy way. There's a lot of files that IMO shouldn't be under
version control (build files specific to machine where said build has
been performed, backup copies of files, *.rej, *.orig files, files
probably created accidentally...)  This makes it impossible to simply
compare freeciv.net copy of freeciv tree to our tree. It would also be
nice to know which freeciv revision this is copy of.
Same goes for patches. Diff-files have relevant information about
code changes buried deep in to noise caused by changes in build system
etc. I think this should be cleaned up in freeciv.net side.

... gotta go without analyzing individual patches... just quick note
that nosave.diff and noquitidle.diff should not be needed once patch
#2536 has been committed.

I agree that making the freeciv -> freeciv.net diff smaller where possible is a great goal. Most of the patches to the Freeciv.org server have been created because some functionality was missing in the Freeciv.net client or Freeciv-proxy. This means that removing most of these diffs would involve implementing the corresponding functionality in the Freeciv.net web client. I have documented all the patches here:


 - Andreas

Freeciv-dev mailing list

Reply via email to