[Bug 226271] Merge Review: perl-Net-IP
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 cw...@alumni.drew.edu changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE --- Comment #7 from Chris Weyl cw...@alumni.drew.edu 2009-02-26 18:40:08 EDT --- Closing... (merge review with fedora-review+) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 226271] Merge Review: perl-Net-IP
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 [EMAIL PROTECTED] changed: What|Removed |Added Severity|normal |medium Priority|normal |medium Product|Fedora Extras |Fedora Version|devel |rawhide [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 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 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 226271] Merge Review: perl-Net-IP
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 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Additional Comments From [EMAIL PROTECTED] 2008-06-02 11:27 EST --- cvs done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 226271] Merge Review: perl-Net-IP
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 [EMAIL PROTECTED] changed: What|Removed |Added AssignedTo|[EMAIL PROTECTED] |[EMAIL PROTECTED] -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 226271] Merge Review: perl-Net-IP
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 [EMAIL PROTECTED] 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. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 226271] Merge Review: perl-Net-IP
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 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] CC||[EMAIL PROTECTED] Flag||fedora-review+ --- Additional Comments From [EMAIL PROTECTED] 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). -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 226271] Merge Review: perl-Net-IP
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 [EMAIL PROTECTED] 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
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 226271] Merge Review: perl-Net-IP
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 [EMAIL PROTECTED] 2007-02-04 13:41 EST --- Built new package for FC7. Let me know what you think. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 226271] Merge Review: perl-Net-IP
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 [EMAIL PROTECTED] changed: What|Removed |Added AssignedTo|[EMAIL PROTECTED] |[EMAIL PROTECTED] -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
