Re: RFS: copher (2nd try)
On Wed, Dec 03, 2008 at 07:06:54PM +0900, Paul Wise wrote: > Package uploaded. > > Please mail this list for future uploads and I will upload if I can. Excellent, cheers :-) Jonathan -- Jonathan Wiltshire signature.asc Description: Digital signature
Re: RFS: copher (2nd try)
On Tue, Dec 2, 2008 at 9:07 PM, Jonathan Wiltshire <[EMAIL PROTECTED]> wrote: > Copher is command line base rather than a GUI, is platform independent, > and works with other -forge-like sites (ex. Rubyforge) and protocols. > So I think it's not a replacement, but complimentary to releaseforge. Fair enough. Package uploaded. Please mail this list for future uploads and I will upload if I can. -- bye, pabs http://wiki.debian.org/PaulWise -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
Re: RFS: copher (2nd try)
On Tue, Dec 02, 2008 at 08:16:49PM +0900, Paul Wise wrote: > I just noticed that releaseforge already exists in Debian. From the > description it sounds like it does the same thing as copher. Is it > still nessecary to add copher to Debian? Copher is command line base rather than a GUI, is platform independent, and works with other -forge-like sites (ex. Rubyforge) and protocols. So I think it's not a replacement, but complimentary to releaseforge. > > -- > bye, > pabs > > http://wiki.debian.org/PaulWise > > > -- > To UNSUBSCRIBE, email to [EMAIL PROTECTED] > with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED] -- Jonathan Wiltshire signature.asc Description: Digital signature
Re: RFS: copher (2nd try)
On Tue, Dec 2, 2008 at 7:49 PM, Jonathan Wiltshire <[EMAIL PROTECTED]> wrote: > Uploaded to > http://mentors.debian.net/debian/pool/main/c/copher/copher_0.1.2+20081201-1.dsc > if you have a chance to look. I just noticed that releaseforge already exists in Debian. From the description it sounds like it does the same thing as copher. Is it still nessecary to add copher to Debian? -- bye, pabs http://wiki.debian.org/PaulWise -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
Re: RFS: copher (2nd try)
On Tue, Dec 02, 2008 at 02:07:24PM +0900, Paul Wise wrote: > A few comments: > > Your package version does not include a Debian revision number. My bad, fixed. > I would use a phrase other than 'project management' in the > descriptions, perhaps 'release management'? > > debian/copyright: You might want to replace "GPL, see above" with "GNU > General Public License version 2 or later, see above". > copher.docs should probably be renamed to copher.examples. I agree, also fixed. > I doubt you need the configure target in debian/rules. Odd, pdebuild whinged at me when I took it out before, but doesn't now. Must have been something else that changed since. > debian/rules doesn't seem to have a .PHONY line, why is that? An oversight. > You got the architecture wrong in debian/control. You want all rather than > any. Fixed. Uploaded to http://mentors.debian.net/debian/pool/main/c/copher/copher_0.1.2+20081201-1.dsc if you have a chance to look. Cheers Jonathan > > -- > bye, > pabs > -- Jonathan Wiltshire signature.asc Description: Digital signature
Re: RFS: copher (2nd try)
On Mon, Dec 1, 2008 at 8:10 PM, Jonathan Wiltshire <[EMAIL PROTECTED]> wrote: > http://mentors.debian.net/debian/pool/main/c/copher/copher_0.1.2+20081201.dsc, > if there are other points to fix just let me know. A few comments: Your package version does not include a Debian revision number. I would use a phrase other than 'project management' in the descriptions, perhaps 'release management'? debian/copyright: You might want to replace "GPL, see above" with "GNU General Public License version 2 or later, see above". I doubt you need the configure target in debian/rules. debian/rules doesn't seem to have a .PHONY line, why is that? copher.docs should probably be renamed to copher.examples. You got the architecture wrong in debian/control. You want all rather than any. -- bye, pabs http://wiki.debian.org/PaulWise -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
Re: RFS: copher (2nd try)
Hi Paul
On Sun, Nov 23, 2008 at 12:01:43PM +0900, Paul Wise wrote:
> It is best not to speculate about future features in the package
> description, please remove the last line.
>
> Uhhh, Depends: is a header for binary packages, not source packages.
> You put ${misc:Depends} in the wrong section:
Done, and misc:Depends - yeh, it was late :)
> The upstream code does not contain any copyright information, please
> ask upstream to fix it (add add your manual page at the same time).
>
> Please also ask upstream to add the standard GPL license grant to the
> script so that there can be no confusion about which version of the
> GPL is to be used.
I spoke to Peter Lunix and he has done both these in CVS, and I've sent
him my man page too. He's not made a release for them yet though,
because there is some other stuff he wants to do first, so I have
checked out the CVS and (I think correctly) compressed it into an
orig.tar.gz, and build the package around that with an epoch.
Also took the opportunity to do some cleaning up, use dh7, etc, so it's
changed quite a bit but I hope for the better. Would you like a
combined changelog or one entry per attempt?
The dsc is at
http://mentors.debian.net/debian/pool/main/c/copher/copher_0.1.2+20081201.dsc,
if there are other points to fix just let me know.
Cheers
Jonathan
>
> --
> bye,
> pabs
>
--
Jonathan Wiltshire
signature.asc
Description: Digital signature
Re: RFS: copher (2nd try)
On Sun, Nov 23, 2008 at 1:30 AM, Jonathan Wiltshire
<[EMAIL PROTECTED]> wrote:
> http://mentors.debian.net/debian/pool/main/c/copher/copher_0.1.2-4.dsc
More comments:
It is best not to speculate about future features in the package
description, please remove the last line.
Uhhh, Depends: is a header for binary packages, not source packages.
You put ${misc:Depends} in the wrong section:
dpkg-gencontrol: warning: unknown information field 'Depends' in input
data in general section of control info file
debian/rules configure doesn't seem to be needed, delete it and remove
it from .PHONY.
The upstream code does not contain any copyright information, please
ask upstream to fix it (add add your manual page at the same time).
Please also ask upstream to add the standard GPL license grant to the
script so that there can be no confusion about which version of the
GPL is to be used.
--
bye,
pabs
http://wiki.debian.org/PaulWise
--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
Re: RFS: copher (2nd try)
Uploaded an updated package to http://mentors.debian.net/debian/pool/main/c/copher/copher_0.1.2-4.dsc Thanks in advance for your feedback. -- Jonathan Wiltshire signature.asc Description: Digital signature
Re: RFS: copher (2nd try)
On Wed, Nov 19, 2008 at 3:50 PM, Paul Wise <[EMAIL PROTECTED]> wrote:
> On Wed, Nov 19, 2008 at 3:44 PM, Jonathan Wiltshire
> <[EMAIL PROTECTED]> wrote:
>
>> Ah, this fixes a query I had from long ago: I had ${misc:Depends} in
>> the build dependencies and it broke things, now that it's in Depends
>> everything is as expected.
>
> build-deps
... are definitely the wrong place for ${misc:Depends}
--
bye,
pabs
http://wiki.debian.org/PaulWise
--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
Re: RFS: copher (2nd try)
On Wed, Nov 19, 2008 at 3:44 PM, Jonathan Wiltshire
<[EMAIL PROTECTED]> wrote:
> Ah, this fixes a query I had from long ago: I had ${misc:Depends} in
> the build dependencies and it broke things, now that it's in Depends
> everything is as expected.
build-deps
> What does this substitution get expanded to and by what? Since taking
> it out made the package build without errors, I didn't take much notice
> of it.
Anything run during the build process can add to it. Specifically:
dpkg -L debhelper | xargs grep misc
You should always use it even if you don't use any of those because
any debhelper command might use it at some point.
The debhelper manual page has a few sentences about it.
--
bye,
pabs
http://wiki.debian.org/PaulWise
--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
Re: RFS: copher (2nd try)
On Mon, Nov 17, 2008 at 03:33:35PM +0900, Paul Wise wrote:
> Standards-Version doesn't need the final .0, 3.8.0.X versions are compatible.
Removed the tailing .0
> Depends should have ${misc:Depends} in it.
Ah, this fixes a query I had from long ago: I had ${misc:Depends} in
the build dependencies and it broke things, now that it's in Depends
everything is as expected.
What does this substitution get expanded to and by what? Since taking
it out made the package build without errors, I didn't take much notice
of it.
> README.Debian doesn't look especially useful.
Agreed, gone.
> Don't forget to send the manual page upstream if you haven't already.
>
> debian/watch looks broken, please see the uscan manual page for the
> correct syntax for magic sf.net support.
Yep, missed this one again! Fixed now, hopefully that's the end of this
bug.
Thanks for you feedback. The updated package is at
http://mentors.debian.net/debian/pool/main/c/copher/copher_0.1.2-4.dsc
if you'd like to have another check.
Cheers
--
Jonathan Wiltshire
signature.asc
Description: Digital signature
Re: RFS: copher (2nd try)
On Mon, Nov 17, 2008 at 3:01 AM, Jonathan Wiltshire
<[EMAIL PROTECTED]> wrote:
> http://mentors.debian.net/debian/pool/main/c/copher/copher_0.1.2-3.dsc
A review of your source package:
Standards-Version doesn't need the final .0, 3.8.0.X versions are compatible.
Depends should have ${misc:Depends} in it.
README.Debian doesn't look especially useful.
Don't forget to send the manual page upstream if you haven't already.
debian/watch looks broken, please see the uscan manual page for the
correct syntax for magic sf.net support.
--
bye,
pabs
http://wiki.debian.org/PaulWise
--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
