Travis failed for GCC 7 Debug Build with "No output has been received in the
last 10m0s" when running the new test. Since all other builds succeeded, I
think this is transitional error. At least, I am not aware of any loops related
to this branch that could end up in an endless loop.
@bunnybot
Yep!
@bunnybot merge
--
https://code.launchpad.net/~widelands-dev/widelands/bug-1795871-lua-set_workers/+merge/357716
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/bug-1795871-lua-set_workers.
___
Mailing list:
Worked fine now, thanks! So I guess this can be merged now?
--
https://code.launchpad.net/~widelands-dev/widelands/bug-1795871-lua-set_workers/+merge/357716
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/bug-1795871-lua-set_workers.
I have added a function to stop the barracks. The test should be green now.
--
https://code.launchpad.net/~widelands-dev/widelands/bug-1795871-lua-set_workers/+merge/357716
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/bug-1795871-lua-set_workers.
Thanks for adding the test. The code is looking good, but for me the test fails:
Error in Lua Coroutine
[../src/scripting/lua_errors.cc:22] [string
"test/maps/plain.wmf/scripting/test_inputqueue..."]:9: expected '8' but was '7'!
I think the problem is that the barracks starts working
Seems indeed as if something failed within Launchpad. For convenience, here is
the change:
https://bazaar.launchpad.net/~widelands-dev/widelands/bug-1795871-lua-set_workers/revision/8897
Interesting is only the first block, the rest are added brackets.
--
This seems to be some Launchpad error. If you click browse the code it is
shown including the diff.
--
https://code.launchpad.net/~widelands-dev/widelands/bug-1795871-lua-set_workers/+merge/357716
Your team Widelands Developers is requested to review the proposed merge of
I don't see any diff here. Did you push your changes?
--
https://code.launchpad.net/~widelands-dev/widelands/bug-1795871-lua-set_workers/+merge/357716
Your team Widelands Developers is requested to review the proposed merge of
lp:~widelands-dev/widelands/bug-1795871-lua-set_workers into
On second thought, this fix is kind of broken. I haven't tested it but it will
most likely fail if a building has the same worker type as an "input" worker
and as a "worker" worker. In that case, the input workers will probably be
deleted as well.
Unfortunately the PlayerImmovable given as a
9 matches
Mail list logo
