Bug 226271 - Merge Review: perl-Net-IP
Merge Review: perl-Net-IP
Status: CLOSED RAWHIDE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Robin Norwood
Fedora Package Reviews List
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2007-01-31 15:36 EST by Nobody's working on this, feel free to take it
Modified: 2011-01-19 04:30 EST (History)
4 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2009-02-26 18:40:08 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
ville.skytta: fedora‑review+
kevin: fedora‑cvs+


Attachments (Terms of Use)

  None (edit)
Description Nobody's working on this, feel free to take it 2007-01-31 15:36:45 EST
Fedora Merge Review: perl-Net-IP

http://cvs.fedora.redhat.com/viewcvs/devel/perl-Net-IP/
Initial Owner: rnorwood@redhat.com
Comment 1 Ville Skyttä 2007-02-04 11:08:40 EST
Just a couple of small issues that can be fixed directly in CVS, no need for a
review roundtrip:

- Source URL is broken (missing slash), should be "ftp://cpan...", not 
  "ftp:/cpan..."

- Change "-type d -depth" to "-depth -type d" in %install to avoid a (discarded)
  warning from find(1).
Comment 2 Robin Norwood 2007-02-04 13:28:27 EST
Thanks for the review, Ville.

I noticed a couple of other things that I fixed as well:

o rpmlint complained about mixed-spaces-and-tabs.  Fixed.

o Removed the redundant BuildRequires: perl

o Fixed the Release: to include magic %{?dist} macro
Comment 3 Robin Norwood 2007-02-04 13:41:00 EST
Built new package for FC7.  Let me know what you think.
Comment 4 Ville Skyttä 2007-02-05 14:41:51 EST
Looks good, although even if currently redundant with today's specified minimal
Fedora buildroot package lists, I would have kept the perl build dependency (I
don't expect it to stick around in the list of "assumed present" packages
forever).  But that's just a personal preference.
Comment 5 Marcela Mašláňová 2008-06-02 06:47:45 EDT
Package Change Request
======================
Package Name: perl-Net-IP
New Branches: EL-4
Updated EPEL Owners: kasal mmaslano
Updated EPEL CC: perl-sig
Comment 6 Kevin Fenzi 2008-06-02 11:27:48 EDT
cvs done.
Comment 7 Chris Weyl 2009-02-26 18:40:08 EST
Closing... (merge review with fedora-review+)
Comment 8 Mathieu Bridon 2011-01-19 04:16:41 EST
This package is in EPEL4, but not in EPEL5 and 6. Is there any reason for that?

I would like to see this package in EPEL6, and I am willing to help co-maintain it.
Comment 9 Robert Scheck 2011-01-19 04:30:38 EST
Because the package is in Red Hat Enterprise Linux 5 and 6.

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