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

2017-11-04 Thread GunChleoc
@bunnybot merge -- https://code.launchpad.net/~widelands-dev/widelands/ai_small_requests/+merge/332519 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/ai_small_requests. ___ Mailing list:

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1716166-ai-strength-assert into lp:widelands

2017-11-04 Thread bunnybot
Continuous integration builds have changed state: Travis build 2750. State: failed. Details: https://travis-ci.org/widelands/widelands/builds/296968163. Appveyor build 2562. State: failed. Details:

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/savegame-menu into lp:widelands

2017-11-04 Thread GunChleoc
The proposal to merge lp:~widelands-dev/widelands/savegame-menu into lp:widelands has been updated. Commit Message changed to: Refactored load- and savegame screens. Cleaned up file system constants. - New common class GameDetails to show information about savegames, analogous to MapDetails

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/savegame-menu into lp:widelands

2017-11-04 Thread GunChleoc
The proposal to merge lp:~widelands-dev/widelands/savegame-menu into lp:widelands has been updated. Commit Message changed to: Refactored load- and savegame screens. Cleaned up file system constants. Save options immediately. - New common class GameDetails to show information about savegames,

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/savegame-menu into lp:widelands

2017-11-04 Thread GunChleoc
Review: Resubmit This is now ready again, the crash is gone. I added the things I couldn't solve right now as TODO comments to the code - those are wider issues with the UI toolbox that are already in trunk. I also cleaned up the file system constants, because it was really annoying me.

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

2017-11-04 Thread noreply
The proposal to merge lp:~widelands-dev/widelands/translation_stats into lp:widelands has been updated. Status: Needs review => Merged For more details, see: https://code.launchpad.net/~widelands-dev/widelands/translation_stats/+merge/332029 -- Your team Widelands Developers is subscribed

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

2017-11-04 Thread bunnybot
Continuous integration builds have changed state: Travis build 2754. State: passed. Details: https://travis-ci.org/widelands/widelands/builds/297152789. Appveyor build 2566. State: success. Details:

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

2017-11-04 Thread TiborB
TiborB has proposed merging lp:~widelands-dev/widelands/bug-1729856 into lp:widelands. Requested reviews: Widelands Developers (widelands-dev) Related bugs: Bug #1729856 in widelands: "AI enhances past its building limit" https://bugs.launchpad.net/widelands/+bug/1729856 For more details,

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

2017-11-04 Thread bunnybot
Continuous integration builds have changed state: Travis build 2760. State: passed. Details: https://travis-ci.org/widelands/widelands/builds/297351434. Appveyor build 2572. State: success. Details:

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

2017-11-04 Thread bunnybot
Continuous integration builds have changed state: Travis build 2748. State: errored. Details: https://travis-ci.org/widelands/widelands/builds/296945317. Appveyor build 2560. State: success. Details:

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

2017-11-04 Thread GunChleoc
I gave it another try, adding https:// results in a string truncation, so I'm leaving it out. I found another bug where I was parsing the wrong column, so I've added some error handling and am parsing the header info. Thanks for all your help in debugging this thing! @bunnybot merge --

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

2017-11-04 Thread GunChleoc
@bunnybot merge force -- https://code.launchpad.net/~widelands-dev/widelands/ai_small_requests/+merge/332519 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/ai_small_requests. ___ Mailing list:

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands-website/searching_with_haystack_whoosh into lp:widelands-website

2017-11-04 Thread kaputtnik
You could also try to open the site in a browsers private window. I am probably not on a machine with ssh access from today noontime until Sunday evening. -- https://code.launchpad.net/~widelands-dev/widelands-website/searching_with_haystack_whoosh/+merge/331605 Your team Widelands Developers

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/net-uuid into lp:widelands

2017-11-04 Thread GunChleoc
Review: Approve I opened a new issue over at the metaserver, because I don't know how to run one. So, I'll just rely on your testing here. As to the 2 NOCOMs, best open new bugs for them and note that they need investigating whether they are bugs. As to the trimming, you could use it for an

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

2017-11-04 Thread bunnybot
Error merging this proposal: Output: stdout: stderr: M src/ai/ai_help_structs.cc M src/ai/ai_help_structs.h M src/ai/defaultai.cc M src/ai/defaultai_warfare.cc Text conflict in src/ai/ai_help_structs.h 1 conflicts encountered. --

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

2017-11-04 Thread bunnybot
Continuous integration builds have changed state: Travis build 2756. State: passed. Details: https://travis-ci.org/widelands/widelands/builds/297200705. Appveyor build 2568. State: success. Details:

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

2017-11-04 Thread noreply
The proposal to merge lp:~widelands-dev/widelands/ai_small_requests into lp:widelands has been updated. Status: Needs review => Merged For more details, see: https://code.launchpad.net/~widelands-dev/widelands/ai_small_requests/+merge/332519 -- Your team Widelands Developers is subscribed

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/savegame-menu into lp:widelands

2017-11-04 Thread bunnybot
Continuous integration builds have changed state: Travis build 2758. State: failed. Details: https://travis-ci.org/widelands/widelands/builds/297334121. Appveyor build 2570. State: failed. Details:

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1716166-ai-strength-assert into lp:widelands

2017-11-04 Thread TiborB
Review: Approve The approach is OK, but I am lost in regard to these all values Approving the code though... -- https://code.launchpad.net/~widelands-dev/widelands/bug-1716166-ai-strength-assert/+merge/333227 Your team Widelands Developers is subscribed to branch