[Bug 226271] New: Merge Review: perl-Net-IP

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226271

Summary: Merge Review: perl-Net-IP
           Product: Fedora Extras
           Version: devel
          Platform: All
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: normal
         Component: Package Review
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]
         QAContact: [e-mail]
                CC: [e-mail]

Fedora Merge Review: perl-Net-IP

http://cvs.fedora.redhat.com/viewcvs/devel/perl-Net-IP/
Initial Owner: [e-mail]

[Bug 226271] Merge Review: perl-Net-IP

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: perl-Net-IP

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226271

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
         AssignedTo|[e-mail]    |[e-mail]
                 CC|                            |[e-mail]
               Flag|                            |fedora-review+

------- Additional Comments From [e-mail]  2007-02-04 11:08 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).

[Bug 226271] Merge Review: perl-Net-IP

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: perl-Net-IP

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226271

------- Additional Comments From [e-mail]  2007-02-04 13:28 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

[Bug 226271] Merge Review: perl-Net-IP

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: perl-Net-IP

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226271

------- Additional Comments From [e-mail]  2007-02-04 13:41 EST -------
Built new package for FC7.  Let me know what you think.

[Bug 226271] Merge Review: perl-Net-IP

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: perl-Net-IP

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226271

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|[e-mail]         |[e-mail]

[Bug 226271] Merge Review: perl-Net-IP

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: perl-Net-IP

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226271

------- Additional Comments From [e-mail]  2007-02-05 14:41 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.

[Bug 226271] Merge Review: perl-Net-IP

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: perl-Net-IP

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226271

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|[e-mail]         |[e-mail]

[Bug 226271] Merge Review: perl-Net-IP

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: perl-Net-IP

https://bugzilla.redhat.com/show_bug.cgi?id=226271

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|normal                      |medium
           Priority|normal                      |medium
            Product|Fedora Extras               |Fedora
            Version|devel                       |rawhide

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|                            |fedora-cvs?

------- Additional Comments From [e-mail]  2008-06-02 06:47 EST -------
Package Change Request
======================
Package Name: perl-Net-IP
New Branches: EL-4
Updated EPEL Owners: kasal mmaslano
Updated EPEL CC: perl-sig

[Bug 226271] Merge Review: perl-Net-IP

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: perl-Net-IP

https://bugzilla.redhat.com/show_bug.cgi?id=226271

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-cvs?                 |fedora-cvs+

------- Additional Comments From [e-mail]  2008-06-02 11:27 EST -------
cvs done.

[Bug 226271] Merge Review: perl-Net-IP

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=226271

Chris Weyl  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |CLOSED
         Resolution|                            |RAWHIDE

--- Comment #7 from Chris Weyl   2009-02-26 18:40:08 EDT ---
Closing... (merge review with fedora-review+)

[Bug 226271] Merge Review: perl-Net-IP

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=226271

Mathieu Bridon  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #8 from 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.

[Bug 226271] Merge Review: perl-Net-IP

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=226271

--- Comment #9 from Robert Scheck  2011-01-19 04:30:38 EST ---
Because the package is in Red Hat Enterprise Linux 5 and 6.