[Freeciv-Dev] [patch #2270] [metaticket] allow recursive transports
Update of patch #2270 (project freeciv): Status: In Progress = Done Open/Closed:Open = Closed ___ Follow-up Comment #14: all tickets are closed; closing also this one as done ___ Reply to this item at: http://gna.org/patch/?2270 ___ 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 #2270] [metaticket] allow recursive transports
Follow-up Comment #13, patch #2270 (project freeciv): found the bug - units on the client there not marked as unloaded; rebased fixed patches are added to all tickets ___ Reply to this item at: http://gna.org/patch/?2270 ___ 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 #2270] [metaticket] allow recursive transports
Follow-up Comment #12, patch #2270 (project freeciv): there seems to be a bug on client exit - some more work to do ... ___ Reply to this item at: http://gna.org/patch/?2270 ___ 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 #2270] [metaticket] allow recursive transports
Follow-up Comment #11, patch #2270 (project freeciv): I will commit this patch set to trunk this week if there are no objections. autogame tests show small differences in the last seen time (I suspect the order of movement of transported units). For large games this can result in bigger differences, i.e one unit not seen by an AI. But I think it is working and should go into 2.4 ... ___ Reply to this item at: http://gna.org/patch/?2270 ___ 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 #2270] [metaticket] allow recursive transports
Follow-up Comment #9, patch #2270 (project freeciv): updated all-in-one patch (svn20285); I will send updated versions of the patch series then I'm back home from holiday (file #14194) ___ Additional Item Attachment: File name: 20111003-transport.diff.bz2Size:34 KB ___ Reply to this item at: http://gna.org/patch/?2270 ___ 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 #2270] [metaticket] allow recursive transports
Follow-up Comment #10, patch #2270 (project freeciv): I need more holiday - that's the svn version on my laptop - it's the one you have tried :-( Got into some hassle to get an up-to-date freeciv version to rebase it (svn20296). I did not compile test this version but it does include some helpers to do autogames (patch #2972; hints from patch #2973). ___ Reply to this item at: http://gna.org/patch/?2270 ___ 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 #2270] [metaticket] allow recursive transports
Additional Item Attachment, patch #2270 (project freeciv): File name: 20111003-svn20296-transport.diff.bz2 Size:34 KB ___ Reply to this item at: http://gna.org/patch/?2270 ___ 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 #2270] [metaticket] allow recursive transports
Follow-up Comment #7, patch #2270 (project freeciv): For info, the last trunk rev I've been able to apply the latest all-in-one (file #13932) to is r20285; it seems to conflict with patch #2965 (haven't investigated). ___ Reply to this item at: http://gna.org/patch/?2270 ___ 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 #2270] [metaticket] allow recursive transports
Follow-up Comment #8, patch #2270 (project freeciv): I know - and there are rebased patches on my computer at home (git is your friend); I will upload it at the time I'm back ... ___ Reply to this item at: http://gna.org/patch/?2270 ___ 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 #2270] [metaticket] allow recursive transports
Follow-up Comment #5, patch #2270 (project freeciv): I did update the entire patch series. It is working but tests are still running. Due to changes in unit_move() autogames differ (transported units are moved after it is sure that the transporter survives). I'm working on an (optional) patch which will restore the old behaviour. Order of the patches: - [01/09] patch #2477 [transport09] use pointer for transported units (server/common) - [02/09] patch #2547 [transport19] cleanup unit_move() - [03/09] patch #2478 [transport10] use pointer for transported units (client) - patch [04/09] patch #2479 [transport11] clean up network (occupied information of transporters) - [05/09] patch #2482 [transport14] redo location selection - [06/09] patch #2483 [transport15] add function to create unit type sprite - [07/09] bug #17236 [transport18] unit selection dialog - [08/09] patch #2480 [transport12] allow recursive transports - [09/09] patch #2481 [transport13] allow Missiles on Mech. Inf. (experimental ruleset) - optional An image of the new unit selection dialog (only gtk2 at the moment) can be found in file #13928 (experimental ruleset with patch #2481) ___ Reply to this item at: http://gna.org/patch/?2270 ___ 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 #2270] [metaticket] allow recursive transports
Follow-up Comment #6, patch #2270 (project freeciv): and an all-in-one patch (file #13932) ___ Additional Item Attachment: File name: 20110819-transport.patch Size:170 KB ___ Reply to this item at: http://gna.org/patch/?2270 ___ 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 #2270] [metaticket] allow recursive transports
Follow-up Comment #4, patch #2270 (project freeciv): I've played with this and been meaning to post some constructive comments for some time, but haven't got round to marshalling them, so for now I'll just say: good show, unit selection has been a weak point of Freeciv. Hopefully I'll get to post my detailed comments at some point. For anyone else wanting to play with this: the latest all-in-one patch doesn't apply to current trunk; the last version I've successfully applied it to is r19263. (You might also want the fix for bug #17677 to get a usable system.) ___ Reply to this item at: http://gna.org/patch/?2270 ___ 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 #2270] [metaticket] allow recursive transports
Update of patch #2270 (project freeciv): Depends on: = patch #2547 ___ Reply to this item at: http://gna.org/patch/?2270 ___ 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 #2270] [metaticket] allow recursive transports
Update of patch #2270 (project freeciv): Depends on: = patch #2426 ___ Reply to this item at: http://gna.org/patch/?2270 ___ 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 #2270] [metaticket] allow recursive transports
Update of patch #2270 (project freeciv): Depends on: = bugs #17644 ___ Reply to this item at: http://gna.org/patch/?2270 ___ 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 #2270] [metaticket] allow recursive transports
Follow-up Comment #3, patch #2270 (project freeciv): this meta ticket implements recursive transports (transporters can be transported) as well as a new unit selection dialog for the gtk client with a common backend which can be used by other clients. For easy testing an all-in-one patch will be available here (file #12279) ___ Additional Item Attachment: File name: 20110204-transport.diff.bz2Size:65 KB ___ Reply to this item at: http://gna.org/patch/?2270 ___ 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 #2270] [metaticket] allow recursive transports
Update of patch #2270 (project freeciv): Assigned to:None = syntron Summary: allow recursive transports = [metaticket] allow recursive transports ___ Follow-up Comment #2: This is the current status for recursive transports. It also includes a new unit slection dialog (usdlg) for the gtk client (bug #17236). See attached images. The patch is not perfect (some gtk errors and debug messages) but it is working. Included is a change to the experimental ruleset to allow Missile on Mech.Inf. I will work on this patch then I find time. (file #12233, file #12234, file #12235) ___ Additional Item Attachment: File name: transport.diff.bz2 Size:64 KB File name: transport01.jpgSize:41 KB File name: transport02.jpgSize:40 KB ___ Reply to this item at: http://gna.org/patch/?2270 ___ Nachricht geschickt von/durch Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev