Bug 226270 - (perl-Net-DNS) Merge Review: perl-Net-DNS
Merge Review: perl-Net-DNS
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: Jochen Schmitt
Fedora Package Reviews List
: Reopened
Depends On:
  Show dependency treegraph
Reported: 2007-01-31 15:36 EST by Nobody's working on this, feel free to take it
Modified: 2012-08-10 09:01 EDT (History)
5 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2010-10-23 23:25:05 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
jochen: fedora‑review+

Attachments (Terms of Use)
log of failed mock build (69.10 KB, text/plain)
2007-04-03 13:00 EDT, Jochen Schmitt
no flags Details

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

Initial Owner: rnorwood@redhat.com
Comment 1 Jochen Schmitt 2007-04-03 12:59:07 EDT
+ 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.

- 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

Comment 2 Jochen Schmitt 2007-04-03 13:00:15 EDT
Created attachment 151589 [details]
log of failed mock build
Comment 3 Paul Howarth 2007-04-03 13:07:58 EDT
Can you post the root.log as well? I'm curious as to how the buildreq
perl-Net-IP was satisfied since the absence of this module is what caused the
build to fail.

(the perl-Net-IP buildreq should really be perl(Net::IP) of course)
Comment 4 Robin Norwood 2007-04-05 14:10:39 EDT
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.
Comment 5 Jochen Schmitt 2007-05-22 10:58:53 EDT
+ 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 ***

Comment 6 Paul Wouters 2010-10-15 10:04:35 EDT
Package Change Request
Package Name: perl-Net-DNS
New Branches: el-5 el-6
Owners: mmaslano pwouters 
InitialCC: mmaslano

perl-Net-DNS is needed as dependancy for perl-Net-DNSSEC which is already in el-5/el-6 but cannot be build now due to this missing dependancy.

It does not matter to me who will be the owner. It can be the original package owner, or I can take it on, or we can co-maintain....
Comment 7 Jason Tibbitts 2010-10-15 10:11:47 EDT
Erm, no, perl-Net-DNS is as far as I can tell a part of base RHEL.  We can't branch it for EPEL.
Comment 8 Paul Wouters 2010-10-23 23:25:05 EDT
ok. i guess perl-Net-DNS-SEC should be removed from el-5 then. I'll look into it.
Comment 9 Petr Pisar 2012-08-10 08:43:33 EDT
Package Change Request
Package Name: perl-Net-DNS
Branches: f16 f17 f18
InitialCC: perl-sig

Please add perl-sig user with watch* permissions only to all Fedora branches.
Comment 10 Gwyn Ciesla 2012-08-10 09:01:11 EDT

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