Review: Approve
Thanks! Probably got lost when we did he big conversion from conf to Lua files
@bunnybot merge
--
https://code.launchpad.net/~widelands-dev/widelands/missing_animations_barbarians_wood_hardener/+merge/352980
Your team Widelands Developers is subscribed to branch
Toni Förster has proposed merging
lp:~widelands-dev/widelands/bug-1786613-500ms-return-skipped into lp:widelands.
Commit message:
reduce waiting time to 500ms for skipped programs
Requested reviews:
hessenfarmer (stephan-lutz)
Related bugs:
Bug #1786613 in widelands: "production times are
Review: Approve
Good catch :)
@bunnybot merge
--
https://code.launchpad.net/~widelands-dev/widelands/remove_executable_bit/+merge/353012
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/remove_executable_bit.
___
Review: Approve
yes
good catch
--
https://code.launchpad.net/~widelands-dev/widelands/missing_animations_barbarians_wood_hardener/+merge/352980
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/mines-worldsavior.
___
The proposal to merge
lp:~widelands-dev/widelands/bug-1786613-10s-return-skipped into lp:widelands
has been updated.
Status: Needs review => Superseded
For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/bug-1786613-10s-return-skipped/+merge/352967
--
Your team
Continuous integration builds have changed state:
Travis build 3782. State: failed. Details:
https://travis-ci.org/widelands/widelands/builds/415390227.
Appveyor build 3581. State: success. Details:
Review: Approve
LGTM!
Thanks for fixing.
--
https://code.launchpad.net/~widelands-dev/widelands/fix-frisian-trainingsites-food-help/+merge/352977
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/fix-frisian-trainingsites-food-help.
Review: Needs Fixing
Ok I tested this with different timings.
My machine is a Core 2 Duo T7500 @ 2,2 Ghz with 4 GB RAM.
My setup was
map: the nile
8 AI players 2 of each tribe
Let the game run for 4:50 hours
saved and loaded with every config.
results were:
original timing: around 30% cpu
The proposal to merge lp:~widelands-dev/widelands/macos_build_app_compiler into
lp:widelands has been updated.
Description changed to:
One can choose between Clang and GCC now. The Wiki has been changed accordingly:
https://wl.widelands.org/wiki/Building Widelands on macOS/
For more details,
The proposal to merge lp:~widelands-dev/widelands/macos_build_app_compiler into
lp:widelands has been updated.
Description changed to:
One can choose between Clang and GCC now. The Wiki has been changed accordingly:
https://wl.widelands.org/wiki/BuildingWidelandsonmacOS/
For more details,
The proposal to merge lp:~widelands-dev/widelands/macos_build_app_compiler into
lp:widelands has been updated.
Commit message changed to:
choose between compiler clang or gcc, specify build type: debug or release
For more details, see:
Continuous integration builds have changed state:
Travis build 3785. State: passed. Details:
https://travis-ci.org/widelands/widelands/builds/415536722.
Appveyor build 3584. State: success. Details:
The proposal to merge lp:~widelands-dev/widelands/macos_build_app_compiler into
lp:widelands has been updated.
Description changed to:
One can choose between Clang and GCC now. The Wiki has been changed accordingly:
https://wl.widelands.org/wiki/Building_Widelands_on_macOS/
For more details,
The proposal to merge lp:~widelands-dev/widelands/macos_build_app into
lp:widelands has been updated.
Status: Needs review => Merged
For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/macos_build_app/+merge/352945
--
Your team Widelands Developers is subscribed to
Continuous integration builds have changed state:
Travis build 3783. State: failed. Details:
https://travis-ci.org/widelands/widelands/builds/415471904.
Appveyor build 3582. State: success. Details:
Toni Förster has proposed merging
lp:~widelands-dev/widelands/macos_build_app_compiler into lp:widelands with
lp:~widelands-dev/widelands/macos_build_app as a prerequisite.
Commit message:
choose between Clang and GCC
Requested reviews:
Widelands Developers (widelands-dev)
For more details,
Review: Approve
Code LGTM - can you please add a commit messages?
I don't have a Mac, so not tested.
--
https://code.launchpad.net/~widelands-dev/widelands/macos_build_app/+merge/352945
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/macos_build_app.
Review: Approve
Code LGTM :)
--
https://code.launchpad.net/~widelands-dev/widelands/bug-1786163/+merge/352947
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/bug-1786163.
___
Mailing list:
Review: Approve
As far as it concerns me, this looks ok ;-P
--
https://code.launchpad.net/~widelands-dev/widelands/bug-1786597-test-suite/+merge/352928
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/bug-1786597-test-suite.
Review: Approve
Code LGTM :)
--
https://code.launchpad.net/~widelands-dev/widelands/bug-1785404-chat-scrolling/+merge/352941
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/bug-1785404-chat-scrolling.
___
Mailing
Let's wait for
https://code.launchpad.net/~widelands-dev/widelands/bug-1786597-test-suite/+merge/352928
and then merge when Travis goes green.
--
https://code.launchpad.net/~widelands-dev/widelands/bug-1619402-port-work-area-on-expedition/+merge/349594
Your team Widelands Developers is
No, it will fix the test suite. See the changes to the binary file on he bottom
;)
--
https://code.launchpad.net/~widelands-dev/widelands/bug-1786597-test-suite/+merge/352928
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/bug-1786597-test-suite.
Review: Needs Information
Does this end this branch' reason of existence?
--
https://code.launchpad.net/~widelands-dev/widelands/bug-1786597-test-suite/+merge/352928
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/bug-1786597-test-suite.
The proposal to merge lp:~widelands-dev/widelands/bug-1786597-test-suite into
lp:widelands has been updated.
Commit message changed to:
Fix the testsuite for carriers
For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/bug-1786597-test-suite/+merge/352928
--
Your team
Transient error on Travis
@bunnybot merge force
--
https://code.launchpad.net/~widelands-dev/widelands/bug-1786597-test-suite/+merge/352928
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/bug-1786597-test-suite.
___
Continuous integration builds have changed state:
Travis build 3780. State: errored. Details:
https://travis-ci.org/widelands/widelands/builds/415308939.
Appveyor build 3579. State: failed. Details:
Thanks for the review! :)
@bunnybot merge
--
https://code.launchpad.net/~widelands-dev/widelands/bug-1786597-test-suite/+merge/352928
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/bug-1786597-test-suite.
___
Continuous integration builds have changed state:
Travis build 3779. State: errored. Details:
https://travis-ci.org/widelands/widelands/builds/415308135.
Appveyor build 3578. State: failed. Details:
Toni Förster has proposed merging
lp:~widelands-dev/widelands/bug-1786613-10s-return-skipped into lp:widelands.
Commit message:
reduce waiting time to 10ms for skipped programs
Requested reviews:
Widelands Developers (widelands-dev)
Related bugs:
Bug #1786613 in widelands: "production times
The proposal to merge lp:~widelands-dev/widelands/macos_build_app into
lp:widelands has been updated.
Commit message changed to:
The script for building on macOS uses gcc-7 now. Also it checks whether the
SDK10.7 is installed or not and uses the latest installed version if it cannot
find the
Toni Förster has proposed merging
lp:~widelands-dev/widelands/bug-1786613-10s-return-skipped into lp:widelands.
Commit message:
Reduce waiting time to 10ms for skipped programs.
Requested reviews:
Widelands Developers (widelands-dev)
Related bugs:
Bug #1786613 in widelands: "production
@bunnybot merge force
--
https://code.launchpad.net/~widelands-dev/widelands/bug-1786597-test-suite/+merge/352928
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/bug-1786597-test-suite.
___
Mailing list:
GunChleoc has proposed merging
lp:~widelands-dev/widelands/fix-frisian-trainingsites-food-help into
lp:widelands.
Commit message:
Fix food help for Frisian Training Camp.
Requested reviews:
Widelands Developers (widelands-dev)
For more details, see:
Originally reported on the forum: https://wl.widelands.org/forum/topic/4312/
--
https://code.launchpad.net/~widelands-dev/widelands/fix-frisian-trainingsites-food-help/+merge/352977
Your team Widelands Developers is requested to review the proposed merge of
@bunnybot merge
--
https://code.launchpad.net/~widelands-dev/widelands/bug-1619402-port-work-area-on-expedition/+merge/349594
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/bug-1619402-port-work-area-on-expedition.
Yes, if scenarios actually use nonexistent buildings, saveloading will tell the
player so via UI if such a building has been built.
@bunnybot merge
--
https://code.launchpad.net/~widelands-dev/widelands/lenient_allowed_buildings/+merge/351749
Your team Widelands Developers is subscribed to
Toni Förster has proposed merging
lp:~widelands-dev/widelands/missing_animations_barbarians_wood_hardener into
lp:~widelands-dev/widelands/mines-worldsavior.
Commit message:
Use working animations for the barbarians wood hardener.
They where in place but idle animation were used instead.
The proposal to merge lp:~widelands-dev/widelands/bug-1786597-test-suite into
lp:widelands has been updated.
Status: Needs review => Merged
For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/bug-1786597-test-suite/+merge/352928
--
Your team Widelands Developers is
@bunnybot merge
--
https://code.launchpad.net/~widelands-dev/widelands/macos_build_app/+merge/352945
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/macos_build_app.
___
Mailing list:
Toni Förster has proposed merging
lp:~widelands-dev/widelands/remove_executable_bit into lp:widelands.
Commit message:
remove executable bit from files that are not supposed to have it
Requested reviews:
Widelands Developers (widelands-dev)
For more details, see:
40 matches
Mail list logo