[Freeciv-Dev] [patch #4279] TerrainAlter requirement to have any extra as target

2013-10-29 Thread Marko Lindqvist 
Follow-up Comment #2, patch #4279 (project freeciv):

> Once we fix is_native_tile_to_extra() to handle extras that are
> specials like it does for bases and roads, things may change a
> bit

Going forward this model has problem of requiring cardinally adjacent
irrigation to city center for it to get irrigation bonus, while currently
that's not the case.

___

Reply to this item at:

  

___
  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 #4279] TerrainAlter requirement to have any extra as target

2013-10-28 Thread Marko Lindqvist 
Update of patch #4279 (project freeciv):

  Status: In Progress => Ready For Test 

___

Follow-up Comment #1:

Rulesets currently work as they are. Once we fix is_native_tile_to_extra() to
handle extras that are specials like it does for bases and roads, things may
change a bit (for example currently "Farmland" alteration would be considered
possible even if tile has no existign "Irrigation")

___

Reply to this item at:

  

___
  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 #4279] TerrainAlter requirement to have any extra as target

2013-10-28 Thread Marko Lindqvist 
Update of patch #4279 (project freeciv):

  Status:  Ready For Test => In Progress


___

Reply to this item at:

  

___
  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 #4279] TerrainAlter requirement to have any extra as target

2013-10-28 Thread Marko Lindqvist 
URL:
  

 Summary: TerrainAlter requirement to have any extra as target
 Project: Freeciv
Submitted by: cazfi
Submitted on: Mon 28 Oct 2013 03:02:18 PM EET
Category: general
Priority: 5 - Normal
  Status: Ready For Test
 Privacy: Public
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 2.6.0

___

Details:

Change TerrainAlter requirement to check possibility of adding any kind of
extra to the tile instead of just one of the hardcoded ones.



___

File Attachments:


---
Date: Mon 28 Oct 2013 03:02:18 PM EET  Name: TerrAltExtra.patch  Size: 8kB  
By: cazfi



___

Reply to this item at:

  

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


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