This is the mail archive of the
cygwin-apps
mailing list for the Cygwin project.
Re: [ITP] AtomicParsley
- From: Yaakov Selkowitz <yselkowitz at cygwin dot com>
- To: cygwin-apps at cygwin dot com
- Date: Tue, 03 Jun 2014 18:32:45 -0500
- Subject: Re: [ITP] AtomicParsley
- Authentication-results: sourceware.org; auth=none
- References: <538E1C94 dot 7000101 at tiscali dot co dot uk>
On 2014-06-03 14:05, David Stacey wrote:
AtomicParsley is a utility for reading and writing metadata tags on MP4
files. It is found in many Linux distros, including Fedora [1].
The package name has been capitalised to reflect the name of the
executable and to mirror the Fedora package name. Debian and Ubuntu both
use the lowercase 'atomicparsley' for the package name, but keep the
executable as 'AtomicParsley'. Personally, I prefer the Fedora
convention, but I'm happy to change this if there's precedent elsewhere.
Generally we base this on the tarball name, but I see there aren't
anymore. The old (<=0.9.0) tarballs were CamelCased, but the hg repo is
lowercase (go figure). Under the circumstances, I'd say it's up to you.
I couldn't decide on which category to use in the 'setup.ini' file -
'Utils' seemed a bit broad, so I settled on 'Audio'. I'm not entirely
convinced it fits in that category either, so I'll gladly change the
category if anyone has a strong opinion.
This is fine.
Finally, I've been asked in the past to provide man pages for
executables in my packages. AtomicParsley doesn't come with a man page,
so I've generated one using help2man(1). I'm tempted to drop the man
page though - even the Debian build comes without a man page these days
[2].
help2man only provides good output if --help itself follows certain
conventions; AtomicParsley does not. Since we don't really have a
policy to *require* manpages where they don't already exist (this isn't
Debian), and the resulting manpage doesn't look all that readable
anyway, I suggest you just drop it.
As for the actual build, you should drop src_compile() and use the
default cygautoreconf/cygconf/cygmake combination. Cygwin-specific
READMEs are optional, so you may drop that too if you wish. Otherwise
this looks GTG.
Yaakov