[Bug 796703] Review Request: yawn - Web-based CIM/WBEM browser

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


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

--- Comment #2 from Vitezslav Crhonek <vcrhonek at redhat.com> 2012-02-27 08:34:21 EST ---
Updated version:
http://vcrhonek.fedorapeople.org/yawn/yawn.spec
http://vcrhonek.fedorapeople.org/yawn/yawn-0-0.1.20120227svn561.fc16.src.rpm


(In reply to comment #1)
> rpmlint output is OK, only one warning in compiled package:
>     rpmlint yawn-1.0-2.fc18.noarch.rpm
>     yawn.noarch: W: no-documentation
> 
> Problems found:
> - BuildRoot: tag is obsolete

Removed.

> - Source0: must point to upstream source file, i.e. probably
> http://pywbem.svn.sourceforge.net/viewvc/pywbem/yawn/trunk/yawn.py?revision=561 

Fixed, I followed this guideline:
https://fedoraproject.org/wiki/Packaging:SourceURL

> - version does not match! You use svn checkout and thus you should follow
> https://fedoraproject.org/wiki/Packaging/NamingGuidelines#Snapshot_packages

Fixed (hopefully:)).

> - the package should own all directories it creates, i.e. also /var/www/yawn

Fixed.

> 
> Things to consider:
> - use systemctl instead of /sbin/service to restart httpd

Done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.