Bug 226234 - Merge Review: patchutils
Summary: Merge Review: patchutils
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review   
(Show other bugs)
Version: rawhide
Hardware: All Linux
Target Milestone: ---
Assignee: Parag AN(पराग)
QA Contact: Fedora Package Reviews List
Depends On:
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:
Story Points: ---
Clone Of:
Last Closed: 2007-12-04 18:03:05 UTC
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
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

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

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.

 1) use of parallel make check =>

 2) avoid makeinstall usage 

 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.


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