Bug 444165

Summary: Review Request: mkrdns - automatic reverse DNS zone generator
Product: [Fedora] Fedora Reporter: Jonathan Steffan <jonathansteffan>
Component: Package ReviewAssignee: Till Maas <opensource>
Status: CLOSED NOTABUG QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: bdpepple, fedora-package-review, notting, opensource
Target Milestone: ---Flags: opensource: fedora‑review-
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2008-12-15 20:29:21 EST Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Bug Depends On:    
Bug Blocks: 201449    

Description Jonathan Steffan 2008-04-25 11:49:06 EDT
Spec URL: http://jsteffan.fedorapeople.org/SRPMS/mkrdns.spec
SRPM URL: http://jsteffan.fedorapeople.org/SRPMS/mkrdns-3.3-1.fc8.src.rpm
Description: An automatic reverse DNS zone generator
Comment 1 Till Maas 2008-05-03 15:44:01 EDT
NEEDSWORK: rpmlint
mkrdns.spec: W: mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 11)
- after Buildarch: and Requires: you inserted tabs but after all other tags you
only used spaces, therefore you should change this. You should remove the
"Requires" anyhow, because they are already added automatically by rpm for perl
requirements.

Naming: ok
License: ok (GPLv2+), not included
Building: ok, builds in koji
Clean section: ok
Macro Usage: ok
Source Package: ok, matches upstream, latest version:
474ffb39d1eb615d0e9d8a7d52a27953  mkrdns-3_3.gz
474ffb39d1eb615d0e9d8a7d52a27953  mkrdns-3.3-1.fc8/mkrdns-3_3.gz

Buildroot: ok
%files: ok

Besides the mixed whitespace, there is another minor issue:
You write /usr/bin/pod2man instead of just pod2man, which does not make sense to
me, because on my machine install is also in /usr/bin and you rely on /usr/bin
being in the path there.

Also the timestamp of the executable is lost, maybe you can change the spec to
preserve it, but I do not know a good way to do this.

In conclusion: Remove the tab after Buildarch, remove the Requires: and remove
the /usr/bin from the pod2man invocation, then I will approve this package.
Comment 2 manuel wolfshant 2008-05-03 22:04:05 EDT
WRT the timestamp": try
  zcat %{SOURCE0} > $RPM_BUILD_ROOT%{_bindir}/mkrdns
  chmod 0755 $RPM_BUILD_ROOT%{_bindir}/mkrdns
  touch -r ${SOURCE0}  $RPM_BUILD_ROOT%{_bindir}/mkrdns
Comment 3 Till Maas 2008-05-14 17:17:37 EDT
ping?
Comment 4 Brian Pepple 2008-08-10 20:19:46 EDT
Are you still wishing to submit this package to Fedora?  There's been no reply to Till's review 3 months ago.
Comment 5 Jonathan Steffan 2008-08-11 15:10:58 EDT
I'll put some more work into this shortly. I sent a patch upstream and never got a response back. I'll try again with upstream, and if that fails, should I carry the patch in the package?
Comment 6 Till Maas 2008-08-13 04:09:49 EDT
You can ship the package with your own patch, if you do not reach upstream. But please keep trying, maybe upstream is on vacation or lost your previous mails.
Comment 7 Till Maas 2008-11-07 06:45:44 EST
Is there some estimate when this review can be finished?
Comment 8 Till Maas 2008-12-15 20:29:21 EST
If you want to finish the package, please reopen.