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

2017-08-09 Thread GunChleoc
@Klaus: Sorry, I overlooked your question. No, you do not need to register at AppVeyor in order to look at logs and download .exe files. -- https://code.launchpad.net/~widelands-dev/widelands/ai_cmakelist/+merge/328630 Your team Widelands Developers is subscribed to branch

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

2017-08-08 Thread noreply
The proposal to merge lp:~widelands-dev/widelands/ai_cmakelist into lp:widelands has been updated. Status: Needs review => Merged For more details, see: https://code.launchpad.net/~widelands-dev/widelands/ai_cmakelist/+merge/328630 -- Your team Widelands Developers is subscribed to bra

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

2017-08-08 Thread GunChleoc
Review: Approve Anything that's in the src dir only needs to be added to a local CMakeList.txt, which you have done. @Klaus: Travis can't do Windows, which is why we have AppVeyor. Let's get this in ASAP, thanks for the fix! @bunnybot merge --

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

2017-08-06 Thread TiborB
Also there are new logic/ai_dna_handler.* files, maybe they should be mentioned somewhere as well.. -- https://code.launchpad.net/~widelands-dev/widelands/ai_cmakelist/+merge/328630 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/ai_cmakelist.

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

2017-08-06 Thread kaputtnik
Thanks Tino :-) Maybe there should be added a comment to the CMakeLists.txt to not forget adjusting the Widelands.iss file if some folders get added/removed? -- https://code.launchpad.net/~widelands-dev/widelands/ai_cmakelist/+merge/328630 Your team Widelands Developers is subscribed to branch

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

2017-08-06 Thread Tino
Yes, new files/directories have to be added to the Innosetup file. Give me a few minutes... -- https://code.launchpad.net/~widelands-dev/widelands/ai_cmakelist/+merge/328630 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/ai_cmakelist.

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

2017-08-06 Thread kaputtnik
>From my understanding the file utils/win32/innosetup/Widelands.iss has to be >adjusted for windows. See: http://bazaar.launchpad.net/~widelands-dev/widelands/trunk/view/head:/utils/win32/innosetup/Widelands.iss Afaik innosetup is the installer program for windows. It's a bit ambiguous that

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

2017-08-06 Thread TiborB
Well, on linux it works, I am but afraid that windows build is not governed by CmakeList, but by some other file - that needs to be updated. On my linux everything works fine, including loading maps. -- https://code.launchpad.net/~widelands-dev/widelands/ai_cmakelist/+merge/328630 Your team

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

2017-08-06 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.

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

2017-08-06 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? --

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

2017-08-06 Thread Klaus Halfmann
ter at appveyor? Will download and cehck this now -- 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:widela

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

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

2017-08-05 Thread bunnybot
/_widelands_dev_widelands_ai_cmakelist-2330. -- 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. ___ Mailing list: https

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

2017-08-05 Thread Klaus Halfmann
is requested to review the proposed merge of lp:~widelands-dev/widelands/ai_cmakelist into lp:widelands. ___ Mailing list: https://launchpad.net/~widelands-dev Post to : widelands-dev@lists.launchpad.net Unsubscribe : https://launchpad.net/~widelands-dev

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

2017-08-05 Thread TiborB
://code.launchpad.net/~widelands-dev/widelands/ai_cmakelist/+merge/328630 I forget to enter AI folder to a list of folders to be copied to installation location -- Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/ai_cmakelist into lp:widelands. === modified

[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