[Bug 222959] Review Request: smolt - Fedora Hardware Profiler

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: smolt - Fedora Hardware Profiler


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222959





------- Additional Comments From jeff at ocjtech.us  2007-01-18 14:41 EST -------
* source files match upstream (but see below)

$ md5sum smolt-0.3.tar.gz\?format\=raw sources/smolt-0.3.tar.gz 
ec06dc7c2f680bb8b433153e9ce33a25  smolt-0.3.tar.gz?format=raw
ec06dc7c2f680bb8b433153e9ce33a25  sources/smolt-0.3.tar.gz

$ sha1sum smolt-0.3.tar.gz\?format\=raw sources/smolt-0.3.tar.gz 
b8c3162714ca5631bc5104635e86e8d7c7cebb62  smolt-0.3.tar.gz?format=raw
b8c3162714ca5631bc5104635e86e8d7c7cebb62  sources/smolt-0.3.tar.gz

* specfile is properly named, is cleanly written and uses macros consistently.
* dist tag is present.
* build root is correct.
* license field matches the actual license.
* license is open source-compatible.  License text included in package.
* latest version is being packaged.
* BuildRequires are proper.
* compiler flags are appropriate.
* %clean is present.
* package builds in mock (fc6 i386).
* package installs properly. 
* rpmlint says:

E: smolt no-cleaning-of-buildroot %install

%install needs to start with "rm -rf %{buildroot}"

W: smolt no-%build-section

Please add, even if it's empty.

E: smolt non-executable-script /usr/share/smolt/client/reboot.py 0644
E: smolt non-executable-script /usr/share/smolt/client/systemid.py 0644
E: smolt non-executable-script /usr/share/smolt/client/errata.py 0644

This is because there's "#!/usr/bin/python" lines in the files.  They
can either be dealt with in the .spec file with something like this:

%{__sed} -i -e '/^#!/,1d'

or since you are the upstream it's probably better to make the change there
and release a new version.

E: smolt zero-length /etc/sysconfig/hw-uuid

It would probably be better to mark this as %ghost

W: smolt-server no-documentation

When you write some documentation you can add it ;).  Until then we
can ignore...

* %check is not present; There is no test code in the distribution.
* no shared libraries are present
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* scriptlets are OK
* code, not content.
* documentation is small, so no -docs subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.
* no headers
* no unversioned .so file
* no pkconfig file
* no libtool .la droppings.

Needs fixing:

* I'd recommend <https://hosted.fedoraproject.org/projects/smolt> for
  the URL rather than a pointer to the main project page.

* The URL specified for the source doesn't work directly, instead this
  is the URL that gets you the actual package.  Unfortunately that
  won't work for the source line. 

https://hosted.fedoraproject.org/projects/smolt/attachment/wiki/WikiStart/smolt-0.3.tar.gz?format=raw

* smolt package needs to Require wget

* I would recommend a slight modification to the %post script:

if ! -s %{_sysconfdir}/sysconfig/hw-uuid
then
    /bin/cat /proc/sys/kernel/random/uuid > %{_sysconfdir}/sysconfig/hw-uuid
    /bin/chmod 0644 %{_sysconfdir}/sysconfig/hw-uuid
    /bin/chown root:root %{_sysconfdir}/sysconfig/hw-uuid
fi


-- 
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.