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

2019-05-30 Thread bunnybot
Continuous integration builds have changed state:

Travis build 5100. State: failed. Details: 
https://travis-ci.org/widelands/widelands/builds/539371759.
Appveyor build 4881. State: failed. Details: 
https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_workarea_fixes-4881.
-- 
https://code.launchpad.net/~widelands-dev/widelands/workarea-fixes/+merge/368083
Your team Widelands Developers is subscribed to branch 
lp:~widelands-dev/widelands/workarea-fixes.

___
Mailing list: https://launchpad.net/~widelands-dev
Post to : widelands-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~widelands-dev
More help   : https://help.launchpad.net/ListHelp


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

2019-05-30 Thread bunnybot
Refusing to merge, since Travis is not green. Use @bunnybot merge force for 
merging anyways.

Travis build 5100. State: failed. Details: 
https://travis-ci.org/widelands/widelands/builds/539371759.
-- 
https://code.launchpad.net/~widelands-dev/widelands/workarea-fixes/+merge/368083
Your team Widelands Developers is subscribed to branch 
lp:~widelands-dev/widelands/workarea-fixes.

___
Mailing list: https://launchpad.net/~widelands-dev
Post to : widelands-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~widelands-dev
More help   : https://help.launchpad.net/ListHelp


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

2019-05-30 Thread GunChleoc
Review: Approve

:P

Looks good though :)

@bunnybot merge
-- 
https://code.launchpad.net/~widelands-dev/widelands/workarea-fixes/+merge/368083
Your team Widelands Developers is subscribed to branch 
lp:~widelands-dev/widelands/workarea-fixes.

___
Mailing list: https://launchpad.net/~widelands-dev
Post to : widelands-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~widelands-dev
More help   : https://help.launchpad.net/ListHelp


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

2019-05-30 Thread Benedikt Straub
Done. The diff is now almost 50% longer :P
-- 
https://code.launchpad.net/~widelands-dev/widelands/workarea-fixes/+merge/368083
Your team Widelands Developers is requested to review the proposed merge of 
lp:~widelands-dev/widelands/workarea-fixes into lp:widelands.

___
Mailing list: https://launchpad.net/~widelands-dev
Post to : widelands-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~widelands-dev
More help   : https://help.launchpad.net/ListHelp


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

2019-05-30 Thread GunChleoc
Code LGTM and it's all working well :)

Could you make the surrounding line thicker? It's too thin to be noticeable, 
really.
-- 
https://code.launchpad.net/~widelands-dev/widelands/workarea-fixes/+merge/368083
Your team Widelands Developers is requested to review the proposed merge of 
lp:~widelands-dev/widelands/workarea-fixes into lp:widelands.

___
Mailing list: https://launchpad.net/~widelands-dev
Post to : widelands-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~widelands-dev
More help   : https://help.launchpad.net/ListHelp


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

2019-05-30 Thread Toni Förster
@hessenfarmer would you be so kind and have a look at the changes I did in the 
trainingsites?
-- 
https://code.launchpad.net/~widelands-dev/widelands/unify_sleep_time/+merge/368086
Your team Widelands Developers is subscribed to branch 
lp:~widelands-dev/widelands/failed_skipped_10s.

___
Mailing list: https://launchpad.net/~widelands-dev
Post to : widelands-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~widelands-dev
More help   : https://help.launchpad.net/ListHelp


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

2019-05-30 Thread noreply
The proposal to merge lp:~widelands-dev/widelands/failed_skipped_10s into 
lp:widelands has been updated.

Status: Needs review => Merged

For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/failed_skipped_10s/+merge/368014
-- 
Your team Widelands Developers is subscribed to branch 
lp:~widelands-dev/widelands/failed_skipped_10s.

___
Mailing list: https://launchpad.net/~widelands-dev
Post to : widelands-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~widelands-dev
More help   : https://help.launchpad.net/ListHelp


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

2019-05-30 Thread Toni Förster
the inputqueues again

@bunnybot merge force
-- 
https://code.launchpad.net/~widelands-dev/widelands/failed_skipped_10s/+merge/368014
Your team Widelands Developers is subscribed to branch 
lp:~widelands-dev/widelands/failed_skipped_10s.

___
Mailing list: https://launchpad.net/~widelands-dev
Post to : widelands-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~widelands-dev
More help   : https://help.launchpad.net/ListHelp


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

2019-05-30 Thread bunnybot
Refusing to merge, since Travis is not green. Use @bunnybot merge force for 
merging anyways.

Travis build 5095. State: errored. Details: 
https://travis-ci.org/widelands/widelands/builds/538979803.
-- 
https://code.launchpad.net/~widelands-dev/widelands/failed_skipped_10s/+merge/368014
Your team Widelands Developers is subscribed to branch 
lp:~widelands-dev/widelands/failed_skipped_10s.

___
Mailing list: https://launchpad.net/~widelands-dev
Post to : widelands-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~widelands-dev
More help   : https://help.launchpad.net/ListHelp


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

2019-05-30 Thread Toni Förster
The 10s increase in the test fixed it. But we only need this as long as the 
unify_sleep_time isn't merged into branch. We can revert these changes there 
because sleeps aren't executed before a possible fail anymore.

@bunnybot merge
-- 
https://code.launchpad.net/~widelands-dev/widelands/failed_skipped_10s/+merge/368014
Your team Widelands Developers is subscribed to branch 
lp:~widelands-dev/widelands/failed_skipped_10s.

___
Mailing list: https://launchpad.net/~widelands-dev
Post to : widelands-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~widelands-dev
More help   : https://help.launchpad.net/ListHelp


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

2019-05-30 Thread bunnybot
Continuous integration builds have changed state:

Travis build 5094. State: errored. Details: 
https://travis-ci.org/widelands/widelands/builds/538978716.
Appveyor build 4874. State: failed. Details: 
https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_unify_sleep_time-4874.
-- 
https://code.launchpad.net/~widelands-dev/widelands/unify_sleep_time/+merge/368086
Your team Widelands Developers is subscribed to branch 
lp:~widelands-dev/widelands/failed_skipped_10s.

___
Mailing list: https://launchpad.net/~widelands-dev
Post to : widelands-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~widelands-dev
More help   : https://help.launchpad.net/ListHelp


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

2019-05-30 Thread GunChleoc
This will run the whole test suite - he will only want to run the casern test 
when fixing.

https://wl.widelands.org/wiki/RegressionTests/
-- 
https://code.launchpad.net/~widelands-dev/widelands/failed_skipped_10s/+merge/368014
Your team Widelands Developers is subscribed to branch 
lp:~widelands-dev/widelands/failed_skipped_10s.

___
Mailing list: https://launchpad.net/~widelands-dev
Post to : widelands-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~widelands-dev
More help   : https://help.launchpad.net/ListHelp


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

2019-05-30 Thread bunnybot
Continuous integration builds have changed state:

Travis build 5093. State: passed. Details: 
https://travis-ci.org/widelands/widelands/builds/538931717.
Appveyor build 4873. State: failed. Details: 
https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_appveyor_opengl-4873.
-- 
https://code.launchpad.net/~widelands-dev/widelands/appveyor_opengl/+merge/368039
Your team Widelands Developers is subscribed to branch 
lp:~widelands-dev/widelands/appveyor_opengl.

___
Mailing list: https://launchpad.net/~widelands-dev
Post to : widelands-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~widelands-dev
More help   : https://help.launchpad.net/ListHelp