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

2014-03-11 Thread noreply
The proposal to merge lp:~widelands-dev/widelands/fix-strings into lp:widelands has been updated. Status: Approved => Merged For more details, see: https://code.launchpad.net/~widelands-dev/widelands/fix-strings/+merge/210465 -- https://code.launchpad.net/~widelands-dev/widelands/fix-string

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

2014-03-11 Thread GunChleoc
Yes, I think this looks fine. It would be great to have the kayboad shortcuts as a table, but that would take some programming. Let's merge it like it is for now. -- https://code.launchpad.net/~widelands-dev/widelands/fix-strings/+merge/210465 Your team Widelands Developers is subscribed to bran

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

2014-03-11 Thread GunChleoc
The proposal to merge lp:~widelands-dev/widelands/fix-strings into lp:widelands has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~widelands-dev/widelands/fix-strings/+merge/210465 -- https://code.launchpad.net/~widelands-dev/widelands/fix-

Re: [Widelands-dev] [Merge] lp:~qcumber-some/widelands/bug1287241 into lp:~widelands-dev/widelands/debian

2014-03-11 Thread Hans Joachim Desserud
I opened a merge request for my merge of build 1:18-3 from Debian (see https://code.launchpad.net/~widelands-dev/widelands/debian-merged-b18-3/+merge/210502). Based on the two comments above, that should resolve this issue. :) -- https://code.launchpad.net/~qcumber-some/widelands/bug1287241/+mer

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/debian-merged-b18-3 into lp:~widelands-dev/widelands/debian

2014-03-11 Thread Hans Joachim Desserud
(Note that this is at the moment completely untested with latest trunk, but I wanted to flag that someone(tm) has done the merge from Debian :) I'll start my test-PPA and look at the results tomorrow.) -- https://code.launchpad.net/~widelands-dev/widelands/debian-merged-b18-3/+merge/210502 Your

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/debian-merged-b18-3 into lp:~widelands-dev/widelands/debian

2014-03-11 Thread Hans Joachim Desserud
Hans Joachim Desserud has proposed merging lp:~widelands-dev/widelands/debian-merged-b18-3 into lp:~widelands-dev/widelands/debian. Requested reviews: Widelands Developers (widelands-dev) For more details, see: https://code.launchpad.net/~widelands-dev/widelands/debian-merged-b18-3/+merge/210

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

2014-03-11 Thread SirVer
Review: Approve Looks good to me, but GunChleoc is mistress of strings imho. -- https://code.launchpad.net/~widelands-dev/widelands/fix-strings/+merge/210465 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/fix-strings.

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

2014-03-11 Thread SirVer
Review: Approve looks good to me. I added one revision adressing all the FIXMEs. hjd, if you are happy with my changes we can merge this imho. -- https://code.launchpad.net/~widelands-dev/widelands/remove-record/+merge/210496 Your team Widelands Developers is subscribed to branch lp:~widelands-

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

2014-03-11 Thread Hans Joachim Desserud
Hans Joachim Desserud has proposed merging lp:~widelands-dev/widelands/remove-record into lp:widelands. Requested reviews: Widelands Developers (widelands-dev) For more details, see: https://code.launchpad.net/~widelands-dev/widelands/remove-record/+merge/210496 Like we talked about back when

Re: [Widelands-dev] [Merge] lp:~qcumber-some/widelands/bug1287241 into lp:~widelands-dev/widelands/debian

2014-03-11 Thread Martin Quinson
I just uploaded a new version of the debian package that incorporates all these cleanups. -- https://code.launchpad.net/~qcumber-some/widelands/bug1287241/+merge/209091 Your team Widelands Developers is requested to review the proposed merge of lp:~qcumber-some/widelands/bug1287241 into lp:~wide

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

2014-03-11 Thread Hans Joachim Desserud
Hans Joachim Desserud has proposed merging lp:~widelands-dev/widelands/fix-strings into lp:widelands. Requested reviews: Widelands Developers (widelands-dev) For more details, see: https://code.launchpad.net/~widelands-dev/widelands/fix-strings/+merge/210465 Various stringfixes, and tweaking

Re: [Widelands-dev] [Merge] lp:~qcumber-some/widelands/bug1287241 into lp:~widelands-dev/widelands/debian

2014-03-11 Thread Martin Quinson
I extended the answer to your question. As I said in there, I have updated the widelands packaging so that the dh arguments are not ignored anymore. I just need some time to polish my changes before uploading them. Thanks for pushing me into debugging this, this was long overdue. -- https://co

Re: [Widelands-dev] [Merge] lp:~qcumber-some/widelands/bug1287241 into lp:~widelands-dev/widelands/debian

2014-03-11 Thread Jens Beyer (Qcumber-some)
I asked at https://answers.launchpad.net/launchpad/+question/245296 what to do. If I interpret the answer correctly, then dh_auto_build should do the parallel building automatically even without --parallel if the build setup process configured it accordingly, which is all done automatically. Thi