[Freeciv-Dev] [bug #20040] Scenarios (including tutorial) set citymindist=0

2012-08-11 Thread Jacob Nevins
Update of bug #20040 (project freeciv):

  Status:  Ready For Test = Fixed  
 Open/Closed:Open = Closed 


___

Reply to this item at:

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

___
  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 #20040] Scenarios (including tutorial) set citymindist=0

2012-08-09 Thread Jacob Nevins
URL:
  http://gna.org/bugs/?20040

 Summary: Scenarios (including tutorial) set citymindist=0
 Project: Freeciv
Submitted by: jtn
Submitted on: Thu Aug  9 17:57:44 2012
Category: None
Severity: 3 - Normal
Priority: 5 - Normal
  Status: In Progress
 Assigned to: jtn
Originator Email: 
 Open/Closed: Open
 Release: 
 Discussion Lock: Any
Operating System: Any
 Planned Release: 2.4.0,2.5.0

___

Details:

And in S2_4 this means that there are no restrictions on the distances between
cities.

I think this is fallout from patch #2537, where min_dist_bw_cities was
removed, and citymindist=0 ceased to have a special meaning.

Since that will debut in 2.4.0, we should fix it before release, ideally.




___

Reply to this item at:

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

___
  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 #20040] Scenarios (including tutorial) set citymindist=0

2012-08-09 Thread Marko Lindqvist
Follow-up Comment #1, bug #20040 (project freeciv):

I assume that all old savegames (with citymindist of 0) are affected, not only
scenarios.

Quite bad, actually. Has quite a big effect on any attempts to continue old
game is 2.4. Reason to postpone beta1?

___

Reply to this item at:

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

___
  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 #20040] Scenarios (including tutorial) set citymindist=0

2012-08-09 Thread Jacob Nevins
Follow-up Comment #2, bug #20040 (project freeciv):

Blargh. But I'd rather not slip the release as I don't know when I'll get a
clear run at it. I'll see if a quick low-risk fix is possible. Bug #20041.

___

Reply to this item at:

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

___
  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 #20040] Scenarios (including tutorial) set citymindist=0

2012-08-09 Thread Jacob Nevins
Update of bug #20040 (project freeciv):

  Status: In Progress = Ready For Test 

___

Follow-up Comment #3:

Fix attached. Trunk tutorial wasn't affected since the rewrite (patch #3377).

(file #16358, file #16359)
___

Additional Item Attachment:

File name: trunk-citymindist-zero-scenarios.patch Size:1 KB
File name: S2_4-citymindist-zero-scenarios.patch Size:1 KB


___

Reply to this item at:

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

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


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