[Freeciv-Dev] [bug #18461] crash using lua function civil_war (in ai_plr_data_get() [aidata.c::310]: assertion 'ai-phase_initialized' failed.)

2011-08-09 Thread Matthias Pfafferodt
Follow-up Comment #1, bug #18461 (project freeciv): civil_war() was designed to be called during turn change (ai phase for all players). Now, it can be executed via a lua function at any time. Thus, the ai phase does not have to be initialized. One possible solution would be to check for this if

[Freeciv-Dev] [bug #18461] crash using lua function civil_war (in ai_plr_data_get() [aidata.c::310]: assertion 'ai-phase_initialized' failed.)

2011-08-09 Thread Jacob Nevins
Follow-up Comment #2, bug #18461 (project freeciv): Oh, rats. (Doesn't civil_war() execute mid-turn when you trigger it by taking over a city? Is that already dealt with?) ___ Reply to this item at: http://gna.org/bugs/?18461

[Freeciv-Dev] [bug #18468] 2.3.0 does not compile on OpenBSD 4.9

2011-08-09 Thread Michal Mazurek
Follow-up Comment #2, bug #18468 (project freeciv): I compiled without debug. Freeciv crashes. I should have tested it while it was in RC! note: I get Lan bind failed for any port. System OpenBSD 4.9 on sparc64. [11:22:ttypd][longturn@spock:~/src/freeciv-2.3.0:36]$ ./ser -p This is the

Re: [Freeciv-Dev] [bug #18468] 2.3.0 does not compile on OpenBSD 4.9

2011-08-09 Thread Marko Lindqvist
On 9 August 2011 12:28, Michal Mazurek no-reply.invalid-addr...@gna.org wrote: Follow-up Comment #2, bug #18468 (project freeciv): I compiled without debug. Freeciv crashes. I should have tested it while it was in RC! note: I get Lan bind failed for any port. Port for listening client

Re: [Freeciv-Dev] [bug #18468] 2.3.0 does not compile on OpenBSD 4.9

2011-08-09 Thread Michal Mazurek
On Tue, Aug 09, 2011 at 02:02:30PM +0300, Marko Lindqvist wrote: On 9 August 2011 12:28, Michal Mazurek no-reply.invalid-addr...@gna.org wrote: Follow-up Comment #2, bug #18468 (project freeciv): I compiled without debug. Freeciv crashes. I should have tested it while it was in RC!

Re: [Freeciv-Dev] [patch #2866] A different autosettler fix for 2.3.0-RC2

2011-08-09 Thread Marko Lindqvist
Thank you for your patches. I'll look them as soon as I'm back on my freeciv dev machine (and can login to gna). Please try to have one ticket / issue in the future for making handling these easier. I think I need to split current one to several separate ones so that each ticket can be closed

[Freeciv-Dev] [bug #18461] crash using lua function civil_war (in ai_plr_data_get() [aidata.c::310]: assertion 'ai-phase_initialized' failed.)

2011-08-09 Thread Matthias Pfafferodt
Update of bug #18461 (project freeciv): Status:None = Ready For Test ___ Follow-up Comment #3: I did some tests and I do use a unique setting: civil_war() is called via the lua signal

[Freeciv-Dev] [bug #18461] crash using lua function civil_war (in ai_plr_data_get() [aidata.c::310]: assertion 'ai-phase_initialized' failed.)

2011-08-09 Thread Marko Lindqvist
Follow-up Comment #4, bug #18461 (project freeciv): Is civil war another case where phasemode != simultaneous would cause problems as described in bug #18333? ___ Reply to this item at: http://gna.org/bugs/?18461

[Freeciv-Dev] [bug #18461] crash using lua function civil_war (in ai_plr_data_get() [aidata.c::310]: assertion 'ai-phase_initialized' failed.)

2011-08-09 Thread Marko Lindqvist
Follow-up Comment #6, bug #18461 (project freeciv): Are you sure about the patch? Just by reading it (without checking from source code more context) I notice that you add initialization to one place, but don't remove it from anywhere. Isn't this going to cause double initialization? (If it is,

[Freeciv-Dev] [bug #18464] AC_CONFIG_MACRO_DIR requires autoconf 2.59

2011-08-09 Thread Marko Lindqvist
Update of bug #18464 (project freeciv): Status: Ready For Test = Fixed Assigned to:None = cazfi Open/Closed:Open = Closed

[Freeciv-Dev] [patch #2840] Autosettler tweak for 2.4 r 20006

2011-08-09 Thread Marko Lindqvist
Follow-up Comment #3, patch #2840 (project freeciv): Current S2_3 versions of this patch are in attached to patch #2841. I'll close that ticket as duplicate of this one, but one may still want to access files from it. ___ Reply to this

[Freeciv-Dev] [patch #2841] Autosettler tweak for 2.3.0-RC1

2011-08-09 Thread Marko Lindqvist
Update of patch #2841 (project freeciv): Status:None = Duplicate Assigned to:None = cazfi Open/Closed:Open = Closed

[Freeciv-Dev] [patch #2866] Autosettlers consider improving city center tiles

2011-08-09 Thread Marko Lindqvist
Update of patch #2866 (project freeciv): Status:None = Ready For Test Assigned to:None = cazfi Planned Release: = 2.3.1, 2.4.0 Summary: A

[Freeciv-Dev] [bug #17767] Advisor code don't handle special resouces

2011-08-09 Thread Marko Lindqvist
Update of bug #17767 (project freeciv): Severity: 4 - Important = 3 - Normal Status:None = Duplicate Assigned to:None = cazfi Open/Closed:

[Freeciv-Dev] [patch #2840] Make autosettlers consider improving resource tiles

2011-08-09 Thread Marko Lindqvist
Update of patch #2840 (project freeciv): Assigned to:None = cazfi Planned Release: = 2.3.1, 2.4.0 Summary: Autosettler tweak for 2.4 r 20006 = Make autosettlers consider improving

[Freeciv-Dev] [patch #2513] [lua01] rename files (preparation for lua module rework)

2011-08-09 Thread Matthias Pfafferodt
) ___ Additional Item Attachment: File name: 20110809-luascript01-rename-files-preparation-for-lua-module-rework.patch.bz2 Size:29 KB ___ Reply to this item at: http://gna.org/patch/?2513

[Freeciv-Dev] [patch #2840] Make autosettlers consider improving resource tiles

2011-08-09 Thread Marko Lindqvist
Update of patch #2840 (project freeciv): Status:None = Ready For Test ___ Follow-up Comment #4: Attached patch is part relevant to this ticket from one of the patches Ryan Riegel rriegs had

[Freeciv-Dev] [patch #2514] [lua02] redo lua script

2011-08-09 Thread Matthias Pfafferodt
Follow-up Comment #3, patch #2514 (project freeciv): updated patch (changes 20110809): * lua script signals (destroy signal_callbacks correctly and use va_copy() for signal callback, i.e. mor ethan one signal) * fix spelling of API_TYPE_CON[N]ECTION (patch also by Marko Lindqvist cazfi: gna

[Freeciv-Dev] [patch #2514] [lua02] redo lua script

2011-08-09 Thread Matthias Pfafferodt
Follow-up Comment #4, patch #2514 (project freeciv): and here is the patch file ... (file #13795) ___ Additional Item Attachment: File name: 20110809-luascript02-redo-lua-script.patch.bz2 Size:37 KB

[Freeciv-Dev] [patch #2869] [lua02a] merge lua and luafile commands

2011-08-09 Thread Matthias Pfafferodt
command: lua cmd script line lua file script file lua script line (compatibility) ___ File Attachments: --- Date: Mi 10 Aug 2011 00:07:47 CEST Name: 20110809-luascript03-merge-lua

[Freeciv-Dev] [patch #2869] [lua02a] merge lua and luafile commands

2011-08-09 Thread Matthias Pfafferodt
Update of patch #2869 (project freeciv): Depends on: = patch #2520 ___ Reply to this item at: http://gna.org/patch/?2869 ___ Nachricht

[Freeciv-Dev] [patch #2870] [lua02b] signal interface in luascript_signal.(ch)

2011-08-09 Thread Matthias Pfafferodt
___ Details: * remove signal callbackss with 'signal.remove()' * plot all signals (+ callbacks) with 'signal.list()' ___ File Attachments: --- Date: Mi 10 Aug 2011 00:09:54 CEST Name: 20110809

[Freeciv-Dev] [patch #2866] Autosettlers consider improving city center tiles

2011-08-09 Thread Marko Lindqvist
Follow-up Comment #3, patch #2866 (project freeciv): I've extracted parts from file #13764 to patch #2840. This ticket is now about city center problem only, latest version is file #13783 in comment #2. ___ Reply to this item at:

[Freeciv-Dev] [patch #2871] [lua02c] print all lua variables

2011-08-09 Thread Matthias Pfafferodt
function listenv() ___ File Attachments: --- Date: Mi 10 Aug 2011 00:10:53 CEST Name: 20110809-luascript04-signal-interface-in-luascript_signal.-ch.patch Size: 2kB By: syntron http

[Freeciv-Dev] [patch #2872] [lua02d] add implementation for lua constants

2011-08-09 Thread Matthias Pfafferodt
: --- Date: Mi 10 Aug 2011 00:11:57 CEST Name: 20110809-luascript06-add-implementation-for-lua-constants.patch Size: 2kB By: syntron http://gna.org/patch/download.php?file_id=13799 ___ Reply

[Freeciv-Dev] [patch #2871] [lua02c] print all lua variables

2011-08-09 Thread Matthias Pfafferodt
Follow-up Comment #1, patch #2871 (project freeciv): wrong file (was for patch #2870); the right one is here (file #13800) ___ Additional Item Attachment: File name: 20110809-luascript05-print-all-lua-variables.patch Size:1 KB

[Freeciv-Dev] [patch #2515] [lua03] client lua script

2011-08-09 Thread Matthias Pfafferodt
(file #13801) ___ Additional Item Attachment: File name: 20110809-luascript07-basic-files-for-client-lua-script.patch.bz2 Size:10 KB ___ Reply to this item at: http://gna.org/patch

[Freeciv-Dev] [patch #2873] [lua03a] lient lua script console (gtk3)

2011-08-09 Thread Matthias Pfafferodt
: untested patch (a copy of gtk2 version) ___ File Attachments: --- Date: Mi 10 Aug 2011 00:16:14 CEST Name: 20110809-luascript08-client-lua-script-gtk3.patch.bz2 Size: 5kB By: syntron http

[Freeciv-Dev] [patch #2874] [lua03b] files needed for the ftwl client if client lua script is added

2011-08-09 Thread Matthias Pfafferodt
___ Details: untested patch is the ftwml client still supported or is it to be removed? ___ File Attachments: --- Date: Mi 10 Aug 2011 00:17:44 CEST Name: 20110809-luascript09

[Freeciv-Dev] [patch #2875] [lua03c] files needed for the xaw client if client lua script is added

2011-08-09 Thread Matthias Pfafferodt
___ Details: untested ___ File Attachments: --- Date: Mi 10 Aug 2011 00:18:22 CEST Name: 20110809-luascript10-files-needed-for-the-xaw-client-if-client-lua-script.patch Size: 4kB

[Freeciv-Dev] [patch #2876] [lua03c] files needed for the win32 client if client lua script is added

2011-08-09 Thread Matthias Pfafferodt
___ Details: untested ___ File Attachments: --- Date: Mi 10 Aug 2011 00:19:00 CEST Name: 20110809-luascript11-files-needed-for-the-win32-client-if-client-lua-scri.patch Size

[Freeciv-Dev] [patch #2876] [lua03d] files needed for the win32 client if client lua script is added

2011-08-09 Thread Matthias Pfafferodt
Update of patch #2876 (project freeciv): Summary: [lua03c] files needed for the win32 client if client lua script is added = [lua03d] files needed for the win32 client if client lua script is added ___ Reply to this item

[Freeciv-Dev] [patch #2877] [lua03e] files needed for the stub client if client lua script is added

2011-08-09 Thread Matthias Pfafferodt
___ Details: untested ___ File Attachments: --- Date: Mi 10 Aug 2011 00:19:59 CEST Name: 20110809-luascript12-files-needed-for-the-stub-client-if-client-lua-scrip.patch Size

[Freeciv-Dev] [patch #2873] [lua03a] client lua script console (gtk3)

2011-08-09 Thread Matthias Pfafferodt
Update of patch #2873 (project freeciv): Summary: [lua03a] lient lua script console (gtk3) = [lua03a] client lua script console (gtk3) ___ Reply to this item at: http://gna.org/patch/?2873

[Freeciv-Dev] [patch #2878] [lua03f] files needed for the sdl client if client lua script is added

2011-08-09 Thread Matthias Pfafferodt
___ Details: untested ___ File Attachments: --- Date: Mi 10 Aug 2011 00:20:58 CEST Name: 20110809-luascript13-files-needed-for-the-sdl-client-if-client-lua-script.patch Size: 4kB

[Freeciv-Dev] [patch #2879] [lua03g] files needed for the qt client if client lua script is added

2011-08-09 Thread Matthias Pfafferodt
___ Details: untested ___ File Attachments: --- Date: Mi 10 Aug 2011 00:21:34 CEST Name: 20110809-luascript14-files-needed-for-the-qt-client-if-client-lua-script-.patch Size: 4kB

[Freeciv-Dev] [patch #2513] [lua01] rename files (preparation for lua module rework)

2011-08-09 Thread Marko Lindqvist
Follow-up Comment #5, patch #2513 (project freeciv): Can you attach Makfile.am configure.ac changes as separate patch. You shouldn't commit just file renames meaning freeciv wouldn't compile. At the same commit that you move files, you have to update Makefile.ams and configure.ac at least. If

[Freeciv-Dev] [patch #2516] [lua04] add the client lua console

2011-08-09 Thread Matthias Pfafferodt
Update of patch #2516 (project freeciv): Status: Ready For Test = Duplicate Open/Closed:Open = Closed ___ Follow-up Comment #1: included in patch

[Freeciv-Dev] [patch #2520] [metaticket] common lua interface

2011-08-09 Thread Matthias Pfafferodt
Update of patch #2520 (project freeciv): Depends on: = patch #2513 ___ Reply to this item at: http://gna.org/patch/?2520 ___ Nachricht

[Freeciv-Dev] [patch #2517] [lua05] fix luaconsole for all other clients

2011-08-09 Thread Matthias Pfafferodt
Update of patch #2517 (project freeciv): Status: Ready For Test = Duplicate Open/Closed:Open = Closed ___ Follow-up Comment #1: see the following

[Freeciv-Dev] [patch #2518] [lua06] add function to print from lua into the chat console

2011-08-09 Thread Matthias Pfafferodt
Update of patch #2518 (project freeciv): Status: In Progress = Duplicate Open/Closed:Open = Closed ___ Follow-up Comment #1: included in patch

[Freeciv-Dev] [patch #2519] [lua07][lua08] add first signal to the client

2011-08-09 Thread Matthias Pfafferodt
Follow-up Comment #1, patch #2519 (project freeciv): rebased version of this example (file #13809) ___ Additional Item Attachment: File name: 20110809-luascript24-client-lua-script-test-patch.patch Size:1 KB

[Freeciv-Dev] [patch #2880] [lua08] lua functions to get the freeciv version

2011-08-09 Thread Matthias Pfafferodt
___ Details: ___ File Attachments: --- Date: Mi 10 Aug 2011 00:29:02 CEST Name: 20110809-luascript15-a-luascript-function-freeciv_version.patch Size: 2kB By: syntron http://gna.org/patch

[Freeciv-Dev] [patch #2881] [lua09]add Player.is_alive and Player:exists()

2011-08-09 Thread Matthias Pfafferodt
: ___ File Attachments: --- Date: Mi 10 Aug 2011 00:29:53 CEST Name: 20110809-luascript16-add-Player.is_alive-and-Player-exists.patch Size: 818B By: syntron http://gna.org/patch/download.php

[Freeciv-Dev] [patch #2866] Autosettlers consider improving city center tiles

2011-08-09 Thread Marko Lindqvist
Follow-up Comment #4, patch #2866 (project freeciv): Ryan: Can you resubmit other parts of your patches to new tickets, one issue / ticket? ___ Reply to this item at: http://gna.org/patch/?2866

[Freeciv-Dev] [patch #2882] [lua10] lua function to check if the game was started

2011-08-09 Thread Matthias Pfafferodt
: ___ File Attachments: --- Date: Mi 10 Aug 2011 00:30:51 CEST Name: 20110809-luascript17-new-lua-function-to-check-if-the-game-was-started.patch Size: 2kB By: syntron http://gna.org

[Freeciv-Dev] [patch #2883] [lua11] lua function to save a game (as 'manual' save)

2011-08-09 Thread Matthias Pfafferodt
: ___ File Attachments: --- Date: Mi 10 Aug 2011 00:31:31 CEST Name: 20110809-luascript18-add-lua-function-to-save-a-game-as-manual-save.patch Size: 2kB By: syntron http://gna.org

[Freeciv-Dev] [patch #2840] Make autosettlers consider improving resource tiles

2011-08-09 Thread Marko Lindqvist
Follow-up Comment #5, patch #2840 (project freeciv): Micke: Can you resubmit parts of your patch not relevant to this ticket to new ticket(s), one issue / ticket, but all code branches handled in same ticket? ___ Reply to this item at:

[Freeciv-Dev] [patch #2884] [lua12] lua test game

2011-08-09 Thread Matthias Pfafferodt
; the test game should be a point to validate all lua functions ___ File Attachments: --- Date: Mi 10 Aug 2011 00:33:18 CEST Name: 20110809-luascript20-lua-test-script.patch.bz2 Size: 5kB

[Freeciv-Dev] [bug #18476] allow to load games without a map (i.e. tutorial)

2011-08-09 Thread Matthias Pfafferodt
2011 00:34:42 CEST Name: 20110809-luascript19-allow-to-load-games-without-a-map.patch Size: 1kB By: syntron http://gna.org/bugs/download.php?file_id=13815 ___ Reply to this item at: http://gna.org/bugs/?18476

[Freeciv-Dev] [patch #2520] [metaticket] common lua interface

2011-08-09 Thread Matthias Pfafferodt
Update of patch #2520 (project freeciv): Depends on: = patch #2880 ___ Reply to this item at: http://gna.org/patch/?2520 ___ Nachricht

[Freeciv-Dev] [patch #2520] [metaticket] common lua interface

2011-08-09 Thread Matthias Pfafferodt
Follow-up Comment #8, patch #2520 (project freeciv): updated patch series; a summary of the old/new functions and structure can be found on the wiki page mentioned in comment #7 ___ Reply to this item at: http://gna.org/patch/?2520

[Freeciv-Dev] [patch #2830] ser - fcser, civ - fcgui

2011-08-09 Thread Marko Lindqvist
Follow-up Comment #3, patch #2830 (project freeciv): - Updated against current svn (file #13816) ___ Additional Item Attachment: File name: fcser_2830-3.diff Size:2 KB

[Freeciv-Dev] [patch #2513] [lua01] rename files (preparation for lua module rework)

2011-08-09 Thread Matthias Pfafferodt
Follow-up Comment #6, patch #2513 (project freeciv): all changes you request are included in patch #2514; a list of files are given in comment #0. If you need it in another format, please tell me, so that I can try to provide it. ___

[Freeciv-Dev] [patch #2513] [lua01] rename files (preparation for lua module rework)

2011-08-09 Thread Marko Lindqvist
Follow-up Comment #7, patch #2513 (project freeciv): patch #2514 is hardly the minimum change to get freeciv compile after files are moved. That patch I'm after clearly should be part of this ticket, as it *must* be part of the same commit as actual file renames. Oh well, I can make that patch

[Freeciv-Dev] [patch #2513] [lua01] rename files (preparation for lua module rework)

2011-08-09 Thread Marko Lindqvist
Update of patch #2513 (project freeciv): Status: Ready For Test = In Progress Assigned to:None = cazfi ___ Follow-up Comment #8: Attached is

[Freeciv-Dev] [patch #2885] Fix autosettler to use rail connectivity bonus when planning for railroads

2011-08-09 Thread anonymous
URL: http://gna.org/patch/?2885 Summary: Fix autosettler to use rail connectivity bonus when planning for railroads Project: Freeciv Submitted by: None Submitted on: Wed 10 Aug 2011 04:28:51 AM UTC Category: ai

[Freeciv-Dev] [patch #2885] Fix autosettler to use rail connectivity bonus when planning for railroads

2011-08-09 Thread Ryan Riegel
Follow-up Comment #1, patch #2885 (project freeciv): Sorry, forgot to log in. The following is my submission and upload. ___ Reply to this item at: http://gna.org/patch/?2885 ___ Message

[Freeciv-Dev] [patch #2886] Fix set but unused variable warning from scripting

2011-08-09 Thread Marko Lindqvist
URL: http://gna.org/patch/?2886 Summary: Fix set but unused variable warning from scripting Project: Freeciv Submitted by: cazfi Submitted on: Wed 10 Aug 2011 07:56:48 AM EEST Category: general Priority:

[Freeciv-Dev] [patch #2887] Build dependencies first

2011-08-09 Thread Marko Lindqvist
URL: http://gna.org/patch/?2887 Summary: Build dependencies first Project: Freeciv Submitted by: cazfi Submitted on: Wed 10 Aug 2011 08:00:03 AM EEST Category: bootstrap Priority: 5 - Normal

[Freeciv-Dev] [patch #2888] Use city_list_size() instead of counting units in cityrepdata.c

2011-08-09 Thread Marko Lindqvist
URL: http://gna.org/patch/?2888 Summary: Use city_list_size() instead of counting units in cityrepdata.c Project: Freeciv Submitted by: cazfi Submitted on: Wed 10 Aug 2011 08:06:25 AM EEST Category: client

[Freeciv-Dev] [patch #2889] Allow autosettlers to plan for farmland

2011-08-09 Thread Ryan Riegel
URL: http://gna.org/patch/?2889 Summary: Allow autosettlers to plan for farmland Project: Freeciv Submitted by: rriegs Submitted on: Wed 10 Aug 2011 05:07:51 AM GMT Category: ai Priority: 5 - Normal

[Freeciv-Dev] [patch #2889] Allow autosettlers to plan for farmland

2011-08-09 Thread Marko Lindqvist
Follow-up Comment #1, patch #2889 (project freeciv): From code comment: farmland takes twice as long, so make it look half as good. Is that always true? Or is it possible that tile already has irrigation, so we are just one step away from farmland benefit?