[Freeciv-Dev] [patch #1235] [Metaticket] variable city radii
Update of patch #1235 (project freeciv): Planned Release: 2.3.0 = 2.4.0 ___ Follow-up Comment #23: change planned release to 2.4.0 ___ Reply to this item at: http://gna.org/patch/?1235 ___ 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 #1235] [Metaticket] variable city radii
Follow-up Comment #22, patch #1235 (project freeciv): And what about the xaw client? This one too ... ___ Reply to this item at: http://gna.org/patch/?1235 ___ 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 #1235] [Metaticket] variable city radii
Update of patch #1235 (project freeciv): Depends on: = bugs #15804 ___ Reply to this item at: http://gna.org/patch/?1235 ___ 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 #1235] [Metaticket] variable city radii
Follow-up Comment #20, patch #1235 (project freeciv): this patch is not closed as a solution for the city dialog of the SDL client ismissing ___ Reply to this item at: http://gna.org/patch/?1235 ___ 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 #1235] [Metaticket] variable city radii
Follow-up Comment #21, patch #1235 (project freeciv): this patch is not closed as a solution for the city dialog of the SDL client ismissing And what about the xaw client? ___ Reply to this item at: http://gna.org/patch/?1235 ___ Message posté via/par Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #1235] [Metaticket] variable city radii
Follow-up Comment #16, patch #1235 (project freeciv): The SDL client is going to need some work... with the default rules and hence traditional city size, the city map on the city dialog is now unusably small. ___ Reply to this item at: http://gna.org/patch/?1235 ___ 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 #1235] [Metaticket] variable city radii
Follow-up Comment #17, patch #1235 (project freeciv): you are right; I did not test it I can compile the sdl client but I did not get it to work. I can't start a game as it does not responce to the mouse. I think it is a configuration / library problem. At the moment I don't know how the city dialog looks like for this client ___ Reply to this item at: http://gna.org/patch/?1235 ___ 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 #1235] [Metaticket] variable city radii
Follow-up Comment #18, patch #1235 (project freeciv): Odd, it's working for me. Here's a screenshot FYI. (file #8716) ___ Additional Item Attachment: File name: trunk_r17182_SDL_city.png Size:471 KB ___ Reply to this item at: http://gna.org/patch/?1235 ___ 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 #1235] [Metaticket] variable city radii
Follow-up Comment #19, patch #1235 (project freeciv): too use it some parts have to be moved. One possibility would be to move the units in their own row: [city map ][buildings] [units ] [food][city info][build now] This would change especially the height of the dialog window. I would also switch food / building status. I did found the code in client/gui-sdl/citydlg.c but need (a lot of?) time to understand it ... ___ Reply to this item at: http://gna.org/patch/?1235 ___ 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 #1235] [Metaticket] variable city radii
Follow-up Comment #15, patch #1235 (project freeciv): This patch series must be rebased again (patch #1529). I will do it today or tomorrow. Is it OK to apply patch 1 to 5 after that? patch #1230 (new savegame format) should be considered in connection with the new savegame format (patch #1541). This way the old format can stay while the changes go into the new one. ___ Reply to this item at: http://gna.org/patch/?1235 ___ 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 #1235] [Metaticket] variable city radii
Follow-up Comment #14, patch #1235 (project freeciv): Patch #1222 introduced an Event Scripting API change and I think it can be avoided. Was it necessary? If possible, the API should be stable, see https://gna.org/bugs/index.php?15587 If needed, the new name can be introduced alongside the old name. The only important thing is to keep the externally visible name stable, so all other things kept constant, I only want to revert this change in server/scripting/api.pkg: -- Tile methods -function Tile:city_exists_within_city_radius(center) - return methods_tile_city_exists_within_city_radius(self, center) +function Tile:city_exists_within_max_city_map(center) + return methods_tile_city_exists_within_max_city_map(self, center) end ___ Reply to this item at: http://gna.org/patch/?1235 ___ Meddelandet skickades via/av Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #1235] [Metaticket] variable city radii
Follow-up Comment #13, patch #1235 (project freeciv): all patches are updated to current trunk ___ Reply to this item at: http://gna.org/patch/?1235 ___ 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 #1235] [Metaticket] variable city radii
Update of patch #1235 (project freeciv): Assigned to:None = syntron ___ Follow-up Comment #12: The savegame changes are available in the following patches: [01/03] patch #1479: use cm_result_from_main_map() as wrapper for cm_result_copy() [02/03] patch #1480: remove pcity-city_map [03/03] patch #1230: save worked tiles from main map I did miss patch #1229: use new logging in cm_print_result() in the list in comment #11 I plan to wait till after patch #1476 is applied; after that I will rebase all of these patches and would like to discuss (and apply) them. ___ Reply to this item at: http://gna.org/patch/?1235 ___ 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 #1235] [Metaticket] variable city radii
Update of patch #1235 (project freeciv): Depends on: = patch #1479 ___ Reply to this item at: http://gna.org/patch/?1235 ___ 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 #1235] [Metaticket] variable city radii
Update of patch #1235 (project freeciv): Depends on: = patch #1480 ___ Reply to this item at: http://gna.org/patch/?1235 ___ 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 #1235] [Metaticket] variable city radii
Update of patch #1235 (project freeciv): Depends on: = patch #1480 ___ Reply to this item at: http://gna.org/patch/?1235 ___ 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 #1235] [Metaticket] variable city radii
Follow-up Comment #10, patch #1235 (project freeciv): updated summary patch for the variable city radii * bug in savegame loading fixed * remove check for changed city radii from refresh_city() * update debugging code (now as macro in city.h) (file #8031) ___ Additional Item Attachment: File name: 20100209-cityradius_sq.diffSize:124 KB ___ Reply to this item at: http://gna.org/patch/?1235 ___ 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 #1235] [Metaticket] variable city radii
Update of patch #1235 (project freeciv): Depends on: = patch #1232 ___ Reply to this item at: http://gna.org/patch/?1235 ___ 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 #1235] [Metaticket] variable city radii
Update of patch #1235 (project freeciv): Planned Release: = 2.3.0 ___ Reply to this item at: http://gna.org/patch/?1235 ___ 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 #1235] [Metaticket] variable city radii
Follow-up Comment #9, patch #1235 (project freeciv): updated version; changes: * use ftc_server for notify_player() * change new city iterator in ./client/tilespec.c (file #7161) ___ Additional Item Attachment: File name: cityradius_sq20091028.diff.bz2 Size:25 KB ___ Reply to this item at: http://gna.org/patch/?1235 ___ 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 #1235] [Metaticket] variable city radii
Follow-up Comment #8, patch #1235 (project freeciv): About: [10/11] patch #1234 - more than one build slot for units I added the answer to your comment in the discussion to patch #1234 ___ Reply to this item at: http://gna.org/patch/?1235 ___ 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 #1235] [Metaticket] variable city radii
Follow-up Comment #7, patch #1235 (project freeciv): About: [10/11] patch #1234 - more than one build slot for units When I add an Engineer, an Armor and Coinage to a big cities build list then I get 3 Engineers and 2 Armors. That is not what I would expect from the multiple build slots. I would expect it to build an Engineer in the first slot and an Armor in the second and then Coinage. ___ Reply to this item at: http://gna.org/patch/?1235 ___ 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 #1235] [Metaticket] variable city radii
Follow-up Comment #6, patch #1235 (project freeciv): add an 'all in one'-patch (file #6952) ___ Additional Item Attachment: File name: cityradius_sq20091010.diff.gz Size:29 KB ___ Reply to this item at: http://gna.org/patch/?1235 ___ 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 #1235] [Metaticket] variable city radii
Update of patch #1235 (project freeciv): Depends on: = patch #1330 ___ Reply to this item at: http://gna.org/patch/?1235 ___ Message posté via/par Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #1235] [Metaticket] variable city radii
Update of patch #1235 (project freeciv): Planned Release:None = 2.3.0 Summary: variable city radii = [Metaticket] variable city radii ___ Reply to this item at: http://gna.org/patch/?1235 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
