[Freeciv-Dev] [bug #13873] [patch 07/07] load game settings from ruleset file

2009-07-24 Thread Matthias Pfafferodt

Follow-up Comment #4, bug #13873 (project freeciv):

this could be done with an extra line in the
game.ruleset file

[game]
; define the game setting
borders = 2
; lock it (first player is _not_ allowed to change it)
borders.lock = 1

The setting.lock defaults to FALSE. If it is
true, the setting can't be changed. This requires
that the 'rulesetdir' command can't be executed by
the first player.

Does warclient/warserver has such an option?


First I have to solve the main problem. My ideas are:

* 'rulesetdir' command loads the ruleset + settings
  from the game.ruleset file

* The reset command has to restore all settings to the
  default values and load the settings from the
  game.ruleset file. An additional hint to changed
  settings could be added.

* executing 'rulesetdir' again does check if another
  ruleset is requested. If it is not the case, the
  current code will do nothing. This must be
  changed to a reset (= default settings + changes
  from the game.ruleset file). This way one will have
  a well defined state after executing the rulesetdir
  command.


___

Reply to this item at:

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

___
  Nachricht geschickt von/durch Gna!
  http://gna.org/


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


[Freeciv-Dev] [patch #1195] AI: Increase want for trade routes when economy is weak

2009-07-24 Thread Marko Lindqvist

Update of patch #1195 (project freeciv):

Category:None = ai 
  Status:None = Done   
 Assigned to:None = cazfi  
 Open/Closed:Open = Closed 


___

Reply to this item at:

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

___
  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 #1189] [Patch] Allow airlifts to allied cities

2009-07-24 Thread Marko Lindqvist

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 loading units to allied transports so no units of transport
owner fit in.

Why are you only allowing airlift *to* allied city? Shouldn't it be
symmetrically also allowed *from* allied city?

___

Reply to this item at:

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

___
  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 #1188] proposed menu and command key revamp

2009-07-24 Thread Marko Lindqvist

Follow-up Comment #2, patch #1188 (project freeciv):

This patch does not apply cleanly to current menu.c

___

Reply to this item at:

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

___
  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 #13891] [Patch] Chat colors and other improvements

2009-07-24 Thread Marko Lindqvist

Follow-up Comment #3, bug #13891 (project freeciv):

text_tag_replace_text() looks for unit or city id and then determines
city/unit name based on that. This means no name is shown when city/unit is
not known by client. This is a problem. Server should be allowed to send
messages about any city/unit without modifying message (selecting between
plain text and link)  based on each client's knowledge.
Think about wonder started messages.

city_link() and unit_link() do insert city/unit name to link tag. It's just
not used. Note that unit name is not translated in current implementation as
it should be if it's used in client end.

___

Reply to this item at:

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

___
  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 #1188] proposed menu and command key revamp

2009-07-24 Thread John Keller

Follow-up Comment #3, patch #1188 (project freeciv):

Huh, that's odd. menu.c hasn't changed since I made the patches. Maybe I made
an error?

Which patch doesn't work? Did you apply them in order?
(menu.c.01_reorg.diff, menu.c.02_accels_revised.diff, then
helpdata.txt.diff)

menu.c.02_accels.diff (attachment 6196) should be marked as obsolete or
superseded or however Gna marks a patch as old...

I had tried to split the big patch into two logical parts, but maybe it
would be better as a single patch. would you prefer that?

In any case, thanks for taking the time to review it (or at least try ;-) ).

___

Reply to this item at:

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

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


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