Re: RFS: cdpr (updated package)
On Thu, May 12, 2011 at 11:00 AM, Sven Hoexter wrote: > On Thu, May 12, 2011 at 10:29:43AM -0500, Matt Zagrabelny wrote: > > Hi, > >> I've made the relevant changes and re-uploaded to mentors > > Uploaded to the archive. Thanks, Sven! -matt -- 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/BANLkTi=1qzwlgz_j2ibjtqaz2+jpcxg...@mail.gmail.com
Re: RFS: cdpr (updated package)
On Thu, May 12, 2011 at 10:29:43AM -0500, Matt Zagrabelny wrote: Hi, > I've made the relevant changes and re-uploaded to mentors Uploaded to the archive. Sven -- And I don't know much, but I do know this: With a golden heart comes a rebel fist. [ Streetlight Manifesto - Here's To Life ] -- 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/20110512160048.GL2118@marvin
Re: RFS: cdpr (updated package)
On Thu, May 12, 2011 at 7:25 AM, Sven Hoexter wrote: >> http://www.debian.org/doc/manuals/maint-guide/dother.en.html#compat > > That will be almost ever outdated. See man 7 debhelper somewhere near > the end about the compatiblity level and the changes. Sure. >> > A minor point is that you reference /usr/share/common-licenses/GPL >> > which is not necessarily GPL2+ but still not totaly wrong if you make use >> > of the '+'. Opinion may vary on that one. >> >> lintian --pedantic complained. I pinged d-devel before and didn't get >> a clear answer, IIRC. > > I'm not sure if there will ever be a final answer to such questions. I don't > mind it if you use it this way. I decided to set it to GPL-2. It gets rid of the lintian message and is also a bit clearer. I've made the relevant changes and re-uploaded to mentors - URL: http://mentors.debian.net/debian/pool/main/c/cdpr - Source repository: deb-src http://mentors.debian.net/debian unstable main contrib non-free - dget http://mentors.debian.net/debian/pool/main/c/cdpr/cdpr_2.4-1.dsc and also pushed it to the git repo on alioth. Thanks again for the review, Sven. -matt -- 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/banlktinnfqv+kqd0_yxgupjdwsoe9dq...@mail.gmail.com
Re: RFS: cdpr (updated package)
On Thu, May 12, 2011 at 07:04:50AM -0500, Matt Zagrabelny wrote: > On Thu, May 12, 2011 at 4:19 AM, Sven Hoexter wrote: Hi, > Not sure about the breakage with compat level 8. This is where I saw > the reference to 7: > > http://www.debian.org/doc/manuals/maint-guide/dother.en.html#compat That will be almost ever outdated. See man 7 debhelper somewhere near the end about the compatiblity level and the changes. > > A minor point is that you reference /usr/share/common-licenses/GPL > > which is not necessarily GPL2+ but still not totaly wrong if you make use > > of the '+'. Opinion may vary on that one. > > lintian --pedantic complained. I pinged d-devel before and didn't get > a clear answer, IIRC. I'm not sure if there will ever be a final answer to such questions. I don't mind it if you use it this way. Sven -- And I don't know much, but I do know this: With a golden heart comes a rebel fist. [ Streetlight Manifesto - Here's To Life ] -- 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/20110512122558.GJ2118@marvin
Re: RFS: cdpr (updated package)
On Thu, May 12, 2011 at 4:19 AM, Sven Hoexter wrote: > On Wed, May 11, 2011 at 10:55:59AM -0500, Matt Zagrabelny wrote: > > Hi, > >> I am looking for a sponsor for the new version 2.4-1 >> of my package "cdpr". > > * Why are you mangling a prior changelog entry when the changes to > debian/rules did not happen back then? Good question. I must have eaten some bad mushrooms. I'm not even sure when that happened, it's been a while since I looked at this package. > * Will something break with compat level 8? If not, why not switch to > it directly and while you're at it document it in the changelog. Not sure about the breakage with compat level 8. This is where I saw the reference to 7: http://www.debian.org/doc/manuals/maint-guide/dother.en.html#compat > * debian/copyright is a bit delicate. > You list Lance O'Connor while all the file headers list > MonkeyMental.com. I guess Lance O'Connor is the person behind the domain > but IMO debian/copyright doesn't reflect the actual copyright holder > given in the code. Seems wrong to me, I'd mention both. But I'm not sure > how to correctly reflect this in this fancy new file format for > debian/copyright which I don't use. > Regarding this new file format I believe most people reference a revision > in a subversion repository instead of the wiki page. Okay. I'll take a look. > A minor point is that you reference /usr/share/common-licenses/GPL > which is not necessarily GPL2+ but still not totaly wrong if you make use > of the '+'. Opinion may vary on that one. lintian --pedantic complained. I pinged d-devel before and didn't get a clear answer, IIRC. > * Regarding the examples you install, I'd recommend to use dh_installexamples > so they end up in the correct location. Okay. I'll take a look. Thanks for the review, Sven! -matt -- 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/BANLkTi=4H+Cki=HNfFuk=6=8ovg6ntg...@mail.gmail.com
Re: RFS: cdpr (updated package)
On Wed, May 11, 2011 at 10:55:59AM -0500, Matt Zagrabelny wrote: Hi, > I am looking for a sponsor for the new version 2.4-1 > of my package "cdpr". * Why are you mangling a prior changelog entry when the changes to debian/rules did not happen back then? * Will something break with compat level 8? If not, why not switch to it directly and while you're at it document it in the changelog. * debian/copyright is a bit delicate. You list Lance O'Connor while all the file headers list MonkeyMental.com. I guess Lance O'Connor is the person behind the domain but IMO debian/copyright doesn't reflect the actual copyright holder given in the code. Seems wrong to me, I'd mention both. But I'm not sure how to correctly reflect this in this fancy new file format for debian/copyright which I don't use. Regarding this new file format I believe most people reference a revision in a subversion repository instead of the wiki page. A minor point is that you reference /usr/share/common-licenses/GPL which is not necessarily GPL2+ but still not totaly wrong if you make use of the '+'. Opinion may vary on that one. * Regarding the examples you install, I'd recommend to use dh_installexamples so they end up in the correct location. Sven -- And I don't know much, but I do know this: With a golden heart comes a rebel fist. [ Streetlight Manifesto - Here's To Life ] -- 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/20110512091906.GA2118@marvin