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

2017-08-05 Thread TiborB
The proposal to merge lp:~widelands-dev/widelands/ai_cmakelist into lp:widelands has been updated. Commit Message changed to: CMakeList.txt modification - adding AI folder to be copied to installation location. For more details, see: https://code.launchpad.net/~widelands-dev/widelands/ai_cmake

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

2017-08-05 Thread TiborB
TiborB has proposed merging lp:~widelands-dev/widelands/ai_cmakelist into lp:widelands. Commit message: CMakeList.txt modification - adding AI folder to be copied to installation location. Requested reviews: Widelands Developers (widelands-dev) For more details, see: https://code.launchpad.n

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

2017-08-05 Thread Klaus Halfmann
When will be executed? I assume when the final binay is build. So the test would be to check if e.g. travis binary contains that folder. I will wait for bunnybot and then check. -- https://code.launchpad.net/~widelands-dev/widelands/ai_cmakelist/+merge/328630 Your team Widelands Developers is requ

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

2017-08-05 Thread bunnybot
Continuous integration builds have changed state: Travis build 2506. State: passed. Details: https://travis-ci.org/widelands/widelands/builds/261323352. Appveyor build 2330. State: success. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_ai_cmakeli

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

2017-08-05 Thread Klaus Halfmann
Once again I dont see the windows build? Where in travis is this hidden? -- https://code.launchpad.net/~widelands-dev/widelands/ai_cmakelist/+merge/328630 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/ai_cmakelist into lp:widelands. _

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

2017-08-05 Thread Klaus Halfmann
Hmm, that Link in the message: > https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_ai_cmakelist-2330. gives me. Build not found or access denied. I can find it however at: https://ci.appveyor.com/project/widelands-dev/widelands Do I have to register at appveyo

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

2017-08-05 Thread Klaus Halfmann
Review: Needs Fixing Mhh, did not find that folder in the fnal deployment? D:\develop\wldev\ai_cmakelist\data>dir /B campaigns i18n images locale maps scripting shaders sound tribes txts world can someone check on linux? -- https://code.launchpad.net/~widelands-dev/widelands/ai_cmakelist/+merg

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

2017-08-05 Thread Klaus Halfmann
Review: Needs Fixing Uhmm this does not even run. when starting a network gae it dies with: ... GameSaver::save() took 98ms SaveHandler::save_game() took 102ms InternetGaming: logout(SERVER_CRASHED) [NetClient] Closing network socket connected to 2a01:238:4326:5100:b5ce:5c0b:f1f4:2e40:7395. Obje