[Freeciv-Dev] [patch #1792] Move ai function pointer setting to ai code

2010-08-03 Thread Marko Lindqvist
Update of patch #1792 (project freeciv): Status: Ready For Test = Done Assigned to:None = cazfi Open/Closed:Open = Closed

[Freeciv-Dev] [bug #16348] Observing immediately crashes gtk2 client

2010-08-03 Thread Marko Lindqvist
Update of bug #16348 (project freeciv): Assigned to:None = cazfi ___ Follow-up Comment #2: As crasher bugs, I'll commit these fixes as soon as possible.

[Freeciv-Dev] [bug #16352] Client does not return to start page

2010-08-03 Thread Marko Lindqvist
URL: http://gna.org/bugs/?16352 Summary: Client does not return to start page Project: Freeciv Submitted by: cazfi Submitted on: Tuesday 08/03/2010 at 11:27 Category: client-gtk-2.0 Severity: 3 - Normal

[Freeciv-Dev] [bug #16348] Observing immediately crashes gtk2 client

2010-08-03 Thread Marko Lindqvist
Update of bug #16348 (project freeciv): Category:None = client-gtk-2.0 Status:None = Fixed Open/Closed:Open = Closed

[Freeciv-Dev] [patch #1794] CALL_AI_FUNC macro

2010-08-03 Thread Marko Lindqvist
Follow-up Comment #5, patch #1794 (project freeciv): This will not work for enum unit_move_result (*auto_explorer)(struct unit *punit); but this function will be moved to advisor, or? Yes, it's removed from AI interface in patch #1798 that this ticket depends on.

[Freeciv-Dev] [patch #1801] Call ai_data_analyze_rulesets through AI function pointer interface

2010-08-03 Thread Marko Lindqvist
Follow-up Comment #2, patch #1801 (project freeciv): have a look at patch #1785 (for this one see patch #1789) I know we are doing duplicate work. But our patches have very different dependencies (your patches depend on lower parts of your own patch stack and my patches depend on my patch

Re: [Freeciv-Dev] [patch #1801] Call ai_data_analyze_rulesets through AI function pointer interface

2010-08-03 Thread Matthias Pfafferodt
Quoting Marko Lindqvist no-reply.invalid-addr...@gna.org: have a look at patch #1785 (for this one see patch #1789) I know we are doing duplicate work. But our patches have very different dependencies (your patches depend on lower parts of your own patch stack and my patches depend on my

Re: [Freeciv-Dev] [bug #16352] Client does not return to start page

2010-08-03 Thread Matthias Pfafferodt
Quoting Marko Lindqvist no-reply.invalid-addr...@gna.org: Details: It seems that client never returns to start page. I assume all of thse instances to be the same bug. Easiest way to reproduce: 1. Start client 2. Start New Game 3. Cancel It says Disconnected from server but nothing more

[Freeciv-Dev] [patch #1804] Move advisor movement code from ai to advisors

2010-08-03 Thread Marko Lindqvist
URL: http://gna.org/patch/?1804 Summary: Move advisor movement code from ai to advisors Project: Freeciv Submitted by: cazfi Submitted on: Tuesday 08/03/2010 at 12:50 Category: ai Priority: 5 - Normal

[Freeciv-Dev] [bug #16352] Client does not return to start page

2010-08-03 Thread Marko Lindqvist
Update of bug #16352 (project freeciv): Status:None = Duplicate Assigned to:None = cazfi Open/Closed:Open = Closed

[Freeciv-Dev] [bug #16301] update queue not working with player pointers

2010-08-03 Thread Marko Lindqvist
Update of bug #16301 (project freeciv): Status: Ready For Test = Fixed Assigned to: syntron = cazfi Open/Closed:Open = Closed

[Freeciv-Dev] [patch #1805] Add ai_type_number()

2010-08-03 Thread Marko Lindqvist
URL: http://gna.org/patch/?1805 Summary: Add ai_type_number() Project: Freeciv Submitted by: cazfi Submitted on: Tuesday 08/03/2010 at 14:29 Category: ai Priority: 5 - Normal Status:

[Freeciv-Dev] [patch #1780] [ai] remove dependency of ./server/citytools.c on aiunit.h

2010-08-03 Thread Marko Lindqvist
Follow-up Comment #1, patch #1780 (project freeciv): Please be more verbose with your patch submissions. This should tell us how behavior is changed - it's far from obvious even when reading the patch (and often I don't have time to read patches but can only comment on patch descripitions) It

[Freeciv-Dev] [patch #1781] [ai] remove dependency of ./server/ruleset.c on aiunit.h

2010-08-03 Thread Marko Lindqvist
Follow-up Comment #1, patch #1781 (project freeciv): Instead of calling interface function for just fc_funcs-ai_get(AI_DEFAULT) (or get_ai_type(AI_DEFAULT)) type, you should iterate through all AI types. ___ Reply to this item at:

[Freeciv-Dev] [patch #1782] [ai] move enemies_at() into aiunit.c

2010-08-03 Thread Marko Lindqvist
Update of patch #1782 (project freeciv): Assigned to: syntron = cazfi Summary: [ai] remove dependency of ./server/unittools.c on aiunit.h = [ai] move enemies_at() into aiunit.c

[Freeciv-Dev] [patch #1783] remove aiunit.h from savegame.c and savegame2.c

2010-08-03 Thread Marko Lindqvist
Update of patch #1783 (project freeciv): Assigned to: syntron = cazfi ___ Reply to this item at: http://gna.org/patch/?1783 ___ Message sent

[Freeciv-Dev] [patch #1787] [ai] remove aicity.h from citytools.c

2010-08-03 Thread Marko Lindqvist
Follow-up Comment #1, patch #1787 (project freeciv): You set trade_want to TRADE_WEIGHTING, then immediately to 1. ___ Reply to this item at: http://gna.org/patch/?1787 ___ Message sent

[Freeciv-Dev] [patch #1783] remove aiunit.h from savegame.c and savegame2.c

2010-08-03 Thread Marko Lindqvist
Update of patch #1783 (project freeciv): Status: Ready For Test = Done Open/Closed:Open = Closed ___ Reply to this item at:

[Freeciv-Dev] [patch #1782] [ai] move enemies_at() into aiunit.c

2010-08-03 Thread Marko Lindqvist
Update of patch #1782 (project freeciv): Status: Ready For Test = Done Open/Closed:Open = Closed ___ Reply to this item at:

[Freeciv-Dev] [patch #1794] CALL_AI_FUNC macro

2010-08-03 Thread Marko Lindqvist
Follow-up Comment #6, patch #1794 (project freeciv): * Updated against svn * Renamed macro CALL_AI_FUNC as CALL_PLR_AI_FUNC since we probably want to introduce CALL_AI_FUNC macro that calls ai type functions without player handle (file #9676)

[Freeciv-Dev] [patch #1800] Move AI city building code from autosettlers.c to aisettler.c

2010-08-03 Thread Marko Lindqvist
Update of patch #1800 (project freeciv): Dependency Removed: = patch #1799 ___ Reply to this item at: http://gna.org/patch/?1800 ___ Message sent

[Freeciv-Dev] [patch #1800] Move AI city building code from autosettlers.c to aisettler.c

2010-08-03 Thread Marko Lindqvist
Update of patch #1800 (project freeciv): Depends on: = patch #1794 ___ Follow-up Comment #1: * Updated against changes in patch stack * No longer depends on #1799 as that patch is on hold

Re: [Freeciv-Dev] [patch #1801] Call ai_data_analyze_rulesets through AI function pointer interface

2010-08-03 Thread Marko Lindqvist
On 3 August 2010 12:37, Matthias Pfafferodt free...@mapfa.de wrote: The 'remove *' patches (see patch #1780 to #1790) are as independent as possible. Perhaps you have to move the definition of the new function pointers but else they only try to remove dependencies of the server code on code

[Freeciv-Dev] [patch #1787] [ai] remove aicity.h from citytools.c

2010-08-03 Thread Marko Lindqvist
Follow-up Comment #2, patch #1787 (project freeciv): Doesn't patch #1790 make this one obsolete? No point in moving stuff around in one commit and then removing it completely in next one. ___ Reply to this item at:

[Freeciv-Dev] [patch #1801] Call ai_data_analyze_rulesets through AI function pointer interface

2010-08-03 Thread Marko Lindqvist
Follow-up Comment #3, patch #1801 (project freeciv): * Updated against changes in patch stack (file #9678) ___ Additional Item Attachment: File name: AnalyzeRulesets2AiInterface_1801-2.diff Size:1 KB

[Freeciv-Dev] [patch #1772] [interface] define an interface to call different functions at client and server

2010-08-03 Thread Marko Lindqvist
Follow-up Comment #2, patch #1772 (project freeciv): I quickly looked for files named interface.h on my computer and found quite a lot of those. It would be more robust to name our header file with some other name to avoid problems with inclusion of wrong file. Maybe fcinterface.h

[Freeciv-Dev] [patch #1772] [interface] define an interface to call different functions at client and server

2010-08-03 Thread pepeto
Follow-up Comment #3, patch #1772 (project freeciv): ... or fc_interface.h? ___ Reply to this item at: http://gna.org/patch/?1772 ___ Message posté via/par Gna! http://gna.org/

[Freeciv-Dev] [patch #1790] [ai] remove ai-trade_want as it is not used

2010-08-03 Thread Marko Lindqvist
Update of patch #1790 (project freeciv): Assigned to: syntron = cazfi ___ Follow-up Comment #1: Version against current svn (file #9679)

[Freeciv-Dev] [patch #1787] [ai] remove aicity.h from citytools.c

2010-08-03 Thread Marko Lindqvist
Update of patch #1787 (project freeciv): Status: Ready For Test = Duplicate Assigned to: syntron = cazfi Open/Closed:Open = Closed

[Freeciv-Dev] [patch #1700] Freeciv 2.2.1 patches for Mac OS X 10.6.3

2010-08-03 Thread Marko Lindqvist
Additional Item Attachment, patch #1700 (project freeciv): File name: FcAiDefault_1700.diff Size:1 KB File name: FcAiDefault_1700_S2_2.diff Size:1 KB ___ Reply to this item at: http://gna.org/patch/?1700

[Freeciv-Dev] [patch #1700] Freeciv 2.2.1 patches for Mac OS X 10.6.3

2010-08-03 Thread Marko Lindqvist
Update of patch #1700 (project freeciv): Category:None = general Status:None = In Progress Planned Release: = 2.2.3

[Freeciv-Dev] [bug #16310] Perhaps use ${sysconfdir} instead of ${prefix}/etc in m4/ggz.m4

2010-08-03 Thread Marko Lindqvist
Update of bug #16310 (project freeciv): Assigned to:None = cazfi Planned Release: = 2.2.3 ___ Follow-up Comment #1: As far as I can

[Freeciv-Dev] [patch #1804] Move advisor movement code from ai to advisors

2010-08-03 Thread Marko Lindqvist
Follow-up Comment #1, patch #1804 (project freeciv): Updated against svn and patch stack changes (file #9683) ___ Additional Item Attachment: File name: AdvGoto_1804-2.diffSize:21 KB

[Freeciv-Dev] [patch #1806] Move ai/aiexplorer.[ch] as server/advisors/autoexplorer.[ch]

2010-08-03 Thread Marko Lindqvist
URL: http://gna.org/patch/?1806 Summary: Move ai/aiexplorer.[ch] as server/advisors/autoexplorer.[ch] Project: Freeciv Submitted by: cazfi Submitted on: Tuesday 08/03/2010 at 18:55 Category: ai Priority:

[Freeciv-Dev] [bug #16353] Breakage: Client stuck when server shuts down

2010-08-03 Thread Daniel Markstedt
URL: http://gna.org/bugs/?16353 Summary: Breakage: Client stuck when server shuts down Project: Freeciv Submitted by: dmarks Submitted on: Wednesday 08/04/2010 at 11:10 Category: client-gtk-2.0 Severity:

[Freeciv-Dev] [patch #1777] Swap Amplio for Amplio 2 as default tileset

2010-08-03 Thread Daniel Markstedt
Update of patch #1777 (project freeciv): Status:None = Ready For Test ___ Follow-up Comment #9: Committed to trunk. Here's a patch to remove 'amplio1' - to be committed before 2.3.0 final

[Freeciv-Dev] [bug #16354] Amplio2 - old specials

2010-08-03 Thread Daniel Markstedt
URL: http://gna.org/bugs/?16354 Summary: Amplio2 - old specials Project: Freeciv Submitted by: dmarks Submitted on: Wednesday 08/04/2010 at 12:00 Category: art Severity: 1 - Wish Priority:

[Freeciv-Dev] [bug #16354] Amplio2 - old specials

2010-08-03 Thread Daniel Markstedt
Follow-up Comment #1, bug #16354 (project freeciv): Hogne, if you come up with any new sprites for these, feel free to attach them here! ___ Reply to this item at: http://gna.org/bugs/?16354

[Freeciv-Dev] [bug #16355] civ doesn't work with supplied scenarios

2010-08-03 Thread Lavrentij P Berija
URL: http://gna.org/bugs/?16355 Summary: civ doesn't work with supplied scenarios Project: Freeciv Submitted by: heilkitty Submitted on: Среда 04.08.2010 at 05:13 Category: None Severity: 3 - Normal

[Freeciv-Dev] [bug #16354] Amplio2 - old specials

2010-08-03 Thread Hogne Håskjold
Follow-up Comment #2, bug #16354 (project freeciv): Hi Daniel If you don't mind I would rather just commit updates for Amplio 2 directly to trunk, it's much more convenient. ___ Reply to this item at: http://gna.org/bugs/?16354