[Freeciv-Dev] [bug #20723] Enabling multiple luasql backends broken on S2_5 trunk
Update of bug #20723 (project freeciv): Status: Ready For Test = Fixed Assigned to:None = cazfi Open/Closed:Open = Closed ___ Reply to this item at: http://gna.org/bugs/?20723 ___ 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 #20723] Enabling multiple luasql backends broken on S2_5 trunk
Follow-up Comment #7, bug #20723 (project freeciv): I don't currently have good setup to test this, so I'd be grateful if someone else could do some testing on this. I confirm that a server binary built with this patch works against both MySQL and SQLite backends. Thanks! ___ Reply to this item at: http://gna.org/bugs/?20723 ___ 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 #20723] Enabling multiple luasql backends broken on S2_5 trunk
Update of bug #20723 (project freeciv): Status:None = Ready For Test ___ Follow-up Comment #6: The LuaSQL docs indicate that an equivalent set of 'require' statements inside Lua can end up with a 'luasql' table containing multiple drivers simultaneously. (That site does not respond, but I found some documentation from luasql git) No, the example shows how 'require' is used to open *multiple* tables, not *one* where everything works. So first step is obviously opening all the tables. That's easy. Ideally I would then like to combine the tables in to one, but that's not very easy, at least to be done right. So I think it's sufficient for now to use multiple tables in database.lua too. Patch attached. I don't currently have good setup to test this, so I'd be grateful if someone else could do some testing on this. (file #21612) ___ Additional Item Attachment: File name: MultibackendLuasql.patch Size:1 KB ___ Reply to this item at: http://gna.org/bugs/?20723 ___ 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 #20723] Enabling multiple luasql backends broken on S2_5 trunk
Follow-up Comment #5, bug #20723 (project freeciv): I want to fix this properly for 2.5.0 rather than have a regression. Can anyone help? However, failing that, we could make configure disallow multiple luasql backends. ___ Reply to this item at: http://gna.org/bugs/?20723 ___ 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 #20723] Enabling multiple luasql backends broken on S2_5 trunk
Update of bug #20723 (project freeciv): Summary: Enabling multiple luasql backends broken on trunk = Enabling multiple luasql backends broken on S2_5 trunk ___ Reply to this item at: http://gna.org/bugs/?20723 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
