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

2016-01-11 Thread GunChleoc
I will try to look at the code shoonish. Regarding a feature freeze, there are 2 things that still need doing on my end: 1. Font renderer switchover: We have 1 wordwrap bug that needs fixing (https://bugs.launchpad.net/widelands/+bug/1530723), and after it has been fixed, I need to merge https

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

2016-01-11 Thread GunChleoc
I have a VMWare VM, so I could eventually test it. I want to review the prerequisite branch first though (the VM makes my whole machine slow when linking...) -- https://code.launchpad.net/~widelands-dev/widelands/use_image_cache/+merge/282106 Your team Widelands Developers is subscribed to branc

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

2016-01-11 Thread TiborB
My understanding of feature freeze is that we will finish all opened work (f.e. branches waiting for merges, or ones that are really follow ups of what is opened now) but no brand new and unrelated changes/improvements will be started. F.e. I am not going to make any changes to AI, there is only

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

2016-01-11 Thread TiborB
AFAIK VirtualBox is bit worse in regard to 3D acceleration, but testing in VMWare is also very usefull -- https://code.launchpad.net/~widelands-dev/widelands/use_image_cache/+merge/282106 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/full_texture_atlas.

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

2016-01-11 Thread GunChleoc
Review: Approve Some small nits, otherwise LGTM Diff comments: > > === modified file 'src/graphic/make_texture_atlas_main.cc' > --- src/graphic/make_texture_atlas_main.cc2016-01-04 20:54:08 + > +++ src/graphic/make_texture_atlas_main.cc2016-01-07 22:08:20 + > @@ -62,11 +88,121 @

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

2016-01-11 Thread GunChleoc
Is this branch ready for review, or are you still playing with it? -- https://code.launchpad.net/~widelands-dev/widelands/ai_ship_tweaks/+merge/280192 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/ai_ship_tweaks. __

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

2016-01-11 Thread GunChleoc
@bunnybot merge -- https://code.launchpad.net/~widelands-dev/widelands/table_align/+merge/279685 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/table_align. ___ Mailing list: https://launchpad.net/~widelands-dev Post

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

2016-01-11 Thread noreply
The proposal to merge lp:~widelands-dev/widelands/table_align into lp:widelands has been updated. Status: Needs review => Merged For more details, see: https://code.launchpad.net/~widelands-dev/widelands/table_align/+merge/279685 -- Your team Widelands Developers is subscribed to branch lp

[Widelands-dev] [Build #8833225] amd64 build of widelands 1:18-ppa0-bzr7699-201601111817~ubuntu14.04.1 in ubuntu trusty RELEASE [~widelands-dev/ubuntu/widelands-daily]

2016-01-11 Thread Launchpad Buildd System
* Source Package: widelands * Version: 1:18-ppa0-bzr7699-20160817~ubuntu14.04.1 * Architecture: amd64 * Archive: ~widelands-dev/ubuntu/widelands-daily * Component: main * State: Failed to build * Duration: 17 minutes * Build Log: https://launchpad.net/~widelands-dev/+archive/ubuntu/wi

[Widelands-dev] [Build #8833226] i386 build of widelands 1:18-ppa0-bzr7699-201601111817~ubuntu14.04.1 in ubuntu trusty RELEASE [~widelands-dev/ubuntu/widelands-daily]

2016-01-11 Thread Launchpad Buildd System
* Source Package: widelands * Version: 1:18-ppa0-bzr7699-20160817~ubuntu14.04.1 * Architecture: i386 * Archive: ~widelands-dev/ubuntu/widelands-daily * Component: main * State: Failed to build * Duration: 17 minutes * Build Log: https://launchpad.net/~widelands-dev/+archive/ubuntu/wid

[Widelands-dev] [Build #8833282] i386 build of widelands 1:18-ppa0-bzr7699-201601111818~ubuntu16.04.1 in ubuntu xenial RELEASE [~widelands-dev/ubuntu/widelands-daily]

2016-01-11 Thread Launchpad Buildd System
* Source Package: widelands * Version: 1:18-ppa0-bzr7699-20160818~ubuntu16.04.1 * Architecture: i386 * Archive: ~widelands-dev/ubuntu/widelands-daily * Component: main * State: Failed to build * Duration: 17 minutes * Build Log: https://launchpad.net/~widelands-dev/+archive/ubuntu/wid

[Widelands-dev] [Build #8833241] i386 build of widelands 1:18-ppa0-bzr7699-201601111817~ubuntu15.10.1 in ubuntu wily RELEASE [~widelands-dev/ubuntu/widelands-daily]

2016-01-11 Thread Launchpad Buildd System
* Source Package: widelands * Version: 1:18-ppa0-bzr7699-20160817~ubuntu15.10.1 * Architecture: i386 * Archive: ~widelands-dev/ubuntu/widelands-daily * Component: main * State: Failed to build * Duration: 22 minutes * Build Log: https://launchpad.net/~widelands-dev/+archive/ubuntu/wid

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

2016-01-11 Thread TiborB
Of course, it is ready and waiting for review... -- https://code.launchpad.net/~widelands-dev/widelands/ai_ship_tweaks/+merge/280192 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/ai_ship_tweaks. ___ Mailing list: ht

[Widelands-dev] [Build #8833239] i386 build of widelands 1:18-ppa0-bzr7699-201601111817~ubuntu15.04.1 in ubuntu vivid RELEASE [~widelands-dev/ubuntu/widelands-daily]

2016-01-11 Thread Launchpad Buildd System
* Source Package: widelands * Version: 1:18-ppa0-bzr7699-20160817~ubuntu15.04.1 * Architecture: i386 * Archive: ~widelands-dev/ubuntu/widelands-daily * Component: main * State: Failed to build * Duration: 23 minutes * Build Log: https://launchpad.net/~widelands-dev/+archive/ubuntu/wid

[Widelands-dev] [Build #8833240] amd64 build of widelands 1:18-ppa0-bzr7699-201601111817~ubuntu15.10.1 in ubuntu wily RELEASE [~widelands-dev/ubuntu/widelands-daily]

2016-01-11 Thread Launchpad Buildd System
* Source Package: widelands * Version: 1:18-ppa0-bzr7699-20160817~ubuntu15.10.1 * Architecture: amd64 * Archive: ~widelands-dev/ubuntu/widelands-daily * Component: main * State: Failed to build * Duration: 23 minutes * Build Log: https://launchpad.net/~widelands-dev/+archive/ubuntu/wi

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

2016-01-11 Thread SirVer
works like a charm. gonna merge this as it only affects travis. @bunnybot merge -- https://code.launchpad.net/~widelands-dev/widelands/travis_clang/+merge/282123 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/travis_clang into lp:wideland

[Widelands-dev] [Build #8833281] amd64 build of widelands 1:18-ppa0-bzr7699-201601111818~ubuntu16.04.1 in ubuntu xenial RELEASE [~widelands-dev/ubuntu/widelands-daily]

2016-01-11 Thread Launchpad Buildd System
* Source Package: widelands * Version: 1:18-ppa0-bzr7699-20160818~ubuntu16.04.1 * Architecture: amd64 * Archive: ~widelands-dev/ubuntu/widelands-daily * Component: main * State: Failed to build * Duration: 35 minutes * Build Log: https://launchpad.net/~widelands-dev/+archive/ubuntu/wi

[Widelands-dev] [Build #8833238] amd64 build of widelands 1:18-ppa0-bzr7699-201601111817~ubuntu15.04.1 in ubuntu vivid RELEASE [~widelands-dev/ubuntu/widelands-daily]

2016-01-11 Thread Launchpad Buildd System
* Source Package: widelands * Version: 1:18-ppa0-bzr7699-20160817~ubuntu15.04.1 * Architecture: amd64 * Archive: ~widelands-dev/ubuntu/widelands-daily * Component: main * State: Failed to build * Duration: 36 minutes * Build Log: https://launchpad.net/~widelands-dev/+archive/ubuntu/wi

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

2016-01-11 Thread noreply
The proposal to merge lp:~widelands-dev/widelands/travis_clang into lp:widelands has been updated. Status: Needs review => Merged For more details, see: https://code.launchpad.net/~widelands-dev/widelands/travis_clang/+merge/282123 -- Your team Widelands Developers is requested to review th

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

2016-01-11 Thread SirVer
SirVer has proposed merging lp:~widelands-dev/widelands/error_shadow into lp:widelands. Requested reviews: Widelands Developers (widelands-dev) For more details, see: https://code.launchpad.net/~widelands-dev/widelands/error_shadow/+merge/282225 I introduced a couple of warnings in my recent

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

2016-01-11 Thread SirVer
The proposal to merge lp:~widelands-dev/widelands/error_shadow into lp:widelands has been updated. Commit Message changed to: Change -Wshadow into a compile error. For more details, see: https://code.launchpad.net/~widelands-dev/widelands/error_shadow/+merge/282225 -- Your team Widelands Devel

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

2016-01-11 Thread SirVer
The proposal to merge lp:~widelands-dev/widelands/error_shadow into lp:widelands has been updated. Commit Message changed to: Change -Wshadow into a compile error. Run with make -k on travis to get as many errors as possible from each failing run. For more details, see: https://code.launchpad.

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

2016-01-11 Thread GunChleoc
Review: Approve LGTM -- https://code.launchpad.net/~widelands-dev/widelands/error_shadow/+merge/282225 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/error_shadow. ___ Mailing list: https://launchpad.net/~widelands-

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

2016-01-11 Thread SirVer
I will look into running with a minimal texture atlas through a commandline option and profile how much performance that cost on my system. Probably not before the weekend though. In the meantime, more testing with virtual machines would be great! -- https://code.launchpad.net/~widelands-dev/

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

2016-01-11 Thread SirVer
cool, thanks. @bunnybot merge -- https://code.launchpad.net/~widelands-dev/widelands/split_overlay_manager/+merge/282112 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/split_overlay_manager. ___ Mailing list: https:

[Widelands-dev] Bunnybot says...

2016-01-11 Thread bunnybot
Hi, I am bunnybot (https://github.com/widelands/bunnybot). I am keeping the source branch lp:~widelands-dev/widelands/error_shadow mirrored to https://github.com/widelands/widelands/tree/_widelands_dev_widelands_error_shadow You can give me commands by starting a line with @bunnybot . I unders

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

2016-01-11 Thread noreply
The proposal to merge lp:~widelands-dev/widelands/split_overlay_manager into lp:widelands has been updated. Status: Needs review => Merged For more details, see: https://code.launchpad.net/~widelands-dev/widelands/split_overlay_manager/+merge/282112 -- Your team Widelands Developers is subs

[Widelands-dev] Bunnybot says...

2016-01-11 Thread bunnybot
Travis build 251 has changed state to: failed. Details: https://travis-ci.org/widelands/widelands/builds/101708331. -- https://code.launchpad.net/~widelands-dev/widelands/error_shadow/+merge/282225 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/error_shadow.

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

2016-01-11 Thread noreply
The proposal to merge lp:~widelands-dev/widelands-website/copyrightyear_and_links into lp:widelands-website has been updated. Status: Needs review => Merged For more details, see: https://code.launchpad.net/~widelands-dev/widelands-website/copyrightyear_and_links/+merge/281450 -- Your team

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

2016-01-11 Thread SirVer
Great! merged and deployed. thanks :) kaputtnik, would you be willing and interested in learning how to deploy/merge on the server so that I am no longer the single point of failure? -- https://code.launchpad.net/~widelands-dev/widelands-website/copyrightyear_and_links/+merge/281450 Your team Wi