Re: Bug#822728: RFS: libtcod/1.6.0~pre1+dfsg-1 [ITP] -- graphics and utility library for roguelike developers

2016-05-01 Thread Jakub Wilk
* Adam Borowski , 2016-04-27, 18:13: For pre-squeeze, there's "getconf _NPROCESSORS_ONLN". At least one Debian derivative, Dyson, has: draconis:[~]$ dpkg --print-architecture illumos-amd64 draconis:[~]$ getconf _NPROCESSORS_ONLN getconf: Invalid argument

Re: Bug#822728: RFS: libtcod/1.6.0~pre1+dfsg-1 [ITP] -- graphics and utility library for roguelike developers

2016-05-01 Thread Jakub Wilk
* Paul Wise , 2016-04-27, 18:47: And for stretch, there's "dpkg-buildpackage -Jauto" (not to be confused with -jauto, which is evil). -jauto replaces auto with the number of processors. Why is that evil? What does -Jauto do? "dpkg-buildpackage -J..." sets

Bug#822728: RFS: libtcod/1.6.0~pre1+dfsg-1 [ITP] -- graphics and utility library for roguelike developers

2016-04-27 Thread Fabian Wolff
On Wed, Apr 27, 2016 at 05:27:37AM +0200, Adam Borowski wrote: > I'm afraid that it builds only on 64-bit architectures (I tried amd64 and > arm64), on 32-bit ones (x32 armhf i386) it fails with: > > [...] > > Also, it'd be nice if you added --parallel to the dh call, it massively > speeds up

Re: Bug#822728: RFS: libtcod/1.6.0~pre1+dfsg-1 [ITP] -- graphics and utility library for roguelike developers

2016-04-27 Thread Adam Borowski
On Wed, Apr 27, 2016 at 12:40:59PM +0200, Jakub Wilk wrote: > * Adam Borowski , 2016-04-27, 12:02: > >>>export DEB_BUILD_OPTIONS=parallel=`grep ^processor /proc/cpuinfo|wc -l` > >>Refuse the temptation to parse /proc/cpuinfo. Use nproc(1) instead. > >Oh, that's something new

Bug#822728: RFS: libtcod/1.6.0~pre1+dfsg-1 [ITP] -- graphics and utility library for roguelike developers

2016-04-27 Thread Fabian Wolff
Hi everyone, thanks for the reviews! > Fabian, why are you trying to package an upstream snapshot? > (not asking to package 1.5.1, I'm just wondering about why a new library > should > eventually enter Debian in a snapshot form) I'm not packaging a random snapshot. I have based the package on

Re: Bug#822728: RFS: libtcod/1.6.0~pre1+dfsg-1 [ITP] -- graphics and utility library for roguelike developers

2016-04-27 Thread Paul Wise
On Wed, Apr 27, 2016 at 6:40 PM, Jakub Wilk wrote: > And for stretch, there's "dpkg-buildpackage -Jauto" (not to be confused with > -jauto, which is evil). -jauto replaces auto with the number of processors. Why is that evil? What does -Jauto do? -- bye, pabs

Re: Bug#822728: RFS: libtcod/1.6.0~pre1+dfsg-1 [ITP] -- graphics and utility library for roguelike developers

2016-04-27 Thread Jakub Wilk
* Adam Borowski , 2016-04-27, 12:02: export DEB_BUILD_OPTIONS=parallel=`grep ^processor /proc/cpuinfo|wc -l` Refuse the temptation to parse /proc/cpuinfo. Use nproc(1) instead. Oh, that's something new (for the value of "new" of "added upstream on 2009-11-06, to Debian in

Bug#822728: RFS: libtcod/1.6.0~pre1+dfsg-1 [ITP] -- graphics and utility library for roguelike developers

2016-04-27 Thread Gianfranco Costamagna
Hi Adam, >Sure, can do. I did most of the review already, and if we decide otherwise >wrt 1.6-pre1, can always unset. thanks a lot for that, it helps in avoiding double checking of packages :) and double reviews :D ># Note that 1.6 is bleeding edge and still needs polishing before it can >#

Bug#822728: RFS: libtcod/1.6.0~pre1+dfsg-1 [ITP] -- graphics and utility library for roguelike developers

2016-04-27 Thread Adam Borowski
Control: owner -1 ! On Wed, Apr 27, 2016 at 09:45:18AM +, Gianfranco Costamagna wrote: > Adam, do you plan to sponsor the package? in this case can I set you as > owner? :) Sure, can do. I did most of the review already, and if we decide otherwise wrt 1.6-pre1, can always unset. > Fabian,

Re: Bug#822728: RFS: libtcod/1.6.0~pre1+dfsg-1 [ITP] -- graphics and utility library for roguelike developers

2016-04-27 Thread Adam Borowski
On Wed, Apr 27, 2016 at 11:40:47AM +0200, Jakub Wilk wrote: > * Adam Borowski , 2016-04-27, 05:27: > >export DEB_BUILD_OPTIONS=parallel=`grep ^processor /proc/cpuinfo|wc -l` > > Refuse the temptation to parse /proc/cpuinfo. Use nproc(1) instead. Oh, that's something new (for

Bug#822728: RFS: libtcod/1.6.0~pre1+dfsg-1 [ITP] -- graphics and utility library for roguelike developers

2016-04-27 Thread Gianfranco Costamagna
Hi, some general notes: Adam, do you plan to sponsor the package? in this case can I set you as owner? :) Fabian, why are you trying to package an upstream snapshot? (not asking to package 1.5.1, I'm just wondering about why a new library should eventually enter Debian in a snapshot form)

Bug#822728: RFS: libtcod/1.6.0~pre1+dfsg-1 [ITP] -- graphics and utility library for roguelike developers

2016-04-27 Thread Jakub Wilk
* Adam Borowski , 2016-04-27, 05:27: export DEB_BUILD_OPTIONS=parallel=`grep ^processor /proc/cpuinfo|wc -l` Refuse the temptation to parse /proc/cpuinfo. Use nproc(1) instead. -- Jakub Wilk

Bug#822728: RFS: libtcod/1.6.0~pre1+dfsg-1 [ITP] -- graphics and utility library for roguelike developers

2016-04-26 Thread Adam Borowski
On Tue, Apr 26, 2016 at 10:22:32PM +0200, Fabian Wolff wrote: > * Package name: libtcod I'm afraid that it builds only on 64-bit architectures (I tried amd64 and arm64), on 32-bit ones (x32 armhf i386) it fails with: dpkg-gensymbols: warning: some new symbols appeared in the symbols file:

Bug#822728: RFS: libtcod/1.6.0~pre1+dfsg-1 [ITP] -- graphics and utility library for roguelike developers

2016-04-26 Thread Fabian Wolff
Package: sponsorship-requests Severity: wishlist Dear mentors, I am looking for a sponsor for my package "libtcod" * Package name: libtcod Version : 1.6.0~pre1+dfsg-1 Upstream Author : Richard Tew * URL :