[Freeciv-Dev] [patch #1901] Micronesian nation

2010-09-08 Thread J.M. Maalderink
Update of patch #1901 (project freeciv): Status: Ready For Test = Done Open/Closed:Open = Closed ___ Reply to this item at:

[Freeciv-Dev] [bug #16655] Trunk contains not translated log_normal() messages

2010-09-08 Thread Matthias Pfafferodt
Follow-up Comment #4, bug #16655 (project freeciv): Well, I have checked that too, but I doubt it is a good solution. I think some of those messages shouldn't be log_normal() but log_verbose(). I did check the messages and here are my ideas about possible changes of the log levels:

[Freeciv-Dev] [bug #16429] Failed sanitycheck.c: Engineers has activity Railroad, but it can't continue at Forest

2010-09-08 Thread Matthias Pfafferodt
Follow-up Comment #11, bug #16429 (project freeciv): Sorry, but I really think your patch doesn't fix the problem on trunk. If you mean the bug raised by file #10221 it is fixed by the patch (file #10231), e.g. after applying the patch no error message occurs. Have a look into

[Freeciv-Dev] [patch #1925] Numunu nation

2010-09-08 Thread Andrzej G.
Follow-up Comment #5, patch #1925 (project freeciv): Changed names from Numunu to Comanche. (file #10241) ___ Additional Item Attachment: File name: comanche.ruleset Size:1 KB

[Freeciv-Dev] [patch #1938] Navajo nation

2010-09-08 Thread Andrzej G.
URL: http://gna.org/patch/?1938 Summary: Navajo nation Project: Freeciv Submitted by: artaxes Submitted on: Wednesday 09/08/2010 at 16:48 Category: rulesets Priority: 5 - Normal Status:

[Freeciv-Dev] [bug #14717] install for all users

2010-09-08 Thread Christian Prochaska
Update of bug #14717 (project freeciv): Status: Ready For Test = Fixed Open/Closed:Open = Closed ___ Reply to this item at:

[Freeciv-Dev] [bug #16440] Voteinfo bar is too long at the right of the screen

2010-09-08 Thread Jacob Nevins
Follow-up Comment #20, bug #16440 (project freeciv): We could also reducing the label size to 60 characters instead of 80, it should be really enough. Hmm, not sure, assuming you mean (struct voteinfo_bar).label. To pick a random example: _(1/2) Vote 1 by billybob: [failed] set

[Freeciv-Dev] [bug #16520] Voteinfo bar never shown if Merge the message notebook and the map notebook set

2010-09-08 Thread Jacob Nevins
Update of bug #16520 (project freeciv): Status:None = Fixed Open/Closed:Open = Closed Planned Release: 2.2.3 = 2.2.3,2.3.0

[Freeciv-Dev] [bug #16520] Voteinfo bar never shown if Merge the message notebook and the map notebook set

2010-09-08 Thread Jacob Nevins
Update of bug #16520 (project freeciv): Assigned to:None = jtn ___ Reply to this item at: http://gna.org/bugs/?16520 ___ Message sent

[Freeciv-Dev] [bug #15347] Bad drawing and floating point exception with trade route line drawing in SDL client

2010-09-08 Thread Jacob Nevins
Update of bug #15347 (project freeciv): Planned Release: = 2.2.3 ___ Follow-up Comment #19: Testing on S2_2 with your patch, all seems fine now on my machine. It even seems to have

[Freeciv-Dev] [patch #1936] limit the number of units a city can support

2010-09-08 Thread David Lowe
Follow-up Comment #8, patch #1936 (project freeciv): Should this be limited to military units? I can't see why we would need to limit Workers, Caravans, etc. For us informed, do Diplomats count as military units? What should happen if the city size is reduced? Kill random units till the

[Freeciv-Dev] [bug #16655] Trunk contains not translated log_normal() messages

2010-09-08 Thread pepeto
Follow-up Comment #5, bug #16655 (project freeciv): I totally agree with your analysis, I would have done exactly the same. Also, I noticed a translation with Q_() of the meta message that looks very strange, since this string can be set by the user. There do you find this? Using _grep

[Freeciv-Dev] [bug #16429] Failed sanitycheck.c: Engineers has activity Railroad, but it can't continue at Forest

2010-09-08 Thread pepeto
Follow-up Comment #12, bug #16429 (project freeciv): I will check this, thanks. ___ Reply to this item at: http://gna.org/bugs/?16429 ___ Message posté via/par Gna! http://gna.org/

[Freeciv-Dev] [bug #16440] Voteinfo bar is too long at the right of the screen

2010-09-08 Thread pepeto
Follow-up Comment #23, bug #16440 (project freeciv): Hmm, not sure, assuming you mean (struct voteinfo_bar).label. To pick a random example: (1/2) Vote 1 by billybob: [failed] set separatepoles 0 (needs 51% in favor). is 76 characters. Feels like we might be able to slim the button area a

[Freeciv-Dev] [bug #16625] save chat message for the sender

2010-09-08 Thread pepeto
Update of bug #16625 (project freeciv): Status: Ready For Test = Fixed Open/Closed:Open = Closed ___ Reply to this item at:

[Freeciv-Dev] [bug #16639] fix assert in dbv_isset() at server exit

2010-09-08 Thread pepeto
Update of bug #16639 (project freeciv): Status: Ready For Test = Fixed Open/Closed:Open = Closed ___ Reply to this item at:

[Freeciv-Dev] [bug #16642] Saving can be very long

2010-09-08 Thread pepeto
Update of bug #16642 (project freeciv): Status: Ready For Test = Fixed Open/Closed:Open = Closed ___ Reply to this item at:

[Freeciv-Dev] [bug #16645] Server record many entries named savefile.options

2010-09-08 Thread pepeto
Update of bug #16645 (project freeciv): Status: Ready For Test = Fixed Open/Closed:Open = Closed ___ Reply to this item at:

[Freeciv-Dev] [bug #16665] update hit point calculation for killunhomed

2010-09-08 Thread Matthias Pfafferodt
08.09.2010 um 23:26 Name: 20100908-09-trunk-update-hit-point-calculatiuon-for-killunhomed.patch Size: 0B By: syntron http://gna.org/bugs/download.php?file_id=10248 ___ Reply to this item at: http://gna.org/bugs/?16665

[Freeciv-Dev] [bug #16647] Fix trunk common server memory issues reported valgrind

2010-09-08 Thread pepeto
Update of bug #16647 (project freeciv): Status: Ready For Test = Fixed Open/Closed:Open = Closed ___ Reply to this item at:

[Freeciv-Dev] [patch #1936] limit the number of units a city can support

2010-09-08 Thread Matthias Pfafferodt
Update of patch #1936 (project freeciv): Depends on: = bugs #16665 ___ Reply to this item at: http://gna.org/patch/?1936 ___ Nachricht

[Freeciv-Dev] [patch #1936] limit the number of units a city can support

2010-09-08 Thread Matthias Pfafferodt
Follow-up Comment #9, patch #1936 (project freeciv): I can't see why we would need to limit Workers, Caravans, etc. For us informed, do Diplomats count as military units? As they have the flag NonMil the answer is given by the ruleset definition.

[Freeciv-Dev] [patch #1910] Command /ignore for server

2010-09-08 Thread pepeto
Follow-up Comment #3, patch #1910 (project freeciv): Patch attached including the following commands (like in warclient): * /ignore [type=]pattern * /unignore range * /list ignore As I made important string changes and I am not a native-English (and some string already appear strange to me), I

[Freeciv-Dev] [bug #16655] Trunk contains not translated log_normal() messages

2010-09-08 Thread Matthias Pfafferodt
the secfile). I did use for both log_verbose(). (file #10250) ___ Additional Item Attachment: File name: 20100908-09-trunk-fix-log_normal-messages.patch Size:7 KB ___ Reply to this item

[Freeciv-Dev] [bug #16449] Document that international traderoutes don't pay attention to trademindist

2010-09-08 Thread Jacob Nevins
Update of bug #16449 (project freeciv): Open/Closed: Closed = Open Planned Release: 2.3.0 = 2.2.4,2.3.0 ___ Follow-up Comment #7: Re-opening; I plan

[Freeciv-Dev] [patch #1939] use specenum for reset_args

2010-09-08 Thread Matthias Pfafferodt
for the arguments for the reset command ___ File Attachments: --- Date: Mittwoch 08.09.2010 um 23:51 Name: 20100908-02-trunk-use-specenum-for-reset_args.patch Size: 2kB By: syntron

[Freeciv-Dev] [patch #1940] use specenum for help_general_args

2010-09-08 Thread Matthias Pfafferodt
value for the arguments for the help command ___ File Attachments: --- Date: Mittwoch 08.09.2010 um 23:53 Name: 20100908-04-trunk-use-specenum-for-help_general_args.patch Size: 3kB

[Freeciv-Dev] [patch #1941] use specenum for list_args

2010-09-08 Thread Matthias Pfafferodt
for the arguments for the list command I will update the patch after patch #1910 is committed. ___ File Attachments: --- Date: Mittwoch 08.09.2010 um 23:53 Name: 20100908-03-trunk-use

[Freeciv-Dev] [bug #16655] Trunk contains not translated log_normal() messages

2010-09-08 Thread pepeto
Follow-up Comment #7, bug #16655 (project freeciv): savegame2.c: The messages are also only additional output. So they should be log_verbose (loading /saving in format x) or log_debug (time needed to create the secfile). I did use for both log_verbose(). I would prefer personally the

[Freeciv-Dev] [patch #1939] use specenum for reset_args

2010-09-08 Thread pepeto
Follow-up Comment #1, patch #1939 (project freeciv): Defined RESET_COUNT probably can be avoided using reset_args_max() as it's not a static definition. Similar comment can be done on patch #1941. ___ Reply to this item at:

[Freeciv-Dev] [bug #16198] infinite loop in the allocation of starting positions

2010-09-08 Thread Jacob Nevins
Update of bug #16198 (project freeciv): Release: = 2.2.0 ___ Reply to this item at: http://gna.org/bugs/?16198 ___ Message sent

[Freeciv-Dev] [bug #16666] Metamessage translated with Q_()?

2010-09-08 Thread pepeto
Follow-up Comment #1, bug #1 (project freeciv): Fix attached: * Remove the translation mark for the meta message because it never translated. * Remove the usage of Q_() totally wrong here because: ** It is not a qualifier translation. ** We are not sure the string returned by

[Freeciv-Dev] [bug #16666] Metamessage translated with Q_()?

2010-09-08 Thread pepeto
Update of bug #1 (project freeciv): Status:None = Ready For Test ___ Reply to this item at: http://gna.org/bugs/?1 ___ Message posté

[Freeciv-Dev] [bug #15652] Can't meet the Persians

2010-09-08 Thread Jacob Nevins
Follow-up Comment #3, bug #15652 (project freeciv): I wonder if this is somehow related to bug #16467, although I can't see exactly how. ___ Reply to this item at: http://gna.org/bugs/?15652

[Freeciv-Dev] [bug #16644] 1: Saved game contains incomplete map data

2010-09-08 Thread pepeto
Update of bug #16644 (project freeciv): Assigned to:None = syntron ___ Follow-up Comment #5: However, I never found a way to reproduce bug #14944. Could someone ensure it doesn't revert

[Freeciv-Dev] [bug #16667] en_GB strings in Freeciv source code

2010-09-08 Thread Jacob Nevins
URL: http://gna.org/bugs/?16667 Summary: en_GB strings in Freeciv source code Project: Freeciv Submitted by: jtn Submitted on: Wednesday 09/08/10 at 23:37 Category: None Severity: 2 - Minor

[Freeciv-Dev] [bug #16668] Per-topology client tileset preference

2010-09-08 Thread Jacob Nevins
URL: http://gna.org/bugs/?16668 Summary: Per-topology client tileset preference Project: Freeciv Submitted by: jtn Submitted on: Wednesday 09/08/10 at 23:43 Category: client Severity: 1 - Wish

[Freeciv-Dev] [patch #1910] Command /ignore for server

2010-09-08 Thread David Lowe
Follow-up Comment #4, patch #1910 (project freeciv): The grammar looks good to me. I just wish line # 10 were a little more specific: - list ignore\n + list ignored users\n ___ Reply to this item at:

[Freeciv-Dev] [bug #15559] Client can't connect to its locally started server

2010-09-08 Thread George Koehler
Follow-up Comment #18, bug #15559 (project freeciv): I changed the error code in server/sernet.c to try to identify the socket that causes the error. I also remade the trunk patch, so now trunk seems to work. This is the *last* version of the patch from me, unless I discover that I made some

[Freeciv-Dev] [bug #16630] Copyleft notices for contributions

2010-09-08 Thread Daniel Markstedt
Follow-up Comment #3, bug #16630 (project freeciv): Most tilset subdirs on the download server now have a README file that includes the copyleft notice. New tarballs, the ones uploaded Sep this year and forward, will include the notice as well.

[Freeciv-Dev] [bug #16667] en_GB strings in Freeciv source code

2010-09-08 Thread David Lowe
Follow-up Comment #1, bug #16667 (project freeciv): I assume the default language of Freeciv is notionally en_US, so those should be Harbor and honor respectively (unless a native speaker corrects me). That is indeed how we spell these things in California...

[Freeciv-Dev] [patch #1942] refactor first portion of ai_airlift into find_neediest_airlift_city

2010-09-08 Thread James Spahlinger
URL: http://gna.org/patch/?1942 Summary: refactor first portion of ai_airlift into find_neediest_airlift_city Project: Freeciv Submitted by: nixeagle Submitted on: Thursday 09/09/2010 at 03:58 Category: ai