[Freeciv-Dev] [bug #18084] Winning, Losing, and Points

2012-10-06 Thread Davide Baldini
Follow-up Comment #3, bug #18084 (project freeciv):

Yes, with patch #3346, in case endturn is reached, the team with highest score
wins the game.

___

Reply to this item at:

  http://gna.org/bugs/?18084

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #3528] Remove unused TOLUA_AS_DEPENDENCY

2012-10-06 Thread Marko Lindqvist
Update of patch #3528 (project freeciv):

  Status:  Ready For Test = Done   
 Assigned to:None = cazfi  
 Open/Closed:Open = Closed 


___

Reply to this item at:

  http://gna.org/patch/?3528

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #3532] Allow usage of adv/ai data in other player's phase

2012-10-06 Thread Marko Lindqvist
URL:
  http://gna.org/patch/?3532

 Summary: Allow usage of adv/ai data in other player's phase
 Project: Freeciv
Submitted by: cazfi
Submitted on: Sat 06 Oct 2012 05:58:29 PM EEST
Category: ai
Priority: 5 - Normal
  Status: Ready For Test
 Privacy: Public
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 2.4.0, 2.5.0

___

Details:

With alternating movement (phasemode != concurrent) diplomacy etc may try to
access data of the other players while it's not their phase. We currently
assert against any out-of-phase adv/ai data access. This patch allows access
in case of alternating movement.



___

File Attachments:


---
Date: Sat 06 Oct 2012 05:58:29 PM EEST  Name: AltDataPhasesAssertFix.patch 
Size: 2kB   By: cazfi

http://gna.org/patch/download.php?file_id=16649

___

Reply to this item at:

  http://gna.org/patch/?3532

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #19030] postgresql include with hardcoded path

2012-10-06 Thread Marko Lindqvist
Update of bug #19030 (project freeciv):

Category:None = bootstrap  
  Status:None = Ready For Test 
Operating System:*BSD = Any
 Summary: gmake of trunk on OpenBSD includes postgresql with
wrong path = postgresql include with hardcoded path

___

Follow-up Comment #2:

Fix attached

We already have also correct include path from pg_config in CPPFLAGS but
source file had include relative to upper level directory with hardcoded
subdirectory name.

___

Reply to this item at:

  http://gna.org/bugs/?19030

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #19030] postgresql include with hardcoded path

2012-10-06 Thread Marko Lindqvist
Additional Item Attachment, bug #19030 (project freeciv):

File name: PgPath.patch   Size:0 KB


___

Reply to this item at:

  http://gna.org/bugs/?19030

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #19926] WORDS_BIGENDIAN MacOS X universal binaries

2012-10-06 Thread Marko Lindqvist
Follow-up Comment #2, bug #19926 (project freeciv):

In that case I would prefer current autoconf provided check instead of rolling
out our own run-time test.

___

Reply to this item at:

  http://gna.org/bugs/?19926

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #20220] Assert fail when losing future tech

2012-10-06 Thread Marko Lindqvist
Update of bug #20220 (project freeciv):

  Status:  Ready For Test = Fixed  
 Assigned to:None = cazfi  
 Open/Closed:Open = Closed 


___

Reply to this item at:

  http://gna.org/bugs/?20220

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #3533] Allow AI to build ocean cities

2012-10-06 Thread Marko Lindqvist
URL:
  http://gna.org/patch/?3533

 Summary: Allow AI to build ocean cities
 Project: Freeciv
Submitted by: cazfi
Submitted on: Sat 06 Oct 2012 09:52:43 PM EEST
Category: ai
Priority: 5 - Normal
  Status: Ready For Test
 Privacy: Public
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 2.5.0

___

Details:

Consider ocean tiles for city placement. This makes AI to build ocean cities
when possible (alien ruleset). That's beneficial for the AI actually building
the city, but note that it has no clue how to attack oceanic enemy cities.



___

File Attachments:


---
Date: Sat 06 Oct 2012 09:52:43 PM EEST  Name: AiSeaCities.patch  Size: 755B  
By: cazfi

http://gna.org/patch/download.php?file_id=16651

___

Reply to this item at:

  http://gna.org/patch/?3533

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #17816] 1: City size 4, citizen count 5 for Nassau

2012-10-06 Thread Marko Lindqvist
Follow-up Comment #8, bug #17816 (project freeciv):

Again reproducible in r21878. Backtrace seems similar to what I explained in
comment #5. Turn 520.

(file #16652)
___

Additional Item Attachment:

File name: regression.servSize:0 KB


___

Reply to this item at:

  http://gna.org/bugs/?17816

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #17816] 1: City size 4, citizen count 5 for Nassau

2012-10-06 Thread Marko Lindqvist
Follow-up Comment #9, bug #17816 (project freeciv):

r21878 of *S2_4*

___

Reply to this item at:

  http://gna.org/bugs/?17816

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #20219] Lua linked against readline even when freeciv configured with --without-readline

2012-10-06 Thread Marko Lindqvist
Update of bug #20219 (project freeciv):

  Status:  Ready For Test = Fixed  
 Assigned to:None = cazfi  
 Open/Closed:Open = Closed 


___

Reply to this item at:

  http://gna.org/bugs/?20219

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #20224] Double click on unit on gtk3 unit selection dialog crashes client

2012-10-06 Thread Marko Lindqvist
URL:
  http://gna.org/bugs/?20224

 Summary: Double click on unit on gtk3 unit selection dialog
crashes client
 Project: Freeciv
Submitted by: cazfi
Submitted on: Sun 07 Oct 2012 02:24:41 AM EEST
Category: client-gtk-3.0
Severity: 3 - Normal
Priority: 5 - Normal
  Status: None
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Release: 
 Discussion Lock: Any
Operating System: None
 Planned Release: 2.4.0, 2.5.0

___

Details:

This has probably been introduced by updating gtk3 rather than change in
freeciv codebase, but gtk3 unit selection dialog is currently useless as it
crashes as soon as I try to select unit by double clicking it.




___

Reply to this item at:

  http://gna.org/bugs/?20224

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #3348] move unit popups in citydlg slightly

2012-10-06 Thread Marko Lindqvist
Follow-up Comment #3, patch #3348 (project freeciv):

Let's make this change as is for now, and adjust it for smaller displays only
if there turns out to be need to. 

___

Reply to this item at:

  http://gna.org/patch/?3348

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #14904] gtk-client city dlg production time updates to previous value

2012-10-06 Thread Marko Lindqvist
Follow-up Comment #2, bug #14904 (project freeciv):

This has happened more frequently recently to me.

Might be related to prior worklist adjustments, such as moving item from lower
from the list up as current production.

___

Reply to this item at:

  http://gna.org/bugs/?14904

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #20077] in handle_city_info() [packhand.c::638]: assertion 'citizens_count(pcity) == city_size_get(pcity)' failed.

2012-10-06 Thread Marko Lindqvist
Update of bug #20077 (project freeciv):

Category:None = general

___

Follow-up Comment #1:

Client side backtrace does not help much when it seems like problem is server
sending inconsistent data.

Attached patch adds sanity checking to server side before sending this data.
It will hopefully produce more meaningful logs to hunt this down.
Going to commit to TRUNK only.

(file #16654)
___

Additional Item Attachment:

File name: SizeCitConsistencySrv.patchSize:0 KB


___

Reply to this item at:

  http://gna.org/bugs/?20077

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #20171] no specials on map after retrying mapseed

2012-10-06 Thread Marko Lindqvist
Update of bug #20171 (project freeciv):

  Status:  Ready For Test = Fixed  
 Assigned to:None = cazfi  
 Open/Closed:Open = Closed 


___

Reply to this item at:

  http://gna.org/bugs/?20171

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #3529] Take caravans enroute in the account when deciding if more are needed

2012-10-06 Thread Marko Lindqvist
Update of patch #3529 (project freeciv):

  Status:  Ready For Test = Done   
 Assigned to:None = cazfi  
 Open/Closed:Open = Closed 


___

Reply to this item at:

  http://gna.org/patch/?3529

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #19852] Multiplayer ruleset: can get free techs from Darwin's Voyage multiple times

2012-10-06 Thread Marko Lindqvist
Update of bug #19852 (project freeciv):

Category:None = general
  Status:None = Ready For Test 
 Planned Release: = 2.3.3, 2.4.0, 2.5.0

___

Follow-up Comment #2:

That scripting expects nil return value from give_technology() if player
already has it. That's what our external documentation say.
However function header says that it returns pointer to A_NONE. That's
beginning of the techs array, not NULL. Implementation is according to that
function header.

I just decided that from the conflicting documentation the interpretation that
it should return NULL is correct, and current implementation is a bug.
Attached patch changes it to return NULL. It's rather unlikely that any script
out there could depend on old behavior.

(file #16655, file #16656)
___

Additional Item Attachment:

File name: DarwinFix.patchSize:0 KB
File name: DarwinFix-S2_3.patch   Size:0 KB


___

Reply to this item at:

  http://gna.org/bugs/?19852

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev