Re: Bug#666243: RFS: peg/0.1.9-1 [ITP] -- recursive-descent parser generators for C

2012-07-19 Thread Giulio Paci
On 20/07/2012 00:37, Jakub Wilk wrote: > * Giulio Paci , 2012-07-19, 21:01: >> 2) when you need to patch one of those files (it was required in a >> previous version of the package). > > dpkg-source is supposed to set the same mtime on all patched files, so > this shouldn't be a problem. The fact

Bug#666243: RFS: peg/0.1.9-1 [ITP] -- recursive-descent parser generators for C

2012-07-19 Thread Jakub Wilk
* Giulio Paci , 2012-07-19, 21:01: It happens 1) when you retrieve files from VCS systems or Fair enough. (Although I'd rather touch the generated files in debian/rules than patch upstream makefile.) 2) when you need to patch one of those files (it was required in a previous version of the

Bug#666243: RFS: peg/0.1.9-1 [ITP] -- recursive-descent parser generators for C

2012-07-19 Thread Giulio Paci
On 19/07/2012 13:41, Jakub Wilk wrote: > * Giulio Paci , 2012-07-19, 04:09: >>> The examples directory could act as a test suite. Please run it at >>> build time. >> Done. > > Please honour DEB_BUILD_OPTIONS=nocheck. Fixed. > That's better. You probably want to add a new line (escaped with a dot

Bug#666243: RFS: peg/0.1.9-1 [ITP] -- recursive-descent parser generators for C

2012-07-19 Thread Giulio Paci
On 19/07/2012 18:53, Antti-Juhani Kaijanaho wrote: > On Thu, Jul 19, 2012 at 04:09:25AM +0200, Giulio Paci wrote: >> On 16/07/2012 17:24, Jakub Wilk wrote: -PREFIX = /usr/local +PREFIX = /usr >>> >>> Could be overridden in debian/rules. >> >> As far as I know I should either move the bina

Bug#666243: RFS: peg/0.1.9-1 [ITP] -- recursive-descent parser generators for C

2012-07-19 Thread Antti-Juhani Kaijanaho
On Thu, Jul 19, 2012 at 04:09:25AM +0200, Giulio Paci wrote: > On 16/07/2012 17:24, Jakub Wilk wrote: > >> -PREFIX = /usr/local > >> +PREFIX = /usr > > > > Could be overridden in debian/rules. > > As far as I know I should either move the binaries by hand or patch the > Makefile so that ?= is use

Bug#666243: RFS: peg/0.1.9-1 [ITP] -- recursive-descent parser generators for C

2012-07-19 Thread Jakub Wilk
* Giulio Paci , 2012-07-19, 04:09: The current version appears to be at: http://mentors.debian.net/debian/pool/main/p/peg/peg_0.1.9-1.dsc I updated it. In addition to your recommendations I also changed section from "misc" to "devel". Very well. The examples directory could act as a test su

Bug#666243: RFS: peg/0.1.9-1 [ITP] -- recursive-descent parser generators for C

2012-07-18 Thread Giulio Paci
On 16/07/2012 17:24, Jakub Wilk wrote: > The current version appears to be at: > http://mentors.debian.net/debian/pool/main/p/peg/peg_0.1.9-1.dsc I updated it. In addition to your recommendations I also changed section from "misc" to "devel". > Lintian says: > P: peg source: package-lacks-versio

Bug#666243: RFS: peg/0.1.9-1 [ITP] -- recursive-descent parser generators for C

2012-07-16 Thread Jakub Wilk
* Nathan Owens , 2012-07-16, 13:15: On the no relo issue try putting DPKG_EXPORT_BUILDFLAGS = 1 include /usr/share/dpkg/buildflags.mk in the debian/rules file Nope, this is neither required nor sufficient to fix this particular problem. -- Jakub Wilk -- To UNSUBSCRIBE, email to debian-men

Bug#666243: RFS: peg/0.1.9-1 [ITP] -- recursive-descent parser generators for C

2012-07-16 Thread Nathan Owens
On the no relo issue try putting DPKG_EXPORT_BUILDFLAGS = 1 include /usr/share/dpkg/buildflags.mk in the debian/rules file Jakub Wilk wrote: The current version appears to be at: http://mentors.debian.net/debian/pool/main/p/peg/peg_0.1.9-1.dsc Lintian says: P: peg source: package-lacks-version

Bug#666243: RFS: peg/0.1.9-1 [ITP] -- recursive-descent parser generators for C

2012-07-16 Thread Jakub Wilk
The current version appears to be at: http://mentors.debian.net/debian/pool/main/p/peg/peg_0.1.9-1.dsc Lintian says: P: peg source: package-lacks-versioned-build-depends-on-debhelper 7 W: peg: hardening-no-relro usr/bin/leg W: peg: hardening-no-relro usr/bin/peg blhc confirms that there's a prob