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

2016-02-26 Thread TiborB
See my reaction in diff Diff comments: > > === modified file 'src/ai/defaultai.cc' > --- src/ai/defaultai.cc 2016-02-22 08:50:04 + > +++ src/ai/defaultai.cc 2016-02-26 09:06:57 + > @@ -3114,74 +3118,130 @@ > > Map& map = game().map(); > > - // first we try to up

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

2016-02-26 Thread GunChleoc
I have added some code review comments. -- https://code.launchpad.net/~widelands-dev/widelands/ai_upgrading_reworked/+merge/287229 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/ai_upgrading_reworked into lp:widelands. ___

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands-website/fix-installation into lp:widelands-website

2016-02-26 Thread GunChleoc
You are right, for some reason that package wants to be fetched with https. I have tested this again and made a small change - is sounded as if we needed sqlite for mysql. No idea what happend to the branch history. -- https://code.launchpad.net/~widelands-dev/widelands-website/fix-installation