Update of patch #1189 (project freeciv):
Status: Ready For Test = Done
Open/Closed:Open = Closed
___
Reply to this item at:
Update of patch #1189 (project freeciv):
Status: In Progress = Ready For Test
___
Additional Item Attachment:
File name: trunk_airlifting_style.diffSize:9 KB
Follow-up Comment #10, patch #1189 (project freeciv):
I tested the patch (see attached savefile). Within the gtk client airlifting
between allied cities is not possible. The airlift value for the allied
cities is not known by the client so these cities are not selected as
possible airlift
Follow-up Comment #11, patch #1189 (project freeciv):
I think this should be fixed in a bug report after this patch is committed.
The client issues are not taken in account in this patch.
Also, I forgot to mention: applying this patch need to update the capability
string.
Follow-up Comment #7, patch #1189 (project freeciv):
I have tried to update this patch to a version which would be accepted (add a
setting; save it in the savefile; ...). One (big) problem occurs:
For airlifting of a unit to an allied city the player has to know if this
city has an airport and
Follow-up Comment #8, patch #1189 (project freeciv):
I was thinking about a solution like it is implemented in warclient.
___
Reply to this item at:
http://gna.org/patch/?1189
___
Message
Update of patch #1189 (project freeciv):
Category:None = general
Status:None = In Progress
Assigned to:None = pepeto
Planned Release:
Follow-up Comment #5, patch #1189 (project freeciv):
Don't know how that happened. Fixed.
(file #6348)
___
Additional Item Attachment:
File name: airlift_allied_city_3.patchSize:0 KB
Follow-up Comment #2, patch #1189 (project freeciv):
From looking at the procedure ai_airlift in /ai/aiunit.c, it seems the AI
only iterates through its own cities when it looks for a place to airlift a
unit too. Therefore I do not believe that we have to worry about the AI
eating up a user's
Follow-up Comment #3, patch #1189 (project freeciv):
Patch that fixes problems I raised earlier. The earlier patch can be ignored.
I'm not sure whether the line:
(city_owner(acity) != city_owner(pcity)
should be kept or not as the next line:
(!pplayers_allied (city_owner(pcity),
Follow-up Comment #1, patch #1189 (project freeciv):
One problem I see with this is that ally may end using the valuable airlift
capacity of your city you would need yourself. Human allies can of course
coordinate things via chat but AI allies are problem. It's already a problem
that they are
URL:
http://gna.org/patch/?1189
Summary: [Patch] Allow airlifts to allied cities
Project: Freeciv
Submitted by: duskeagle
Submitted on: Sunday 07/12/2009 at 07:31
Category: None
Priority: 3 - Low
12 matches
Mail list logo