[Freeciv-Dev] [bug #13709] Switch from toluaxx to tolua-5.1b

2010-04-12 Thread Ulrik Sverdrup
Update of bug #13709 (project freeciv): Status: In Progress => Fixed ___ Follow-up Comment #17: The compilation error is tracked in a separate bug #15860 ___

[Freeciv-Dev] [bug #13709] Switch from toluaxx to tolua-5.1b

2010-04-12 Thread Matthias Pfafferodt
Follow-up Comment #16, bug #13709 (project freeciv): > The trunk error might be a warning from a newer gcc (I compiled with v 4.3). Here is a diff that might fix that (Attached). Do you see an error or warning before / after the patch? I did now a clean checkout (git clean -f) and the error is

[Freeciv-Dev] [bug #13709] Switch from toluaxx to tolua-5.1b

2010-04-12 Thread Ulrik Sverdrup
Update of bug #13709 (project freeciv): Status: Fixed => In Progress Planned Release: 2.3.0 => 2.2.1, 2.3.0 ___ Follow-up Comment #14: The trunk error m

[Freeciv-Dev] [bug #13709] Switch from toluaxx to tolua-5.1b

2010-04-12 Thread Matthias Pfafferodt
Follow-up Comment #13, bug #13709 (project freeciv): The second error seems to be related to an unclean directory. It does not happen after cleaning up ... ___ Reply to this item at: ___

[Freeciv-Dev] [bug #13709] Switch from toluaxx to tolua-5.1b

2010-04-12 Thread Ulrik Sverdrup
Follow-up Comment #12, bug #13709 (project freeciv): I made a rather massive error on the 2.2 branch, since tolua-5.1 does not even exist there. I'm sorry for that. I don't understand the trunk error at all. It compiled without warnings.

[Freeciv-Dev] [bug #13709] Switch from toluaxx to tolua-5.1b

2010-04-12 Thread Matthias Pfafferodt
Follow-up Comment #11, bug #13709 (project freeciv): I get errors trying to compile S2_2 and trunk S2_2: dependencies/Makefile.am:3: required directory dependencies/tolua-5.1 does not exist utility/Makefile.am:60: `%'-style pattern rules are a GNU make extension configure.ac:942: required file

[Freeciv-Dev] [bug #13709] Switch from toluaxx to tolua-5.1b

2010-04-12 Thread Ulrik Sverdrup
Update of bug #13709 (project freeciv): Status: Ready For Test => Fixed Open/Closed:Open => Closed ___ Follow-up Comment #10: Switched to tolua

[Freeciv-Dev] [bug #13709] Switch from toluaxx to tolua-5.1b

2010-04-11 Thread Ulrik Sverdrup
Follow-up Comment #7, bug #13709 (project freeciv): That's a good idea. I think we can be without the first patch for tolua_outside, see bug #15841. The second patch for tolua_notnil is still interesting since there is a lot less duplicate code/duplicate checks needed with it. But we don't need

[Freeciv-Dev] [bug #13709] Switch from toluaxx to tolua-5.1b

2010-04-11 Thread Matthias Pfafferodt
Follow-up Comment #6, bug #13709 (project freeciv): Perhaps you could add the patches to this external program code in an extra directory. This way they are available if a new version has to be patched. ___ Reply to this item at:

[Freeciv-Dev] [bug #13709] Switch from toluaxx to tolua-5.1b

2010-04-10 Thread Ulrik Sverdrup
Update of bug #13709 (project freeciv): Assigned to: cazfi => englabenny Depends on: => bugs #15841 ___ Follow-up Comment #5: I hope you don't m

[Freeciv-Dev] [bug #13709] Switch from toluaxx to tolua-5.1b

2010-04-10 Thread Ulrik Sverdrup
Follow-up Comment #4, bug #13709 (project freeciv): Thanks for your thoughts. I've been missing more discussion about script/api changes. Given how little that happened with scripting before, and how much that is possible now given the momentum we have (right now), I think we should just go forw

[Freeciv-Dev] [bug #13709] Switch from toluaxx to tolua-5.1b

2010-04-10 Thread Marko Lindqvist
Follow-up Comment #3, bug #13709 (project freeciv): Replying in ticket so discussion can be found from tracker later. > Is it okay to patch tolua-5.1 to use tolua_outside. > The implementation is already attached and is relatively > trivial. This is a feature that makes a lot of sense for > Free

Re: [Freeciv-Dev] [bug #13709] Switch from toluaxx to tolua-5.1b

2010-04-10 Thread Ulrik Sverdrup
2010/4/10 Matthias Pfafferodt : > > Follow-up Comment #2, bug #13709 (project freeciv): > > I do not know lua. But if there are no compatibility problems I think this > can go into 2.2.1. Do we need toluaxx after this change? In my testing, it seems to work almost exactly the same. Only corner ca

[Freeciv-Dev] [bug #13709] Switch from toluaxx to tolua-5.1b

2010-04-10 Thread Matthias Pfafferodt
Follow-up Comment #2, bug #13709 (project freeciv): I do not know lua. But if there are no compatibility problems I think this can go into 2.2.1. Do we need toluaxx after this change? ___ Reply to this item at:

[Freeciv-Dev] [bug #13709] Switch from toluaxx to tolua-5.1b

2010-04-09 Thread Ulrik Sverdrup
Update of bug #13709 (project freeciv): Priority: 1 - Later => 5 - Normal Status: In Progress => Ready For Test ___ Follow-up Comment #1: Here are patches 1

[Freeciv-Dev] [bug #13709] Switch from toluaxx to tolua-5.1b

2009-12-18 Thread Marko Lindqvist
Update of bug #13709 (project freeciv): Planned Release: => 2.3.0 ___ Reply to this item at: ___ Message sen

[Freeciv-Dev] [bug #13709] Switch from toluaxx to tolua-5.1b

2009-06-21 Thread Marko Lindqvist
Update of bug #13709 (project freeciv): Category:None => bootstrap Status:None => In Progress Assigned to:None => cazfi

[Freeciv-Dev] [bug #13709] Switch from toluaxx to tolua-5.1b

2009-06-16 Thread Marko Lindqvist
URL: Summary: Switch from toluaxx to tolua-5.1b Project: Freeciv Submitted by: cazfi Submitted on: Wednesday 06/17/2009 at 02:33 Category: None Severity: 3 - Normal