[Freeciv-Dev] [patch #3471] Set S2_4 "beta bit" on before actual beta1 release

2012-07-31 Thread Marko Lindqvist
URL: Summary: Set S2_4 "beta bit" on before actual beta1 release Project: Freeciv Submitted by: cazfi Submitted on: Wed 01 Aug 2012 06:37:57 AM EEST Category: general Priority:

[Freeciv-Dev] [patch #3457] Set estimated release month for 2.4.0 to February

2012-07-31 Thread Marko Lindqvist
Update of patch #3457 (project freeciv): Status: Ready For Test => Done Assigned to:None => cazfi Open/Closed:Open => Closed

[Freeciv-Dev] [bug #20015] Initial default_metaserver value is not magic value "default" but explicit value

2012-07-31 Thread Marko Lindqvist
Update of bug #20015 (project freeciv): Status: In Progress => Ready For Test Planned Release: => 2.3.3, 2.4.0, 2.5.0 ___ Follow-up Comment #1: Fixes (file #16264

[Freeciv-Dev] [bug #20002] x.m4 uses "==" instead of "=" for comparison

2012-07-31 Thread Marko Lindqvist
Update of bug #20002 (project freeciv): Status: Ready For Test => Fixed Open/Closed:Open => Closed ___ Reply to this item at:

[Freeciv-Dev] [bug #20015] Initial default_metaserver value is not magic value "default" but explicit value

2012-07-31 Thread Marko Lindqvist
URL: Summary: Initial default_metaserver value is not magic value "default" but explicit value Project: Freeciv Submitted by: cazfi Submitted on: Wed 01 Aug 2012 05:48:14 AM EEST Category: clie

[Freeciv-Dev] [bug #20014] References to metaserver.phtml

2012-07-31 Thread Marko Lindqvist
Follow-up Comment #2, bug #20014 (project freeciv): > The metaserver.phtml URL is also going to persist in people's > config files for a long time, unless we do something special to > rewrite it to the magic value "default". Indeed, there seems to be bug in setting magic value "default" for new i

[Freeciv-Dev] [patch #3470] Document use of --with-followtag in README.packaging

2012-07-31 Thread Marko Lindqvist
Follow-up Comment #2, patch #3470 (project freeciv): (Previous comment brought to you by frustation of porting patch to S2_0 where separate builddirs are broken and one has to use builddir == srcdir) ___ Reply to this item at:

[Freeciv-Dev] [patch #3470] Document use of --with-followtag in README.packaging

2012-07-31 Thread Marko Lindqvist
Follow-up Comment #1, patch #3470 (project freeciv): Nitpick: Could you avoid "./configure", i.e., giving example of running configure in srcdir. While we are not even recommending (some gnu packages even force) use of separate builddir(s), could we at least not to recommend the opposite. ___

[Freeciv-Dev] [bug #20003] Security advisory

2012-07-31 Thread Marko Lindqvist
Update of bug #20003 (project freeciv): Status:None => Ready For Test ___ Reply to this item at: ___ Message sent

[Freeciv-Dev] [bug #20003] Security advisory

2012-07-31 Thread Marko Lindqvist
Follow-up Comment #10, bug #20003 (project freeciv): > I see two possible ways to fix this. Well, third one is what I've planned to do for a long time*: give dio_get_xxx() functions return values telling if they succeeded or failed. Patch attached. *) According to very old TODO I had actually fo

[Freeciv-Dev] [patch #3456] Add language names list

2012-07-31 Thread Marko Lindqvist
Follow-up Comment #4, patch #3456 (project freeciv): > Is omission of nb = "Norwegian Bokmål" deliberate? Not intentional. I just listed .po files from svn and forgot that nb.po gets generated (=is not in svn) by a ugly hack. Do we want it listed? And as language names are currently in English, w

[Freeciv-Dev] [patch #3469] Get rid of most deprecated warning

2012-07-31 Thread RafałMużyło
Follow-up Comment #3, patch #3469 (project freeciv): Also, there's a little thing I've missed in the initial port (mainly cause it was still hybrid then) - except for most gdk_window_get_pointer calls, those calls can be made far more simple via the content of events they handle.

[Freeciv-Dev] [patch #3456] Add language names list

2012-07-31 Thread Jacob Nevins
Follow-up Comment #3, patch #3456 (project freeciv): Sorry late comments but: * Typo "Czesh" should be "Czech" * Is omission of nb = "Norwegian Bokmål" deliberate? I'm guessing so given the arrangements in patch #3463, which only include languages on this list in the Windows installer; I think nb.

[Freeciv-Dev] [bug #20006] Server option "-a" help says "-D" required, but omitting one is not considered error

2012-07-31 Thread Jacob Nevins
Update of bug #20006 (project freeciv): Planned Release: => 2.4.0,2.5.0 ___ Follow-up Comment #1: True. On the other hand, it's useless to specify -a without -D, because there's nowhere to get c

[Freeciv-Dev] [bug #20007] script_fcdb_init() calls luascript_destroy(NULL)

2012-07-31 Thread Jacob Nevins
Follow-up Comment #1, bug #20007 (project freeciv): Probably related thing I noticed a while ago: if fcdb is compiled in but not enabled, a command like "/fcdb lua print('Hello')" causes the server to segfault. '/fcdb reload' is not particularly happy either. I looked briefly at fixing it but it w

Re: [Freeciv-Dev] Ancients modpack (was: [task #7606] Setup freeciv 2.4 files in modpack.freeciv.org)

2012-07-31 Thread Marko Lindqvist
On 1 August 2012 02:14, Jacob Nevins <0jacobnk.fc...@chiark.greenend.org.uk> wrote: > Marko Lindqvist writes: >> I actually would like to hear opinions about Ancients ruleset's >> status. Should it be in default modpack list? > > I've been meaning to ask about that. READMEs that are around >

[Freeciv-Dev] [bug #20005] Compile time decided database.lua location & Windows installations

2012-07-31 Thread Jacob Nevins
Update of bug #20005 (project freeciv): Planned Release: => 2.4.0,2.5.0 ___ Reply to this item at: ___ Message sent

[Freeciv-Dev] [patch #3470] Document use of --with-followtag in README.packaging

2012-07-31 Thread Jacob Nevins
Update of patch #3470 (project freeciv): Status: In Progress => Ready For Test ___ Additional Item Attachment: File name: trunk-S2_4-followtag-doc.patch Size:1 KB ___

[Freeciv-Dev] [bug #20014] References to metaserver.phtml

2012-07-31 Thread Jacob Nevins
Follow-up Comment #1, bug #20014 (project freeciv): The metaserver.phtml URL is also going to persist in people's config files for a long time, unless we do something special to rewrite it to the magic value "default". ___ Reply to this ite

[Freeciv-Dev] [bug #20014] References to metaserver.phtml

2012-07-31 Thread Jacob Nevins
Update of bug #20014 (project freeciv): Status: In Progress => Ready For Test ___ Additional Item Attachment: File name: trunk-metaserver-phtml-references.patch Size:2 KB File name: S2_4-metaserver-ph

[Freeciv-Dev] [patch #3470] Document use of --with-followtag in README.packaging

2012-07-31 Thread Jacob Nevins
URL: Summary: Document use of --with-followtag in README.packaging Project: Freeciv Submitted by: jtn Submitted on: Wed Aug 1 01:28:38 2012 Category: docs Priority: 5 - Normal

[Freeciv-Dev] [patch #3462] Separate metaserver version tag for Windows installer builds

2012-07-31 Thread Marko Lindqvist
Follow-up Comment #2, patch #3462 (project freeciv): I think single "win32" should mean latest stable release(1). If we had 2.4 releases following 2.4 specific tag, they would never inform users about 2.5 series existence. I have added branch specific "S2_4" and "S2_5" tags to metaserver side, bu

[Freeciv-Dev] [bug #20014] References to metaserver.phtml

2012-07-31 Thread Jacob Nevins
URL: Summary: References to metaserver.phtml Project: Freeciv Submitted by: jtn Submitted on: Wed Aug 1 01:23:54 2012 Category: docs Severity: 2 - Minor Priorit

[Freeciv-Dev] [bug #20003] Security advisory

2012-07-31 Thread Marko Lindqvist
Follow-up Comment #9, bug #20003 (project freeciv): > (What seems odd is that the exploit seems to send many 0xff's, > and I would have expected the opposite) Problem is that in error situation - when there's no more data - dio_get_uint8() returns 0, not 255. So if there's not enough data, it wil

[Freeciv-Dev] [patch #3462] Separate metaserver version tag for Windows installer builds

2012-07-31 Thread Jacob Nevins
Follow-up Comment #1, patch #3462 (project freeciv): Do we want a "win32-S2_4" tag for 2.4.0-betaX/rcX, rather than plain "win32"? When 2.5.0-beta1 comes round, we presumably want all the 2.4.x Windows builds to display the current stable Win32 release, not the betas. Is that what "win32" will me

[Freeciv-Dev] [bug #17175] Tileset priority not respected

2012-07-31 Thread Jacob Nevins
Update of bug #17175 (project freeciv): Status:None => Duplicate Open/Closed:Open => Closed ___ Follow-up Comment #1: Same issue as bug #

[Freeciv-Dev] [bug #20008] Tileset priority not respected

2012-07-31 Thread Jacob Nevins
Follow-up Comment #2, bug #20008 (project freeciv): See also bug #17175. ___ Reply to this item at: ___ Message sent via/by Gna! http://gna.org/

[Freeciv-Dev] Ancients modpack (was: [task #7606] Setup freeciv 2.4 files in modpack.freeciv.org)

2012-07-31 Thread Jacob Nevins
Marko Lindqvist writes: > I actually would like to hear opinions about Ancients ruleset's > status. Should it be in default modpack list? I've been meaning to ask about that. READMEs that are around suggest that Dan

[Freeciv-Dev] [patch #3469] Get rid of most deprecated warning

2012-07-31 Thread RafałMużyło
Follow-up Comment #2, patch #3469 (project freeciv): Thank you, but I'd rather not. First of all, the list of deprecation warnings is a bit incomplete, as we're targeting 3.4 and the list covers only 3.2 (due to a quirk of how are bounds for those warnings defined). Second, related to first, wit

[Freeciv-Dev] [patch #3469] Get rid of most deprecated warning

2012-07-31 Thread mir3x
Follow-up Comment #1, patch #3469 (project freeciv): I missed something before gdk_device_ungrab in 2 occurences. (file #16255) ___ Additional Item Attachment: File name: get_rid_gtk3_deprecated.patch.gz Size:12 KB _

[Freeciv-Dev] [patch #3456] Add language names list

2012-07-31 Thread Marko Lindqvist
Update of patch #3456 (project freeciv): Status: Ready For Test => Done Assigned to:None => cazfi Open/Closed:Open => Closed

[Freeciv-Dev] [patch #3469] Get rid of most deprecated warning

2012-07-31 Thread mir3x
URL: Summary: Get rid of most deprecated warning Project: Freeciv Submitted by: mir3x Submitted on: Wed 01 Aug 2012 12:22:57 AM CEST Category: client-gtk-3.0 Priority: 5 - Norma

Re: [Freeciv-Dev] [task #7606] Setup freeciv 2.4 files in modpack.freeciv.org

2012-07-31 Thread David Lowe
On 2012 Jul 31, at 2:38 PM, Marko Lindqvist wrote: > I actually would like to hear opinions about Ancients ruleset's status. Should > it be in default modpack list? In my extremely personal opinion, i didn't enjoy it when i tried. Most likely it came down to the tech tree being so very

[Freeciv-Dev] [bug #20013] utility/support.c:688:7: warning: variable 'r' set but not used

2012-07-31 Thread Marko Lindqvist
URL: Summary: utility/support.c:688:7: warning: variable 'r' set but not used Project: Freeciv Submitted by: cazfi Submitted on: Wed 01 Aug 2012 01:21:27 AM EEST Category: bootstrap

[Freeciv-Dev] [patch #3468] Metaticket: Clang clean build

2012-07-31 Thread Marko Lindqvist
Update of patch #3468 (project freeciv): Planned Release: => 2.5.0 ___ Follow-up Comment #1: I don't think anybody wants to fight warnings from lua-5.1 and friends when we hopefully get lu

[Freeciv-Dev] [patch #3468] Metaticket: Clang clean build

2012-07-31 Thread Marko Lindqvist
URL: Summary: Metaticket: Clang clean build Project: Freeciv Submitted by: cazfi Submitted on: Wed 01 Aug 2012 12:59:27 AM EEST Category: bootstrap Priority: 5 - Normal

[Freeciv-Dev] [task #7606] Setup freeciv 2.4 files in modpack.freeciv.org

2012-07-31 Thread Marko Lindqvist
Follow-up Comment #2, task #7606 (project freeciv): modpack.list and .modpack files are already on modpack.freeciv.org, but latter still point to actual files elsewhere. I hope to move those currently hosted at cazfi.net to modpack.freeciv.org before beta1 - but that's not a blocker as everything

[Freeciv-Dev] [task #7606] Setup freeciv 2.4 files in modpack.freeciv.org

2012-07-31 Thread Jacob Nevins
Follow-up Comment #1, task #7606 (project freeciv): Is there much to do here? I see Alien isn't on modpack.freeciv.org; anything else? (BTW, I changed the "version" of the 2.4 Amplio modpack on modpack.freeciv.org from "2.3.99-alpha" to "2.4-svn", since it always points directly into the latest v

[Freeciv-Dev] [patch #3467] README.rulesets: Do not claim already lifted limitations to exist

2012-07-31 Thread Marko Lindqvist
URL: Summary: README.rulesets: Do not claim already lifted limitations to exist Project: Freeciv Submitted by: cazfi Submitted on: Tue 31 Jul 2012 10:33:29 PM EEST Category: docs

[Freeciv-Dev] [bug #17843] City radius effects work only with city size requirements

2012-07-31 Thread Marko Lindqvist
Follow-up Comment #10, bug #17843 (project freeciv): > Did you test if the savegame still varies if you do a second > load/save cycle? It's not temporary difference at saving time - same autogame run 200 turns longer with the patch. Still, I don't think this is necessarily a bug. Since city gets

[Freeciv-Dev] [patch #3466] Include direct.h to shared.c for _mkdir() for Windows builds

2012-07-31 Thread Marko Lindqvist
URL: Summary: Include direct.h to shared.c for _mkdir() for Windows builds Project: Freeciv Submitted by: cazfi Submitted on: Tue 31 Jul 2012 06:17:14 PM EEST Category: bootstrap

[Freeciv-Dev] [patch #3356] --enable-crosser

2012-07-31 Thread Marko Lindqvist
Update of patch #3356 (project freeciv): Status: Ready For Test => Done Assigned to:None => cazfi Open/Closed:Open => Closed

[Freeciv-Dev] [bug #20003] Security advisory

2012-07-31 Thread Patrick Welche
Follow-up Comment #3, bug #20003 (project freeciv): Thank you for your patch which fixes part A]. As to part B], it seems that the infinite loop comes from this part of common/generate_packets.py: 544 else: 545 return ''' 546 for (;;) { 547int i; 548

[Freeciv-Dev] [patch #3449] "Nation sets": initial cut-down implementation for 2.4.x

2012-07-31 Thread Jacob Nevins
Update of patch #3449 (project freeciv): Status: Ready For Test => Done Open/Closed:Open => Closed ___ Follow-up Comment #11: Try declaring thi

[Freeciv-Dev] [bug #20001] Some colours (e.g., border colours on map) are wrong

2012-07-31 Thread Jacob Nevins
Update of bug #20001 (project freeciv): Status: Ready For Test => Fixed Open/Closed:Open => Closed ___ Reply to this item at: