Bug 226367

Summary: Merge Review: reiserfs-utils
Product: [Fedora] Fedora Reporter: Nobody's working on this, feel free to take it <nobody>
Component: Package ReviewAssignee: Gwyn Ciesla <gwync>
Status: CLOSED ERRATA QA Contact: Fedora Package Reviews List <fedora-package-review>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: gwync, itamar, jgarzik, mgarski
Target Milestone: ---Flags: gwync: fedora-review+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2009-04-01 14:45:18 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Nobody's working on this, feel free to take it 2007-01-31 20:49:47 UTC
Fedora Merge Review: reiserfs-utils

http://cvs.fedora.redhat.com/viewcvs/devel/reiserfs-utils/
Initial Owner: jgarzik@redhat.com

Comment 1 Gwyn Ciesla 2008-09-16 17:05:18 UTC
rpmlint on SRPM and RPMS are clean.

Upstream Source is 404 due to namesys.com DNS issues, but md5sum matches kernel.org version, which I think is Good Enough under the circumstances.

Any reason it's called reiserfs-utils and not the upstream reiserfsprogs?  I assume some sort of fs-util nameing convention?  If so, document in spec.

Otherwise, full review is great, very tidy package, no other blockers.

Comment 2 Gwyn Ciesla 2008-12-10 14:10:34 UTC
Ping?

Comment 3 Gwyn Ciesla 2009-03-31 15:27:40 UTC
Ping again?

Comment 4 Jeff Garzik 2009-03-31 20:19:40 UTC
Somebody needs to poke Fedora Project about this package, as I have not cared about it in a long, long time.

Feel free to offer to maintain this Fedora package, if you wish!

Comment 5 Jason Tibbitts 2009-03-31 21:52:46 UTC
If you don't want to maintain it, why don't you orphan it?  The only reason it persists in the distro is because you're still listed as the owner.

Comment 6 Jason Tibbitts 2009-03-31 22:04:25 UTC
Actually, I think I'll just save you the trouble.  Sorry for the pkgdb spam you'll receive.  Maybe someone who wants this package will pick it up.

Comment 7 Jeff Garzik 2009-03-31 22:10:26 UTC
Thanks!

Comment 8 Itamar Reis Peixoto 2009-04-01 01:42:45 UTC
please tell me what's need's to be fixed.

Comment 9 acount closed by user 2009-04-01 11:50:24 UTC
(In reply to comment #8)

> please tell me what's need's to be fixed.

Latest release is 3.6.21:

This is first stable release since 2004-10-13, and it contains changes made by
Jeff Mahoney (everything got testing as a part of latest SuSE distros).

2009-01-09
        Patches from Jeff Mahoney:
        - reiserfsprogs-mkreiserfs-quiet.diff
        - reiserfsprogs-large-block-warning.diff
        - reiserfsprogs-fsck-mapid.diff
        - reiserfsprogs-external-journal-changes.diff
        - reiserfsprogs-remove-stupid-fsck_sleep.diff
        - reiserfsprogs-mkfs-use-o_excl.diff
        - reiserfsprogs-enforce-block-limit.diff
        - reiserfsprogs-large-fs.diff
        - reiserfsprogs-better-fsck-a-behavior.diff
        - reiserfsprogs-remove-dependency-on-asm_unaligned.h.diff
        - reiserfsprogs-progress.diff
        - reiserfsprogs-reorder-libs.diff

        Patches from Ludwig Nussel:
        -
mkreiserfs-set-the-owner-of-the-root-directory-to-the-calling-user.diff

        Patches from Edward Shishkin:
        - reiserfsprogs-disable-small-block.diff
        - reiserfsprogs-new-version.diff


The URL tag in the .spec file should be changed to the new $HOME at:
http://ftp.kernel.org/pub/linux/utils/fs/reiserfs/


-thanks-

Comment 10 Gwyn Ciesla 2009-04-01 12:46:18 UTC
Itamar, basically correct the URL tag.  I'd have liked to have had some explanation of the naming being different than upstream (reiserfs-utils/reiserfsprogs), but that would have come from Jeff, ideally, so I'll let that go.

Comment 11 Itamar Reis Peixoto 2009-04-01 14:09:20 UTC
(In reply to comment #10)
> Itamar, basically correct the URL tag.  I'd have liked to have had some
> explanation of the naming being different than upstream
> (reiserfs-utils/reiserfsprogs), but that would have come from Jeff, ideally, so
> I'll let that go.  

the future of reiserfs is still obscure [1], about the name , I recommend to let the current name, this package will probably removed in next merge review

[1] - http://en.wikipedia.org/wiki/Reiser4


http://ispbrasil.com.br/reiserfs-utils/reiserfs-utils.spec
http://ispbrasil.com.br/reiserfs-utils/reiserfs-utils-3.6.21-1.fc11.src.rpm


koji scratch build dist-f11

http://koji.fedoraproject.org/koji/taskinfo?taskID=1270246

Can I go ahead and commit to cvs ?

Comment 12 Gwyn Ciesla 2009-04-01 14:45:18 UTC
Looks great.  APPROVED.  Commit and build.

Thanks for finishing this, and taking over the package.