Re: [Widelands-dev] [Merge] lp:~hjd/widelands/debian-revert-fonts into lp:widelands

2014-10-19 Thread Hans Joachim Desserud
See https://code.launchpad.net/~hjd/widelands/debian-revert-fonts/+merge/238827 instead. -- https://code.launchpad.net/~hjd/widelands/debian-revert-fonts/+merge/238826 Your team Widelands Developers is requested to review the proposed merge of lp:~hjd/widelands/debian-revert-fonts into lp:widela

[Widelands-dev] [Merge] lp:~hjd/widelands/debian-revert-fonts into lp:widelands

2014-10-19 Thread Hans Joachim Desserud
The proposal to merge lp:~hjd/widelands/debian-revert-fonts into lp:widelands has been updated. Status: Needs review => Rejected For more details, see: https://code.launchpad.net/~hjd/widelands/debian-revert-fonts/+merge/238826 -- https://code.launchpad.net/~hjd/widelands/debian-revert-font

[Widelands-dev] [Merge] lp:~hjd/widelands/debian-revert-fonts into lp:~widelands-dev/widelands/debian

2014-10-19 Thread Hans Joachim Desserud
Hans Joachim Desserud has proposed merging lp:~hjd/widelands/debian-revert-fonts into lp:~widelands-dev/widelands/debian. Requested reviews: Widelands Developers (widelands-dev) For more details, see: https://code.launchpad.net/~hjd/widelands/debian-revert-fonts/+merge/238827 Revert changes f

[Widelands-dev] [Merge] lp:~hjd/widelands/debian-revert-fonts into lp:widelands

2014-10-19 Thread Hans Joachim Desserud
Hans Joachim Desserud has proposed merging lp:~hjd/widelands/debian-revert-fonts into lp:widelands. Requested reviews: Widelands Developers (widelands-dev) For more details, see: https://code.launchpad.net/~hjd/widelands/debian-revert-fonts/+merge/238826 Revert changes for font dependencies.

[Widelands-dev] [Merge] lp:~hjd/widelands/assert-to-exception into lp:widelands

2014-10-19 Thread Hans Joachim Desserud
Hans Joachim Desserud has proposed merging lp:~hjd/widelands/assert-to-exception into lp:widelands. Requested reviews: Widelands Developers (widelands-dev) For more details, see: https://code.launchpad.net/~hjd/widelands/assert-to-exception/+merge/238824 As mentioned, turn this assert into an

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

2014-10-19 Thread wl-zocker
We can also use the bln - I think it should be empty in every case. I am not sure if the old way is 100% errorproof: What if all fields in the region are cluttered with roads? The while loop could run forever. I thought it would be illustrative to show the correct tab, too. The old way could als

[Widelands-dev] [Merge] lp:~hjd/widelands/debian-absolute-paths into lp:~widelands-dev/widelands/debian

2014-10-19 Thread noreply
The proposal to merge lp:~hjd/widelands/debian-absolute-paths into lp:~widelands-dev/widelands/debian has been updated. Status: Needs review => Merged For more details, see: https://code.launchpad.net/~hjd/widelands/debian-absolute-paths/+merge/238818 -- https://code.launchpad.net/~hjd/wide

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

2014-10-19 Thread GunChleoc
Thanks for checking. -- https://code.launchpad.net/~widelands-dev/widelands/i18nfixes/+merge/238137 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/i18nfixes. ___ Mailing list: https://launchpad.net/~widelands-dev Pos

Re: [Widelands-dev] [Merge] lp:~hjd/widelands/debian-absolute-paths into lp:~widelands-dev/widelands/debian

2014-10-19 Thread SirVer
Review: Approve sorry for not notifying you. I forgot about the PPA when writing this patch. And the patch was needed to fix the mac os nighlies. -- https://code.launchpad.net/~hjd/widelands/debian-absolute-paths/+merge/238818 Your team Widelands Developers is subscribed to branch lp:~widelands

[Widelands-dev] [Merge] lp:~hjd/widelands/debian-absolute-paths into lp:~widelands-dev/widelands/debian

2014-10-19 Thread Hans Joachim Desserud
Hans Joachim Desserud has proposed merging lp:~hjd/widelands/debian-absolute-paths into lp:~widelands-dev/widelands/debian. Requested reviews: Widelands Developers (widelands-dev) Related bugs: Bug #1382832 in widelands: "Latest PPA version refuse to start (could not find file or directory:

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

2014-10-19 Thread noreply
The proposal to merge lp:~widelands-dev/widelands/fix_building_statistics into lp:widelands has been updated. Status: Needs review => Merged For more details, see: https://code.launchpad.net/~widelands-dev/widelands/fix_building_statistics/+merge/238699 -- https://code.launchpad.net/~widela

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

2014-10-19 Thread SirVer
Review: Approve -- https://code.launchpad.net/~widelands-dev/widelands/fix_building_statistics/+merge/238699 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/fix_building_statistics. ___ Mailing list: https://launchp

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

2014-10-19 Thread wl-zocker
I changed some strings, please review them. I hope I fixed all issues that we discovered so far. The issue with the missing panel happened again (despite my sleep(100)). I have no idea why. -- https://code.launchpad.net/~widelands-dev/widelands/new-tutorials/+merge/238682 Your team Widelands Dev

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

2014-10-19 Thread GunChleoc
2. How about checking for just 1 of the quarries to complete? The user needs to connect the roads for both of them, so we can assume that if he can manage one, he will manage the other. 3. I had another look at the function. Since it only needs a random empty field, hard coding the start locati