[Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1535065 into lp:widelands

2016-02-03 Thread noreply
The proposal to merge lp:~widelands-dev/widelands/bug-1535065 into lp:widelands has been updated. Status: Needs review => Merged For more details, see: https://code.launchpad.net/~widelands-dev/widelands/bug-1535065/+merge/284658 -- Your team Widelands Developers is subscribed to branch lp

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1535065 into lp:widelands

2016-02-03 Thread GunChleoc
@bunnybot merge -- https://code.launchpad.net/~widelands-dev/widelands/bug-1535065/+merge/284658 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/bug-1535065. ___ Mailing list: https://launchpad.net/~widelands-dev Post

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1535065 into lp:widelands

2016-02-02 Thread SirVer
Review: Approve -- https://code.launchpad.net/~widelands-dev/widelands/bug-1535065/+merge/284658 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/bug-1535065. ___ Mailing list: https://launchpad.net/~widelands-dev Po

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1535065 into lp:widelands

2016-02-02 Thread bunnybot
Continuous integration builds have changed state: Travis build 534. State: passed. Details: https://travis-ci.org/widelands/widelands/builds/106464683. Appveyor build 388. State: success. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_bug_1535065-

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1535065 into lp:widelands

2016-02-02 Thread bunnybot
Continuous integration builds have changed state: Travis build 507. State: errored. Details: https://travis-ci.org/widelands/widelands/builds/106288371. Appveyor build 388. State: success. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_bug_1535065

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1535065 into lp:widelands

2016-02-01 Thread GunChleoc
I agree that this still needs more fixing - right now, I just want to get rid of the crash. I tried fiddling wit it a bit, but as soon as I try to get the FCoords from the Coords, the player positions get messed up, and I have no idea why yet. So, I am for merging this and leaving the bug open.

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1535065 into lp:widelands

2016-02-01 Thread kaputtnik
So a user make a random map, saves it and when trying to start a game with this map an error message is shown because no player position is set. Then he has to start the editor again, load the map again and probably figure out how to set player positions... And he may think he has to do this alw

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1535065 into lp:widelands

2016-02-01 Thread bunnybot
Hi, I am bunnybot (https://github.com/widelands/bunnybot). I am keeping the source branch lp:~widelands-dev/widelands/bug-1535065 mirrored to https://github.com/widelands/widelands/tree/_widelands_dev_widelands_bug_1535065 You can give me commands by starting a line with @bunnybot . I understa

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1535065 into lp:widelands

2016-02-01 Thread GunChleoc
GunChleoc has proposed merging lp:~widelands-dev/widelands/bug-1535065 into lp:widelands. Commit message: Unset player starting position in MapGenerator if coordinates are illegal. Requested reviews: Widelands Developers (widelands-dev) Related bugs: Bug #1535065 in widelands: "Editor crashe