Bug 611476
Summary: | Review Request: dhcp_probe - Tool for discover DHCP and BootP servers | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Guillermo Gómez <guillermo.gomez> |
Component: | Package Review | Assignee: | Juan Manuel Rodriguez <nushio> |
Status: | CLOSED DUPLICATE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | low | ||
Version: | rawhide | CC: | dbruno, fedora-package-review, notting, nushio, randyn3lrx, terje.rosten |
Target Milestone: | --- | Keywords: | Reopened |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2010-08-18 07:45:27 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Guillermo Gómez
2010-07-05 12:06:26 UTC
I'm not a sponsor but I can tell you a few things just from looking at the source. 1) You misspelled dhcp in the description note its spelled dchp instead. 2) In the spec file the %files section the attributes should be %defattr(-,root,root,-) not %defattr(-,root,root) 3) I'm not too sure about the licensing of the source. It contains GPL licenses in the tar ball but also some of the source code contains the statements like "Copyright, ... All rights reserved". Perhaps there should be some clarification there. Just a few thoughts as I pass by.. Randall, thanks a lot for your comments, first two issues has been corrected. spec : http://gomix.fedorapeople.org/dhcp_probe/1.3.0-2/dhcp_probe.spec srpm : http://gomix.fedorapeople.org/dhcp_probe/1.3.0-2/dhcp_probe-1.3.0-2.fc13.src.rpm $ rpmlint -i dhcp_probe-1.3.0-1.fc13.x86_64.rpm dhcp_probe.x86_64: W: spelling-error %description -l en_US dhcp -> dhow, Dhaka, dhoti The value of this tag appears to be misspelled. Please double-check. 1 packages and 0 specfiles checked; 0 errors, 1 warnings. $ rpmlint -i SPECS/dhcp_probe.spec 0 packages and 1 specfiles checked; 0 errors, 0 warnings. Quick comments: - use version macro in source url - add -p to the install of dhcp_probe.cf (In reply to comment #3) > Quick comments: > - use version macro in source url done > - add -p to the install of dhcp_probe.cf done thanks srpm url: http://gomix.fedorapeople.org/dhcp_probe/1.3.0-3/dhcp_probe-1.3.0-3.fc13.src.rpm spec url: http://gomix.fedorapeople.org/dhcp_probe/1.3.0-3/dhcp_probe.spec $ rpmlint -i dhcp_probe-1.3.0-3.fc13.src.rpm dhcp_probe.spec dhcp_probe.src: W: spelling-error %description -l en_US dhcp -> dhow, Dhaka, dhoti The value of this tag appears to be misspelled. Please double-check. 1 packages and 1 specfiles checked; 0 errors, 1 warnings. Please modify file listing to drop INSTALL and INSTALL.dhcp_probe and add AUTHORS, NEWS and TODO. Does it make sense to ship a init script? koji is happy: http://koji.fedoraproject.org/koji/taskinfo?taskID=2311300 (In reply to comment #5) > Please modify file listing to drop INSTALL and INSTALL.dhcp_probe and add > AUTHORS, NEWS and TODO. :) thanks for noticing, done > Does it make sense to ship a init script? hm, maybe, but still not yet there. New spec and srpm spec url: http://gomix.fedorapeople.org/dhcp_probe/1.3.0-4/dhcp_probe.spec srpm url: http://gomix.fedorapeople.org/dhcp_probe/1.3.0-4/dhcp_probe-1.3.0-4.fc13.src.rpm [gomix@movix 1.3.0-4]$ rpmlint -i dhcp_probe.spec dhcp_probe-1.3.0-4.fc13.src.rpm dhcp_probe.src: W: spelling-error %description -l en_US dhcp -> dhow, Dhaka, dhoti The value of this tag appears to be misspelled. Please double-check. 1 packages and 1 specfiles checked; 0 errors, 1 warnings. abandoning this one, dont have the time... I'm opening the ticket because Daniel Bruno wants to work on it. He'll post the SPEC/SRPM to the package, and I'll continue the review. I'm working on this package now. SPEC URL: http://dbruno.fedorapeople.org/dhcp_probe/dhcp_probe.spec SRPM URL: http://dbruno.fedorapeople.org/dhcp_probe/dhcp_probe-1.3.0-2.fc13.src.rpm rpmlint -i rpmbuild/SPECS/dhcp_probe.spec 0 packages and 1 specfiles checked; 0 errors, 0 warnings. I believe at this point in the game since it has changed owners it would be easier just to close this bug and start another under the new owners account. *** This bug has been marked as a duplicate of bug 624833 *** |