Bug 226367
| Summary: | Merge Review: reiserfs-utils | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Nobody's working on this, feel free to take it <nobody> |
| Component: | Package Review | Assignee: | Gwyn Ciesla <gwync> |
| Status: | CLOSED ERRATA | QA Contact: | Fedora Package Reviews List <fedora-package-review> |
| Severity: | medium | Docs Contact: | |
| Priority: | medium | ||
| Version: | rawhide | CC: | 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: | |
| Embargoed: | |||
|
Description
Nobody's working on this, feel free to take it
2007-01-31 20:49:47 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. Ping? Ping again? 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! 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. 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. Thanks! please tell me what's need's to be fixed. (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- 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. (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 ? Looks great. APPROVED. Commit and build. Thanks for finishing this, and taking over the package. |