[Bug 226270] Merge Review: perl-Net-DNS
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-DNS Alias: perl-Net-DNS https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226270 [EMAIL PROTECTED] changed: What|Removed |Added Product|Fedora Extras |Fedora [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |CLOSED Resolution||NEXTRELEASE -- 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 226270] Merge Review: perl-Net-DNS
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-DNS Alias: perl-Net-DNS https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226270 [EMAIL PROTECTED] changed: What|Removed |Added Priority|normal |medium [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Additional Comments From [EMAIL PROTECTED] 2007-05-22 10:58 EST --- Good: + Rpmlint quite on source package. + Rpmlint quite on binary package. + Rpmlint quite on debuginfo package. + Local install/update of package works fine. + Build on Mock works fine. *** APPROVED *** -- 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 226270] Merge Review: perl-Net-DNS
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-DNS Alias: perl-Net-DNS https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226270 [EMAIL PROTECTED] changed: What|Removed |Added AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] Flag||fedora-review? --- Additional Comments From [EMAIL PROTECTED] 2007-04-05 14:10 EST --- o Fixed the buildroot -- This string is in the README: All rights reserved. This program is free software; you may redistribute it and/or modify it under the same terms as Perl itself. Which is sufficient for perl packages - the text of the license doesn't have to be included unless it is included in the source tarball. --- o I can't reproduce the build failure you show maybe the root.log that Paul requests in comment #3 will help. o Fixed the BuildRequires - removed 'perl', and changed the others to the perl(Module::Name) format. --- 0.59-2.fc7 should fix these issues - if the mock build failure is geniune, we can look at the root.log and maybe figure out what's up. -- 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 226270] Merge Review: perl-Net-DNS
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-DNS
Alias: perl-Net-DNS
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226270
--- Additional Comments From [EMAIL PROTECTED] 2007-04-03 12:59 EST ---
Good:
+ Rpmlint quite on source rpm
+ Tar ball matches with upstream.
+ License ok.
+ Local build works fine.
+ Rpmlint quite for binary package
+ Rpmlint quite for debuginfo package.
Bad:
- Wrong Buildroot:
uildRoot should be %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u}
-n)
(wiki: PackagingGuidelines#BuildRoot)
- Perl must not be included as an BR.
BuildRequires: perl should not be included
(wiki: Packaging/Guidelines#Exceptions)
- Package contains no verbatin copy of the license text.
- Mock build failed
--
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 226270] Merge Review: perl-Net-DNS
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-DNS Alias: perl-Net-DNS https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226270 --- Additional Comments From [EMAIL PROTECTED] 2007-04-03 13:00 EST --- Created an attachment (id=151589) -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=151589action=view) log of failed mock build -- 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
