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

2010-09-10 Thread Jacob Nevins
Follow-up Comment #23, bug #15559 (project freeciv): Just found time to play a bit more. I did sysctl -w net.ipv6.bindv6only=1 to try to reproduce the problem. Without the patch, starting a server from the client works, but starting a server manually and then trying to connect to it with IPv4

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

2010-09-10 Thread Marko Lindqvist
Follow-up Comment #24, bug #15559 (project freeciv): I did look in the some older version of the patch, and I'm certain it works on linux. My only concern is that it has not been much tested with other systems (I can *try* to find time to make Windows/MinGW build). It needs more exposure before

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

2010-09-10 Thread pepeto
Follow-up Comment #3, patch #1942 (project freeciv): Other things (not so important): * line 127 contains only spaces (necessary) ; * new header comment length usually is 77 characters (to fill the line). ___ Reply to this item at:

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

2010-09-10 Thread Andrzej G.
Follow-up Comment #2, patch #1938 (project freeciv): The Unicode in this ruleset is well displayed by the game. The names of cities and leaders come from wikipedia in the Navajo language. Hastiin is a title reserved for older or very respectable people.

[Freeciv-Dev] [patch #1905] Elamite nation

2010-09-10 Thread Andrzej G.
Follow-up Comment #6, patch #1905 (project freeciv): The Fleur-de-lis is a symbol of the Babylonian trinity of Nimrod, Queen Semiramis and Tammuz, among others. ___ Reply to this item at: http://gna.org/patch/?1905

[Freeciv-Dev] [patch #1929] Gothic nation

2010-09-10 Thread Andrzej G.
Follow-up Comment #8, patch #1929 (project freeciv): Civilwar_nations option should include the nations Alans or Sarmatians rather than Scythians. Scythians disappeared from the plains around the second century AD when comes the Sarmatian tribes on the steppes of Ukraine (probably succumbed to

[Freeciv-Dev] [patch #1929] Gothic nation

2010-09-10 Thread Andrzej G.
Additional Item Attachment, patch #1929 (project freeciv): File name: gepid.ruleset Size:1 KB File name: greuthungi.ruleset Size:2 KB ___ Reply to this item at: http://gna.org/patch/?1929

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

2010-09-10 Thread Daniel Markstedt
Follow-up Comment #3, patch #1938 (project freeciv): Here's a flag that I drew a couple of years ago. (file #10284) ___ Additional Item Attachment: File name: navajo.svg Size:27 KB

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

2010-09-10 Thread Daniel Markstedt
Follow-up Comment #25, bug #15559 (project freeciv): I'd prefer to retarget for 2.2.4. AFAIK it's been like this in all 2.2.x releases so I wouldn't consider it a showstopper. ___ Reply to this item at: http://gna.org/bugs/?15559

[Freeciv-Dev] [patch #1929] Gothic nation

2010-09-10 Thread Andrzej G.
Additional Item Attachment, patch #1929 (project freeciv): File name: greuthungi.ruleset Size:2 KB ___ Reply to this item at: http://gna.org/patch/?1929 ___ Message sent

[Freeciv-Dev] [patch #1929] Gothic nation

2010-09-10 Thread Andrzej G.
Additional Item Attachment, patch #1929 (project freeciv): File name: gepid.ruleset Size:1 KB ___ Reply to this item at: http://gna.org/patch/?1929 ___ Message sent

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

2010-09-10 Thread Matthias Pfafferodt
Follow-up Comment #2, patch #1939 (project freeciv): remove RESET_COUNT; replace it by by 'reset_args_max() + 1' (file #10287) ___ Additional Item Attachment: File name: 20100910-01-trunk-use-specenum-for-reset_args.patch Size:2 KB

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

2010-09-10 Thread Matthias Pfafferodt
Follow-up Comment #1, patch #1941 (project freeciv): remove LIST_COUNT; replace it by by 'list_args_max() + 1' see also patch #1939 (file #10288) ___ Additional Item Attachment: File name: 20100910-02-trunk-use-specenum

[Freeciv-Dev] [patch #1882] Seljuk nation

2010-09-10 Thread Andrzej G.
Follow-up Comment #2, patch #1882 (project freeciv): Now I think that cities should be taken of the nations that lie within the former state of Seljuks who as wandering from the north nomads conquered the alien cities. ___ Reply to this

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

2010-09-10 Thread Matthias Pfafferodt
Follow-up Comment #8, bug #16655 (project freeciv): updated version of the patch with the changes proposed on comment #7 (file #10289) ___ Additional Item Attachment: File name: 20100910-07-trunk-fix-log_normal-messages.patch Size:6 KB

[Freeciv-Dev] [bug #16675] Trident tileset and ocean layers

2010-09-10 Thread Daniel Markstedt
URL: http://gna.org/bugs/?16675 Summary: Trident tileset and ocean layers Project: Freeciv Submitted by: dmarks Submitted on: Saturday 09/11/2010 at 03:36 Category: rulesets Severity: 2 - Minor

[Freeciv-Dev] [patch #1902] Apache nation

2010-09-10 Thread Andrzej G.
Follow-up Comment #6, patch #1902 (project freeciv): Proposition for flag of all Apaches: http://img834.imageshack.us/img834/4999/apache.png ___ Reply to this item at: http://gna.org/patch/?1902

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

2010-09-10 Thread Matthias Pfafferodt
Follow-up Comment #9, bug #16644 (project freeciv): I will check it. I did test it and is working for the case mentioned in bug #14944. If there are no objections I will commit this patch this weekend. ___ Reply to this item at:

[Freeciv-Dev] [bug #16676] unused variables in struct player_ai

2010-09-10 Thread Matthias Pfafferodt
URL: http://gna.org/bugs/?16676 Summary: unused variables in struct player_ai Project: Freeciv Submitted by: syntron Submitted on: Freitag 10.09.2010 um 21:35 Category: general Severity: 3 - Normal

[Freeciv-Dev] [patch #1902] Apache nation

2010-09-10 Thread Andrzej G.
Additional Item Attachment, patch #1902 (project freeciv): File name: apache.ruleset Size:3 KB ___ Reply to this item at: http://gna.org/patch/?1902 ___ Message sent

[Freeciv-Dev] [bug #16677] take into account the gold upkeep for units paid by the cityies

2010-09-10 Thread Matthias Pfafferodt
: 2.2.4,2.3.0 ___ Details: ___ File Attachments: --- Date: Freitag 10.09.2010 um 21:48 Name: 20100910-09-trunk-take-into-account

[Freeciv-Dev] [bug #16413] Gold upkeep and Military unhappiness

2010-09-10 Thread Matthias Pfafferodt
Follow-up Comment #17, bug #16413 (project freeciv): I have to check this but if it is the case, the AI has to learn to take into account all the over expenses could you test the patch for bug #16677? ___ Reply to this item at:

[Freeciv-Dev] [bug #16413] Gold upkeep and Military unhappiness

2010-09-10 Thread Matthias Pfafferodt
Update of bug #16413 (project freeciv): Planned Release: 2.3.0 = 2.2.4, 2.3.0 ___ Reply to this item at: http://gna.org/bugs/?16413 ___ Nachricht

[Freeciv-Dev] [bug #16413] Gold upkeep and Military unhappiness

2010-09-10 Thread Matthias Pfafferodt
Update of bug #16413 (project freeciv): Depends on: = bugs #16677 ___ Reply to this item at: http://gna.org/bugs/?16413 ___ Nachricht

[Freeciv-Dev] [bug #16677] take into account the gold upkeep for units paid by the cityies

2010-09-10 Thread pepeto
Update of bug #16677 (project freeciv): Status:None = Ready For Test Assigned to:None = pepeto ___ Reply to this item at:

[Freeciv-Dev] [bug #16676] unused variables in struct player_ai

2010-09-10 Thread pepeto
Follow-up Comment #1, bug #16676 (project freeciv): They looks like old fields. _prev_gold_ has been added in initial revision (rev #77), and _est_upkeep_ at revision #1821. I guess we can remove them. ___ Reply to this item at:

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

2010-09-10 Thread Matthias Pfafferodt
Follow-up Comment #1, bug #16665 (project freeciv): update patch; changes: really use an int for save_hp ... (file #10292) ___ Additional Item Attachment: File name: 20100908-06-trunk-update-hit-point-calculation-for-killunhomed.patch

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

2010-09-10 Thread Matthias Pfafferodt
Update of patch #1936 (project freeciv): Status:None = In Progress ___ Follow-up Comment #11: untested patch: limit units with gold upkeep support by a city if upkeep is paid in a lumped

[Freeciv-Dev] [bug #16672] civilwar_nations for city names

2010-09-10 Thread pepeto
Follow-up Comment #1, bug #16672 (project freeciv): Looking at the code. Actually, it chooses the cities in: 0 parent nations; 0 civil war nations; 0 all remaining nations. I guess what you request is to invert the order of the two first items?

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

2010-09-10 Thread pepeto
Follow-up Comment #2, bug #16668 (project freeciv): I like the idea, and the comment by HanduMan also makes sense: how the client will guess what tileset to load? Maybe should we move the tileset loading when the client receive the packet_map_info?

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

2010-09-10 Thread pepeto
Follow-up Comment #2, bug #16667 (project freeciv): (Obviously ruleset names etc can't change, I'm only talking about user-visible text.) I think rulesets can be modified also in trunk, there shouldn't break anything (I did a quick grep on harbour, I didn't find any reference to the

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

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

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

2010-09-10 Thread pepeto
Update of bug #16665 (project freeciv): Status:None = Ready For Test Assigned to:None = pepeto ___ Reply to this item at:

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

2010-09-10 Thread James Spahlinger
Follow-up Comment #4, patch #1942 (project freeciv): Alright, new patch attached fixing comment header length to 77 chars. I copied from the header for ai_airlift which is 75. I left the empty line on line 127 as it separates the variable initialization for that block from the use of it. This

[Freeciv-Dev] [bug #16662] Some more string corrections

2010-09-10 Thread pepeto
Update of bug #16662 (project freeciv): Assigned to:None = jtn ___ Reply to this item at: http://gna.org/bugs/?16662 ___ Message posté

[Freeciv-Dev] [bug #16677] take into account the gold upkeep for units paid by the cityies

2010-09-10 Thread Matthias Pfafferodt
Follow-up Comment #1, bug #16677 (project freeciv): ups; the gold upkeep was accounted for. So this is now only a cleanup in city.c ;-) The AI needs to know about the gold needed for the units in aihand.c. Please test this patch. (file #10295)

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

2010-09-10 Thread James Spahlinger
Follow-up Comment #5, patch #1942 (project freeciv): Bah, the empty line had spaces in it, removed those 4 in this patch. I read that as empty line. ;) (file #10296) ___ Additional Item Attachment: File name:

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

2010-09-10 Thread pepeto
Follow-up Comment #5, patch #1910 (project freeciv): Taking in account your comment... (file #10297) ___ Additional Item Attachment: File name: trunk_ignore_command2.diff Size:23 KB

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

2010-09-10 Thread Christian Prochaska
Update of bug #15347 (project freeciv): Status: In Progress = Fixed Open/Closed:Open = Closed ___ Reply to this item at:

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

2010-09-10 Thread pepeto
Update of patch #1942 (project freeciv): Status:None = Ready For Test Assigned to:None = pepeto ___ Reply to this item at:

[Freeciv-Dev] [bug #16657] Server segfault in pf_fuel_map_get_path()

2010-09-10 Thread pepeto
Update of bug #16657 (project freeciv): Status:None = Ready For Test Assigned to:None = pepeto Planned Release: = 2.2.4, 2.3.0

[Freeciv-Dev] [bug #16657] Server segfault in pf_fuel_map_get_path()

2010-09-10 Thread pepeto
Update of bug #16657 (project freeciv): Category: general = ai ___ Reply to this item at: http://gna.org/bugs/?16657 ___ Message posté

[Freeciv-Dev] [bug #16656] Arithmetic exception in improvement_effect_value()

2010-09-10 Thread pepeto
Follow-up Comment #2, bug #16656 (project freeciv): This points out an other problem about _team_pooled_research_. It assumes the research is shared by the team. ___ Reply to this item at: http://gna.org/bugs/?16656

[Freeciv-Dev] [bug #16656] Arithmetic exception in improvement_effect_value()

2010-09-10 Thread pepeto
Update of bug #16656 (project freeciv): Status:None = Ready For Test ___ Follow-up Comment #3: My proposal... (file #10299) ___

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

2010-09-10 Thread pepeto
Update of bug #16655 (project freeciv): Category:None = general Status:None = Ready For Test Assigned to:None = pepeto

[Freeciv-Dev] [bug #16678] /take /observe and /detach messages are not saved into the event cache

2010-09-10 Thread pepeto
URL: http://gna.org/bugs/?16678 Summary: /take /observe and /detach messages are not saved into the event cache Project: Freeciv Submitted by: pepeto Submitted on: vendredi 10.09.2010 à 22:19 Category: general

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

2010-09-10 Thread pepeto
Update of bug #15652 (project freeciv): Category:None = general Status:None = Duplicate Assigned to:None = pepeto Open/Closed:

[Freeciv-Dev] [bug #16678] /take /observe and /detach messages are not saved into the event cache

2010-09-10 Thread pepeto
Update of bug #16678 (project freeciv): Status:None = Ready For Test ___ Additional Item Attachment: File name: trunk_S2_2_event_cache_C_OK.diff Size:0 KB

[Freeciv-Dev] [patch #1948] Remove unit_att_rating_sq, replace only use with pow.

2010-09-10 Thread James Spahlinger
URL: http://gna.org/patch/?1948 Summary: Remove unit_att_rating_sq, replace only use with pow. Project: Freeciv Submitted by: nixeagle Submitted on: Friday 09/10/2010 at 22:37 Category: ai Priority: 1 -

[Freeciv-Dev] [patch #1948] Remove unit_att_rating_sq, replace only use with pow.

2010-09-10 Thread James Spahlinger
Follow-up Comment #1, patch #1948 (project freeciv): I ran M-x rgrep RET _sq( RET on the working directory and found that the vast majority of functions doing nothing but squaring something lie in aiunit.c. The one replacement with pow() does not exist in a loop and thus really should not be a

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

2010-09-10 Thread J.M. Maalderink
Follow-up Comment #4, patch #1938 (project freeciv): I attached a screenshot of how the ruleset looks like on my computer Do you perhaps use a different program to make rulesets? Most of your recent nations won't open well, I never had that problem with your earlier nations. Daniel, could you

[Freeciv-Dev] [patch #1929] Gothic nation

2010-09-10 Thread J.M. Maalderink
Follow-up Comment #9, patch #1929 (project freeciv): I'll add your suggestions for the civilwar_nations. Making a separate Gothic nation in addition to the Visigoths and Ostrogoths is OK too, we should just have them conflict with each other. But I'm not really sure about using modern day

[Freeciv-Dev] [bug #16672] civilwar_nations for city names

2010-09-10 Thread J.M. Maalderink
Follow-up Comment #2, bug #16672 (project freeciv): Yes. How does it work? It first exhausts 1, then 2 and then 3? Because I am under the impression that at the moment it picks cities from both parent and civil war nations after it has run out of its own cities.

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

2010-09-10 Thread David Fernandez
Follow-up Comment #12, patch #1936 (project freeciv): I'm glad you implemented the population limit, I'll test it. Although I still think the Trade limit was a good alternative. With gold_upkeep_style = 0 a city can be the home for more units than his gold production, as long as your global

[Freeciv-Dev] [patch #1905] Elamite nation

2010-09-10 Thread J.M. Maalderink
Update of patch #1905 (project freeciv): Status: In Progress = Ready For Test ___ Reply to this item at: http://gna.org/patch/?1905 ___ Message sent

[Freeciv-Dev] [bug #16677] take into account the gold upkeep for units paid by the cityies

2010-09-10 Thread David Fernandez
Follow-up Comment #2, bug #16677 (project freeciv): Testing it. Just let me explain my perception of AI behavior BEFORE this patch (playing with gold upkeep for every unit): - gold_upkeep_style = 2: the AI seems to build improvements and units with no restrictions, and they reach negative

[Freeciv-Dev] [patch #1912] Mazovian nation

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

[Freeciv-Dev] [bug #16413] Gold upkeep and Military unhappiness

2010-09-10 Thread David Fernandez
Follow-up Comment #18, bug #16413 (project freeciv): Ooops, while I was testing the other patch I have noticed a bug in this one. With gold_upkeep_style = 2, it is possible to reach negative treasure! In my test, there were no buildings to sell, and only one unit was disbanded. Note I have seen

[Freeciv-Dev] [patch #1897] Imaginary nation group

2010-09-10 Thread David Fernandez
Follow-up Comment #2, patch #1897 (project freeciv): Now that there are so many nations, it would be very useful some kind of Custom group where you can pick manually the nations you want to see in your games. ___ Reply to this item at:

[Freeciv-Dev] [patch #1914] Improved P'urhepecha ruleset

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

[Freeciv-Dev] [bug #16413] Gold upkeep and Military unhappiness

2010-09-10 Thread David Fernandez
Follow-up Comment #19, bug #16413 (project freeciv): Oh, true, the savegame is useless without my changes to experimental ruleset... you may trust me treasure can become negative with this patch. (there is no way to edit my posts nor my attaches, doesn't it?)