Re: RFS: autoconf-archive (updated package)
Hi Bastien, On Fri, Jul 22, 2011 at 02:05:31PM +0200, Bastien ROUCARIES wrote: I am looking for a sponsor for the new version 20110717-1 of my package autoconf-archive. It builds these binary packages: autoconf-archive - Autoconf Macro Archive The package appears to be lintian clean. actually no: E: autoconf-archive source: git-patches-not-exported As we've had this in the last upload already and it seems to be git-pkg and lintian not equally sure on what is considered good and what isn't, I've nevertheless built, signed and uploaded your new version. Thanks! -- Best regards, Kilian signature.asc Description: Digital signature
Re: RFS: autoconf-archive (updated package)
Hi, On Fri, Jul 22, 2011 at 08:02:36PM +0200, Kilian Krause wrote: Hi Bastien, On Fri, Jul 22, 2011 at 02:05:31PM +0200, Bastien ROUCARIES wrote: I am looking for a sponsor for the new version 20110717-1 of my package autoconf-archive. It builds these binary packages: autoconf-archive - Autoconf Macro Archive The package appears to be lintian clean. actually no: E: autoconf-archive source: git-patches-not-exported I knew I had seen this before. Sorry for having just looked at the old version. New version now correctly in the archive. ;-) -- Cheers, Kilian signature.asc Description: Digital signature
Re: RE : : RFS: autoconf-archive (updated package)
Hi Bastien, On Thu, 2011-07-14 at 12:34 +0200, roucaries bastien wrote: I have done the work needed On Wed, Jul 13, 2011 at 11:17 AM, Kilian Krause kil...@debian.org wrote: Hi Bastien, On Tue, 2011-07-12 at 09:17 +0200, roucaries bastien wrote: Ping ? 1. debian/watch needs fixing to avoid your ugly epoch hack: opts=uversionmangle=s/[-\.]//g \ is the correct way to update Done 2. DEP5 is now at http://anonscm.debian.org/viewvc/dep/web/deps/dep5.mdwn?view=markuppathrev=174 Please update in debian/copyright Done 3. Lintian still warns about: E: autoconf-archive source: git-patches-not-exported P: autoconf-archive source: unneeded-build-dep-on-quilt Which seems to be correct as debian/patches is empty except for debian/patches/series. Do you reckon that there should be no patches at all or are they plain missing in your source? They are no patches. It is a bug in gitpkg even after david Bremner patch lintian complain. Renamed to the proper version without epoch, built, signed, uploaded. I've not fixed the unneeded-build-dep-on-quilt warning of lintian, but maybe that could be part of the gitpkg fix that would only ship debian/patches (including debian/patches/series) if it's really required and also ship the quilt-b-dep if it's really required. It may very well be that it's not even required due to the dpkg-source v3. Thanks for your contribution! -- Best regards, Kilian signature.asc Description: This is a digitally signed message part
Re: RE : : RFS: autoconf-archive (updated package)
On Fri, Jul 15, 2011 at 10:47 AM, Kilian Krause kil...@debian.org wrote: Hi Bastien, On Thu, 2011-07-14 at 12:34 +0200, roucaries bastien wrote: I have done the work needed On Wed, Jul 13, 2011 at 11:17 AM, Kilian Krause kil...@debian.org wrote: Hi Bastien, On Tue, 2011-07-12 at 09:17 +0200, roucaries bastien wrote: Ping ? 1. debian/watch needs fixing to avoid your ugly epoch hack: opts=uversionmangle=s/[-\.]//g \ is the correct way to update Done 2. DEP5 is now at http://anonscm.debian.org/viewvc/dep/web/deps/dep5.mdwn?view=markuppathrev=174 Please update in debian/copyright Done 3. Lintian still warns about: E: autoconf-archive source: git-patches-not-exported P: autoconf-archive source: unneeded-build-dep-on-quilt Which seems to be correct as debian/patches is empty except for debian/patches/series. Do you reckon that there should be no patches at all or are they plain missing in your source? They are no patches. It is a bug in gitpkg even after david Bremner patch lintian complain. Renamed to the proper version without epoch, built, signed, uploaded. I've not fixed the unneeded-build-dep-on-quilt warning of lintian, but maybe that could be part of the gitpkg fix that would only ship debian/patches (including debian/patches/series) if it's really required and also ship the quilt-b-dep if it's really required. It may very well be that it's not even required due to the dpkg-source v3. Thanks for your contribution! Thanks to you -- Best regards, Kilian -- To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org Archive: http://lists.debian.org/CAE2SPAYm=SVsEeArjdSWXJP3HFWbj=2nc57zmkary+ajdxg...@mail.gmail.com
Re: RE : : RFS: autoconf-archive (updated package)
On Fri, 15 Jul 2011 10:47:25 +0200, Kilian Krause kil...@debian.org wrote: Non-text part: multipart/signed I've not fixed the unneeded-build-dep-on-quilt warning of lintian, but maybe that could be part of the gitpkg fix that would only ship debian/patches (including debian/patches/series) if it's really required This is already fixed in gitpkg 0.20; the lintian false-positive is about something different; see #633891 for details. and also ship the quilt-b-dep if it's really required. It may very well be that it's not even required due to the dpkg-source v3. The quilt build-dep has nothing to do with gitpkg, as far as I know. I suspect you (and lintian) are correct that it is not needed. d -- To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org Archive: http://lists.debian.org/87aacfkfrf.fsf@zancas.localnet
Re: RE : : RFS: autoconf-archive (updated package)
I have done the work needed On Wed, Jul 13, 2011 at 11:17 AM, Kilian Krause kil...@debian.org wrote: Hi Bastien, On Tue, 2011-07-12 at 09:17 +0200, roucaries bastien wrote: Ping ? 1. debian/watch needs fixing to avoid your ugly epoch hack: opts=uversionmangle=s/[-\.]//g \ is the correct way to update Done 2. DEP5 is now at http://anonscm.debian.org/viewvc/dep/web/deps/dep5.mdwn?view=markuppathrev=174 Please update in debian/copyright Done 3. Lintian still warns about: E: autoconf-archive source: git-patches-not-exported P: autoconf-archive source: unneeded-build-dep-on-quilt Which seems to be correct as debian/patches is empty except for debian/patches/series. Do you reckon that there should be no patches at all or are they plain missing in your source? They are no patches. It is a bug in gitpkg even after david Bremner patch lintian complain. Bastien Apart from that, IMHO ready to be put into Debian. -- Best regards, Kilian -- To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org Archive: http://lists.debian.org/cae2spazvu9jwf+h4yc8pq3hugom1ykjvqoagc-+zf+0cruw...@mail.gmail.com
Re: RE : : RFS: autoconf-archive (updated package)
On Thu, 14 Jul 2011 12:34:30 +0200, roucaries bastien roucaries.bastien+deb...@gmail.com wrote: They are no patches. It is a bug in gitpkg even after david Bremner patch lintian complain. I'm a bit surprised by that. In any case, try version 0.20 of gitpkg which contains a different fix. If there is still a bug, please report it. d -- To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org Archive: http://lists.debian.org/87wrfljdzt.fsf@zancas.localnet
Re: RE : : RFS: autoconf-archive (updated package)
On Thu, Jul 14, 2011 at 1:30 PM, David Bremner brem...@debian.org wrote: On Thu, 14 Jul 2011 12:34:30 +0200, roucaries bastien roucaries.bastien+deb...@gmail.com wrote: They are no patches. It is a bug in gitpkg even after david Bremner patch lintian complain. I'm a bit surprised by that. In any case, try version 0.20 of gitpkg which contains a different fix. If there is still a bug, please report it. I use 0.20, it does not export patches but lintian complain. See the file in mentors. I believe it is a lintian bug. What do you think ? d -- To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org Archive: http://lists.debian.org/CAE2SPAZxo9fWg+LixnA=ryxp_ezob5dfzpzonznfhdov7uy...@mail.gmail.com
Re: RE : : RFS: autoconf-archive (updated package)
On Thu, 14 Jul 2011 16:41:05 +0200, roucaries bastien roucaries.bastien+deb...@gmail.com wrote: On Thu, Jul 14, 2011 at 1:30 PM, David Bremner brem...@debian.org wrote: I use 0.20, it does not export patches but lintian complain. See the file in mentors. I believe it is a lintian bug. What do you think ? Certainly E: autoconf-archive source: unknown-file-in-debian-source git-patches is a bug in lintian. For what it is worth, you refer to the wrong bug on mentors.debian.net. This was 607502, also fixed in 2.5.1 (and in 2.5.0~rc1). d -- To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org Archive: http://lists.debian.org/874o2og32z@convex-new.cs.unb.ca
Re: RE : : RFS: autoconf-archive (updated package)
On Thu, 14 Jul 2011 20:06:06 +0200, roucaries bastien roucaries.bastien+deb...@gmail.com wrote: Not fixed in my case: lintian *.changes || lintian --version E: autoconf-archive source: git-patches-not-exported Lintian v2.5.1 Ah, yes. I agree this is a bug, different than the bug we just fixed in gitpkg. I'm not sure at the moment where this should be fixed. It is not possible for lintian to detect that your git-patches file is trivial, i.e. that it will not result in the export of any patches. On the other hand, it seems like something we should try to support, even if we didn't think about it before. Please file a bug on gitpkg, and we can reassign to lintian if that turns out to be the place to fix it. d -- To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org Archive: http://lists.debian.org/871uxsg0op@convex-new.cs.unb.ca
Re: RE : : RFS: autoconf-archive (updated package)
Hi Bastien, On Tue, 2011-07-12 at 09:17 +0200, roucaries bastien wrote: Ping ? 1. debian/watch needs fixing to avoid your ugly epoch hack: opts=uversionmangle=s/[-\.]//g \ is the correct way to update 2. DEP5 is now at http://anonscm.debian.org/viewvc/dep/web/deps/dep5.mdwn?view=markuppathrev=174 Please update in debian/copyright 3. Lintian still warns about: E: autoconf-archive source: git-patches-not-exported P: autoconf-archive source: unneeded-build-dep-on-quilt Which seems to be correct as debian/patches is empty except for debian/patches/series. Do you reckon that there should be no patches at all or are they plain missing in your source? Apart from that, IMHO ready to be put into Debian. -- Best regards, Kilian signature.asc Description: This is a digitally signed message part
Re: RE : : RFS: autoconf-archive (updated package)
On Wed, 13 Jul 2011 11:17:46 +0200, Kilian Krause kil...@debian.org wrote: 3. Lintian still warns about: E: autoconf-archive source: git-patches-not-exported P: autoconf-archive source: unneeded-build-dep-on-quilt Hi Bastien, Hi Kilian; As lintian-info -t git-patches-not-exported indicates, that warning is because the source package contains a non-empty debian/source/git-patches, but no actual patches. I couldn't actually find a tag or branch named upstream/2011.04.12 in your git repo on collab-maint, but if I use the current head of upstream, then it is identical to debian-patches/2011.04.12. This seems to have triggered a bug in the gitpkg hook which fails to detect that you have not exported any patches. For the moment, you can just comment out the line in debian/source/git-patches (or delete that file), and I'll see about fixing the gitpkg hook. I append a patch, but I'm not sure yet that this is the best fix. David pgppRfioD7q7Q.pgp Description: PGP signature From e0002bf481955279fbd9e930f08afb25403ac00d Mon Sep 17 00:00:00 2001 From: David Bremner brem...@debian.org Date: Wed, 13 Jul 2011 08:23:24 -0300 Subject: [PATCH] quilt-patches-deb-export-hook: Count only non-blank lines in debian/patches/series --- hooks/quilt-patches-deb-export-hook |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/hooks/quilt-patches-deb-export-hook b/hooks/quilt-patches-deb-export-hook index 37afb60..1c74544 100644 --- a/hooks/quilt-patches-deb-export-hook +++ b/hooks/quilt-patches-deb-export-hook @@ -91,7 +91,7 @@ echo # $patch_list exported from git by quilt-patches-deb-export-hook $patc $patch_list ; echo ) | do_patches || exit 1 -count=$(wc -l $patch_dir/series | cut -f1 -d' ') +count=$(grep -c -v '^\s*$' $patch_dir/series | cut -f1 -d' ') if [ $count -eq 1 ]; then echo No patches generated from $patch_list -- 1.7.5.4
Re: RE : : RFS: autoconf-archive (updated package)
Ping ? On Fri, Jul 8, 2011 at 11:03 AM, roucaries bastien roucaries.bastien+deb...@gmail.com wrote: Reuploaded a new version comments online On Thu, Jul 7, 2011 at 5:27 PM, Kilian Krause kil...@debian.org wrote: Hi Bastien, On Thu, 2011-07-07 at 07:50 +0200, roucaries bastien wrote: Please do not upload directly, i will upload git tree before under collab maint and postthe final ppackage here. It is more a rfc. http://mentors.debian.net/debian/pool/main/a/autoconf-archive/autoconf-archive_2011.04.12-1.dsc Comments are: 1. debian/autoconf-archive.doc-base.autoconf-archive should most probably be named debian/autoconf-archive.doc-base Done 2. Me personally I'd prefer seeing that CDBS be ditched in favour of dh I prefer cdbs (taste mater) 3. your indentation in debian/changelog is somewhat lacking the second level for your - items Done 4. The version that's on mentors.d.n does FTBFS for me in pbuilder with: Add texinfo as build depend I could not test under pbuilder uml due to http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=585352 --(snip)-- make[2]: Entering directory `/tmp/buildd/autoconf-archive-2011.04.12/doc' rm -rf autoconf-archive.htp if /bin/bash /tmp/buildd/autoconf-archive-2011.04.12/build-aux/missing --run makeinfo --html -I . \ -o autoconf-archive.htp autoconf-archive.texi; \ then \ rm -rf autoconf-archive.html; \ if test ! -d autoconf-archive.htp test -d autoconf-archive; then \ mv autoconf-archive autoconf-archive.html; else mv autoconf-archive.htp autoconf-archive.html; fi; \ else \ if test ! -d autoconf-archive.htp test -d autoconf-archive; then \ rm -rf autoconf-archive; else rm -Rf autoconf-archive.htp autoconf-archive.html; fi; \ exit 1; \ fi /tmp/buildd/autoconf-archive-2011.04.12/build-aux/missing: line 52: makeinfo: command not found WARNING: `makeinfo' is missing on your system. You should only need it if you modified a `.texi' or `.texinfo' file, or any other file indirectly affecting the aspect of the manual. The spurious call might also be the consequence of using a buggy `make' (AIX, DU, IRIX). You might want to install the `Texinfo' package or the `GNU make' package. Grab either from any GNU archive site. make[2]: *** [autoconf-archive.html] Error 1 make[2]: Leaving directory `/tmp/buildd/autoconf-archive-2011.04.12/doc' make[1]: *** [html-recursive] Error 1 make[1]: Leaving directory `/tmp/buildd/autoconf-archive-2011.04.12' make: *** [debian/stamp-makefile-build] Error 2 dpkg-buildpackage: error: debian/rules build gave error exit status 2 --(snip)-- 5. As Deng Xiyue has retired, I think it'd be ok to remove him entirely from the Uploaders field. Done put myself as maintainer and it is Qingning Huo q...@mayhq.co.uk that retired. Deng Xiyue is still an uploader (but busy by real life) Bastien -- Best regards, Kilian -- To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org Archive: http://lists.debian.org/CAE2SPAZA6=mbrE-H0AQrB5i6RWK-no49u0OKxb22EUZsmS=u...@mail.gmail.com
Re: RE : : RFS: autoconf-archive (updated package)
Reuploaded a new version comments online On Thu, Jul 7, 2011 at 5:27 PM, Kilian Krause kil...@debian.org wrote: Hi Bastien, On Thu, 2011-07-07 at 07:50 +0200, roucaries bastien wrote: Please do not upload directly, i will upload git tree before under collab maint and postthe final ppackage here. It is more a rfc. http://mentors.debian.net/debian/pool/main/a/autoconf-archive/autoconf-archive_2011.04.12-1.dsc Comments are: 1. debian/autoconf-archive.doc-base.autoconf-archive should most probably be named debian/autoconf-archive.doc-base Done 2. Me personally I'd prefer seeing that CDBS be ditched in favour of dh I prefer cdbs (taste mater) 3. your indentation in debian/changelog is somewhat lacking the second level for your - items Done 4. The version that's on mentors.d.n does FTBFS for me in pbuilder with: Add texinfo as build depend I could not test under pbuilder uml due to http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=585352 --(snip)-- make[2]: Entering directory `/tmp/buildd/autoconf-archive-2011.04.12/doc' rm -rf autoconf-archive.htp if /bin/bash /tmp/buildd/autoconf-archive-2011.04.12/build-aux/missing --run makeinfo --html -I . \ -o autoconf-archive.htp autoconf-archive.texi; \ then \ rm -rf autoconf-archive.html; \ if test ! -d autoconf-archive.htp test -d autoconf-archive; then \ mv autoconf-archive autoconf-archive.html; else mv autoconf-archive.htp autoconf-archive.html; fi; \ else \ if test ! -d autoconf-archive.htp test -d autoconf-archive; then \ rm -rf autoconf-archive; else rm -Rf autoconf-archive.htp autoconf-archive.html; fi; \ exit 1; \ fi /tmp/buildd/autoconf-archive-2011.04.12/build-aux/missing: line 52: makeinfo: command not found WARNING: `makeinfo' is missing on your system. You should only need it if you modified a `.texi' or `.texinfo' file, or any other file indirectly affecting the aspect of the manual. The spurious call might also be the consequence of using a buggy `make' (AIX, DU, IRIX). You might want to install the `Texinfo' package or the `GNU make' package. Grab either from any GNU archive site. make[2]: *** [autoconf-archive.html] Error 1 make[2]: Leaving directory `/tmp/buildd/autoconf-archive-2011.04.12/doc' make[1]: *** [html-recursive] Error 1 make[1]: Leaving directory `/tmp/buildd/autoconf-archive-2011.04.12' make: *** [debian/stamp-makefile-build] Error 2 dpkg-buildpackage: error: debian/rules build gave error exit status 2 --(snip)-- 5. As Deng Xiyue has retired, I think it'd be ok to remove him entirely from the Uploaders field. Done put myself as maintainer and it is Qingning Huo q...@mayhq.co.uk that retired. Deng Xiyue is still an uploader (but busy by real life) Bastien -- Best regards, Kilian -- To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org Archive: http://lists.debian.org/CAE2SPAYEugfNzejOG8dkHVRYo=o8mdwoadho+g0qcgtmk-o...@mail.gmail.com
Re: RE : : RFS: autoconf-archive (updated package)
Hi Bastien, On Thu, 2011-07-07 at 07:50 +0200, roucaries bastien wrote: Please do not upload directly, i will upload git tree before under collab maint and postthe final ppackage here. It is more a rfc. http://mentors.debian.net/debian/pool/main/a/autoconf-archive/autoconf-archive_2011.04.12-1.dsc Comments are: 1. debian/autoconf-archive.doc-base.autoconf-archive should most probably be named debian/autoconf-archive.doc-base 2. Me personally I'd prefer seeing that CDBS be ditched in favour of dh 3. your indentation in debian/changelog is somewhat lacking the second level for your - items 4. The version that's on mentors.d.n does FTBFS for me in pbuilder with: --(snip)-- make[2]: Entering directory `/tmp/buildd/autoconf-archive-2011.04.12/doc' rm -rf autoconf-archive.htp if /bin/bash /tmp/buildd/autoconf-archive-2011.04.12/build-aux/missing --run makeinfo --html -I . \ -o autoconf-archive.htp autoconf-archive.texi; \ then \ rm -rf autoconf-archive.html; \ if test ! -d autoconf-archive.htp test -d autoconf-archive; then \ mv autoconf-archive autoconf-archive.html; else mv autoconf-archive.htp autoconf-archive.html; fi; \ else \ if test ! -d autoconf-archive.htp test -d autoconf-archive; then \ rm -rf autoconf-archive; else rm -Rf autoconf-archive.htp autoconf-archive.html; fi; \ exit 1; \ fi /tmp/buildd/autoconf-archive-2011.04.12/build-aux/missing: line 52: makeinfo: command not found WARNING: `makeinfo' is missing on your system. You should only need it if you modified a `.texi' or `.texinfo' file, or any other file indirectly affecting the aspect of the manual. The spurious call might also be the consequence of using a buggy `make' (AIX, DU, IRIX). You might want to install the `Texinfo' package or the `GNU make' package. Grab either from any GNU archive site. make[2]: *** [autoconf-archive.html] Error 1 make[2]: Leaving directory `/tmp/buildd/autoconf-archive-2011.04.12/doc' make[1]: *** [html-recursive] Error 1 make[1]: Leaving directory `/tmp/buildd/autoconf-archive-2011.04.12' make: *** [debian/stamp-makefile-build] Error 2 dpkg-buildpackage: error: debian/rules build gave error exit status 2 --(snip)-- 5. As Deng Xiyue has retired, I think it'd be ok to remove him entirely from the Uploaders field. -- Best regards, Kilian signature.asc Description: This is a digitally signed message part
RE : : RFS: autoconf-archive (updated package)
Please do not upload directly, i will upload git tree before under collab maint and postthe final ppackage here. It is more a rfc. Bastien Le 6 juil. 2011 15:14, Bastien ROUCARIES roucaries.bast...@gmail.com a écrit : Dear mentors, I am looking for a sponsor for the new version 1:2011.04.12-1 of the package autoconf-archive. It builds these binary packages: autoconf-archive - Autoconf Macro Archive The upload would fix these bugs: 214089, 568413, 572669, 584180, 593838, 622369 The package can be found on mentors.debian.net: - URL: http://mentors.debian.net/debian/pool/main/a/autoconf-archive - Source repository: deb-src http://mentors.debian.net/debian unstable main contrib non-free - dget http://mentors.debian.net/debian/pool/main/a/autoconf-archive/autoconf-archive_2011.04.12-1.dsc I would be glad if someone uploaded this package for me. Note that previous maintainer was marked MIA and I will comaintain it with Deng Xiyue Notice also that copyright was fixed and DEP5 They are two lintian warnings that are bug on lintian: - #607694 if you are using stable lintian - new version: autoconf-archive source: git-patches-not-exported but they are exported see debian/patches/series (will report) and I have no patch for now Kind regards bastien roucaries
Re: RFS: autoconf-archive (updated package)
On Tue, May 12, 2009 at 11:37, Deng Xiyue manphiz-gu...@users.alioth.debian.org wrote: Dear mentors, I am looking for a sponsor for the new version 20090426-1 of my package autoconf-archive. It builds these binary packages: autoconf-archive - The Autoconf Macro Archive The package appears to be lintian clean. The package can be found on mentors.debian.net: - URL: http://mentors.debian.net/debian/pool/main/a/autoconf-archive - Source repository: deb-src http://mentors.debian.net/debian unstable main contrib non-free - dget http://mentors.debian.net/debian/pool/main/a/autoconf-archive/autoconf-archive_20090426-1.dsc I tried to contact the main maintainer(CCing) for review, but his box was unavailable, so he granted me to find others for review and sponsor. Thanks Huo. Hence, I would be glad if someone uploaded this package for me. uploaded. -- Best Regards LI Daobing -- To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
