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

2018-03-13 Thread Notabilis
They are running fast but linking takes relatively long. At least it feels that way, I haven't done any measurements. Just a thought: maybe its possible to link all of them in parallel instead of one after the other. -- https://code.launchpad.net/~widelands-dev/widelands/unlink_test_richtext/+me

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

2018-03-12 Thread GunChleoc
The other tests run really fast, so I don't think that we need to ne able disable them at this time. Or are they costing you significant time? @bunnybot merge -- https://code.launchpad.net/~widelands-dev/widelands/unlink_test_richtext/+merge/340264 Your team Widelands Developers is subscribed to

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

2018-03-10 Thread Notabilis
Review: Approve Assuming you mean "deactivated" it seems to be working fine. Tested by applying the makefile change locally. I haven't tested the changes for Debian, though. Feature request (not for this branch): Add a compile.sh parameter to disable all tests. -- https://code.launchpad.net/~w