SPEC: http://jzeleny.fedorapeople.org/packages/lldpad/lldpad.spec SRPM: http://jzeleny.fedorapeople.org/packages/lldpad/lldpad-0.9.26-1.fc12.src.rpm This is substitude for original dcbd package, it has the same functionality plus it adds lldp protocol support.
[ ] rpmlint $ rpmlint -i ./*.rpm lldpad.src:9: W: mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 9) The specfile mixes use of spaces and tabs for indentation, which is a cosmetic annoyance. Use either spaces or tabs for indentation, not both. lldpad.x86_64: E: incoherent-subsys /etc/rc.d/init.d/lldpad dcbd The filename of your lock file in /var/lock/subsys/ is incoherent with your actual init script name. For example, if your script name is httpd, you have to use 'httpd' as the filename in your subsys directory. It is also possible that rpmlint gets this wrong, especially if the init script contains nontrivial shell variables and/or assignments. These cases usually manifest themselves when rpmlint reports that the subsys name starts a with '$'; in these cases a warning instead of an error is reported and you should check the script manually. 4 packages and 0 specfiles checked; 1 errors, 1 warnings. [*] meets Naming Guidelines [*] meets Licensing Guilelines [*] sources match upstream version [?] all patches have upstream bug link or comment No comments at all (although the meaning is quite obvious from their names). [*] license specified in specfile, license file included [*] specfile in American English [*] specfile is legible [*] valid BuildRoot [*] buildroot cleanup before %install [ ] uses macros consitently Please, use either %{xxx} style or $XXX style macros. Pick a style and use it consistently throughout your packaging. I suggest replacing $RPM_BUILD_ROOT with %{buildroot} Example from your specfile: mkdir -p $RPM_BUILD_ROOT%{_initddir} [*] pkgconfig files in -devel subpackage [*] no static libraries included [*] no staticly linked executables [*] scriptlets requirements splitted [*] %preun, %postun scriptlets running only in certain situations [*] scriptlets are sane [*] package is relocatable [*] all files and directories included [*] permissions and ownership specified [*] all filenames valid UTF-8 [*] contains code, or permissible content [*] documentation doesn't need separate package [*] %doc does not affect runtime [*] subpackages with fully versioned dependency [*] builds in koji Package rename requirements: [*] Provides is correct [*] Obsoletes is correct
Updated SPEC: http://jzeleny.fedorapeople.org/packages/lldpad/lldpad.spec Updated SRPM: http://jzeleny.fedorapeople.org/packages/lldpad/lldpad-0.9.26-2.fc12.src.rpm
Thank you. ACCEPT.
Just to be clear: I'm aware of the fact that this is a re-review.
New Package CVS Request ======================= Package Name: lldpad Short Description: This package contains the Linux user space daemon and configuration tool for Intel LLDP Agent with Enhanced Ethernet support for the Data Center. Owners: jzeleny Branches: F-12 InitialCC:
If your "short description" wraps onto three lines, it's not really "short". I'll use "Daemon and configuration tool for Intel LLDP Agent". Of course, it would be really nice if your %description actually included some mention of what an LLDP agent actually is. This is the kind of thing that's supposed to be noticed as part of the package review.
CVS done (by process-cvs-requests.py). I also added an F-13 branch as that seems to have been missed.
Package added, closing this review request.