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=557254

--- Comment #1 from Xavier Bachelot <[email protected]> 2010-01-20 15:52:31 
EST ---
You cannot set the review flag yourself, the reviewer will do.

A couple comment on the spec while I'm at it :

The rpmlint warning is valid, you need to have the version-release in the
specfile changelog, like :
* Thu Jan 21 2010 Hiemanshu Sharma <hiemanshu AT fedoraproject DOT org> 1.5-1

Replace %{_usr}/bin/medusa with %{_bindir}/medusa

The rpm doesn't properly own the directories it creates :
%{_libdir}/medusa and %{_libdir}/medusa/modules/ are not owned.
You can replace the whole block with : %{_libdir}/medusa, everything below it
will be properly owned.

Replace %{_usr}/share/man/man1/medusa.1.gz with %{_mandir}/man1/medusa.1.gz

-- 
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.
_______________________________________________
package-review mailing list
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to