Re: Review/Approval needed: mfstools

["Tom 'spot' Callaway" <tcallawa redhat com>]

On Thu, 2005-05-19 at 13:58 +0200, Matthias Saou wrote:
> Tom 'spot' Callaway wrote :
> 
> > This package is in CVS and needs review and approval. Its quite small.
> > 
> > MFS Tools is a set of utilities for TiVo drive upgrades. This includes
> > MFS specific backup and restore, as well as MFS volume expansion and
> > shrinking.
> > 
> > No libraries, one binary, and a few symlinks. :)
> > 
> > Please review, thanks.
> 
> Eek!
> 
> Simple points :
> - Missing zlib-devel, which doesn't bother configure but makes the build
>   fail miserably :-/

Fixed.

> - You mix spaces and tabs in your spec file headers -> Readability issues
>   when using 4 spaces vs. 8 for tabs or vice versa.

Yeah, dunno how I did that. :/ Fixed.

> - The release fields seems weird, why two 1s in different places?

OK, so here's the logic. First 1 marks it as a "post" release from 2.0.
The middle is the snapshot revision, and the last digit is the build
number. In these fixes, I'll increment the last digit.

> Now the nasty one : The package contains files with names way too generic!
> And the /usr/share/doc/howto.html file should not be there, copy it to the
> pwd and include it as %doc instead of installing it to _docdir.
> 
> /usr/bin/backup
> /usr/bin/restore
> 
> These files definitely need to have their name changed... not sure what
> it'll break, though.

Well, the only thing its likely to break is the documentation. Those are
symlinks to mfstool. (/usr/bin/backup == /usr/bin/mfstool backup)

And even then, the docs seem to refer to /usr/bin/mfstool backup
instead. Renamed in the spec, should be fixed.

Changes commited to CVS, take two? :)

~spot
-- 
Tom "spot" Callaway: Red Hat Sales Engineer || GPG Fingerprint: 93054260
Fedora Extras Steering Committee Member (RPM Standards and Practices)
Aurora Linux Project Leader: http://auroralinux.org
Lemurs, llamas, and sparcs, oh my!