Note: This is a beta release of Red Hat Bugzilla 5.0. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Also email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback here.
Bug 226234 - Merge Review: patchutils
Summary: Merge Review: patchutils
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Parag AN(पराग)
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-01-31 20:21 UTC by Nobody's working on this, feel free to take it
Modified: 2007-12-04 18:03 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2007-12-04 18:03:05 UTC
panemade: fedora-review+


Attachments (Terms of Use)

Description Nobody's working on this, feel free to take it 2007-01-31 20:21:23 UTC
Fedora Merge Review: patchutils

http://cvs.fedora.redhat.com/viewcvs/devel/patchutils/
Initial Owner: twaugh@redhat.com

Comment 1 Parag AN(पराग) 2007-12-01 11:18:01 UTC
rpmlint gave me ->
patchutils.src:11: W: unversioned-explicit-obsoletes interdiff
The specfile contains an unversioned Obsoletes: token, which will match all
older, equal and newer versions of the obsoleted thing.  This may cause update
problems, restrict future package/provides naming, and may match something it
was originally not inteded to match -- make the Obsoletes versioned if
possible.

patchutils.src:12: W: unversioned-explicit-provides interdiff
The specfile contains an unversioned Provides: token, which will match all
older, equal, and newer versions of the provided thing.  This may cause
update problems and will make versioned dependencies, obsoletions and conflicts
on the provided thing useless -- make the Provides versioned if possible.

patchutils.src:32: W: make-check-outside-check-section make check
Make check or other automated regression test should be run in %check, as
they can be disabled with a rpm macro for short circuiting purposes.

Also,
 1) use of parallel make check =>
http://fedoraproject.org/wiki/Packaging/Guidelines#head-525c7d76890cb22df33b759c65c35c82bf434d2e

 2) avoid makeinstall usage 
=>http://fedoraproject.org/wiki/Packaging/Guidelines#head-fcaf3e6fcbd51194a5d0dbcfbdd2fcb7791dd002

 3)Good to have defattr used as %defattr(-,root,root,-)

other thing man pages timestamps are changing and not following what upstream
released with timestamps.


Comment 2 Tim Waugh 2007-12-03 12:32:23 UTC
The man pages are deliberately rebuilt so that they are consistent with any
other xmlto-built man pages on the system.

Other fixes applied.

New package: patchutils-0.2.31-4.fc9

Comment 3 Parag AN(पराग) 2007-12-04 12:31:08 UTC
with new updated release packaging looks ok then.

APPROVED.


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