Bug 226367 - Merge Review: reiserfs-utils
Summary: Merge Review: reiserfs-utils
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Gwyn Ciesla
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-01-31 20:49 UTC by Nobody's working on this, feel free to take it
Modified: 2009-04-01 14:48 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-04-01 14:45:18 UTC
Type: ---
Embargoed:
gwync: fedora-review+


Attachments (Terms of Use)

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

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.


Note You need to log in before you can comment on or make changes to this bug.