[ITP] AtomicParsley

[ITP] AtomicParsley

[David Stacey]

AtomicParsley is a utility for reading and writing metadata tags on MP4 
files. It is found in many Linux distros, including Fedora [1].


# 32-bit:
BASEURL=https://dl.dropboxusercontent.com/u/119453582/Cygwin/32bit/release
wget --no-check-certificate --no-host-directories --force-directories 
--cut-dirs=5 \
${BASEURL}/AtomicParsley/AtomicParsley-0.9.6-1-src.tar.xz \
${BASEURL}/AtomicParsley/AtomicParsley-0.9.6-1.tar.xz \
${BASEURL}/AtomicParsley/AtomicParsley-debuginfo/AtomicParsley-debuginfo-0.9.6-1.tar.xz 
\
${BASEURL}/AtomicParsley/AtomicParsley-debuginfo/setup.hint \
${BASEURL}/AtomicParsley/setup.hint


# 64-bit:
BASEURL=https://dl.dropboxusercontent.com/u/119453582/Cygwin/64bit/release
wget --no-check-certificate --no-host-directories --force-directories 
--cut-dirs=5 \
${BASEURL}/AtomicParsley/AtomicParsley-0.9.6-1-src.tar.xz \
${BASEURL}/AtomicParsley/AtomicParsley-0.9.6-1.tar.xz \
${BASEURL}/AtomicParsley/AtomicParsley-debuginfo/AtomicParsley-debuginfo-0.9.6-1.tar.xz 
\
${BASEURL}/AtomicParsley/AtomicParsley-debuginfo/setup.hint \
${BASEURL}/AtomicParsley/setup.hint


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.

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.

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].

Cheers,

Dave.

[1] http://pkgs.org/search/AtomicParsley
[2] https://packages.debian.org/wheezy/i386/atomicparsley/filelist

Re: [ITP] AtomicParsley

[Yaakov Selkowitz]

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

Re: [ITP] AtomicParsley

[David Stacey]

On 04/06/2014 00:32, Yaakov Selkowitz wrote:
> 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.

Thank you for taking the time to look at this. I think I would prefer to 
keep with 'AtomicParsley', as this reflects the executable name.

>> 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.

It's gone. I wasn't overly keen on the man page either :-)

> 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.

As per your suggestion I have dropped src_compile(). I've also removed 
my custom src_install() - I don't need this since I'm not installing a 
man page anymore. I've kept the Cygwin-specific README, as this is 
generated and it would be more work to remove it than keep it.

If you're happy with this then please could you (or CGF) add an entry 
for AtomicParsley to cygwin-pkg-maint. I'll give the new build one last 
test tomorrow and then upload it.

Thanks again for your constructive suggestions,

Dave.

Re: [ITP] AtomicParsley

[Yaakov Selkowitz]

On 2014-06-04 17:02, David Stacey wrote:
> On 04/06/2014 00:32, Yaakov Selkowitz wrote:
>> 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.
>
> As per your suggestion I have dropped src_compile(). I've also removed
> my custom src_install() - I don't need this since I'm not installing a
> man page anymore. I've kept the Cygwin-specific README, as this is
> generated and it would be more work to remove it than keep it.
>
> If you're happy with this then please could you (or CGF) add an entry
> for AtomicParsley to cygwin-pkg-maint. I'll give the new build one last
> test tomorrow and then upload it.

Added to cygwin-pkg-maint; please upload whenever you're ready.

Thanks for the contribution!


Yaakov