Bug 226277 - Merge Review: perl-RPM-Specfile
Merge Review: perl-RPM-Specfile
Status: CLOSED CURRENTRELEASE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Steven Pritchard
Fedora Package Reviews List
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2007-01-31 15:37 EST by Nobody's working on this, feel free to take it
Modified: 2009-02-05 04:26 EST (History)
1 user (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2009-02-05 04:26:12 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
steve: fedora‑review+


Attachments (Terms of Use)
spec patch (2.14 KB, patch)
2007-03-20 14:44 EDT, Steven Pritchard
no flags Details | Diff

  None (edit)
Description Nobody's working on this, feel free to take it 2007-01-31 15:37:44 EST
Fedora Merge Review: perl-RPM-Specfile

http://cvs.fedora.redhat.com/viewcvs/devel/perl-RPM-Specfile/
Initial Owner: rnorwood@redhat.com
Comment 1 Steven Pritchard 2007-03-20 14:44:25 EDT
Created attachment 150516 [details]
spec patch

- Fix find option order.
- Use fixperms macro instead of our own chmod incantation.
- Improve description.
- Add Archive::Tar, ExtUtils::MakeMaker, and URI::Escape build deps.
- Remove redundant YAML explicit dep.
- Package README.
Comment 2 Steven Pritchard 2007-06-27 18:56:20 EDT
My patch still applies fine...
Comment 3 Robin Norwood 2007-06-28 11:20:36 EDT
applied patch and built for f8!

I think this needs a '+' fedora-review from you before closing.
Comment 4 Steven Pritchard 2007-06-28 15:45:05 EDT
Apparently I missed two (really, really minor) things that rpmlint complains 
about:

W: perl-RPM-Specfile macro-in-%changelog _libdir
W: perl-RPM-Specfile mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 
16)

It looks like you can fix those with the following if you want:

  sed -i -e '100s/%/%%/;16s/\t/     /' perl-RPM-Specfile.spec

I think those are minor enough to call this APPROVED though.
Comment 5 Robin Norwood 2007-07-03 12:24:58 EDT
fixed in 1.51-4
Comment 6 Steven Pritchard 2008-05-08 15:05:26 EDT
We can close this now, right?
Comment 7 Marcela Mašláňová 2008-12-15 03:29:41 EST
This package is needed in EPEL-{4,5} for perl-Satcon. I can maintain it for EPEL, if you don't want it.
Comment 8 Marcela Mašláňová 2008-12-15 03:44:23 EST
Hm false alarm. This package have branches for RHEL, but it wasn't included into comps.
Comment 9 Marcela Mašláňová 2009-02-05 04:26:12 EST
Ok, now we can close it because problem with EPEL/RHEL will be solved in different tickets.

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