[Bug 222594] Review Request: seedit: SELinux Policy Editor

[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: seedit: SELinux Policy Editor


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





------- Additional Comments From ynakam at hitachisoft.jp  2007-01-18 22:36 EST -------
(In reply to comment #6)
>more info:
>your directory /usr/lib/python2.4/site-packages/seedit
>should look like this --> %{python_sitelib}/%{name}/
>where your defined python_sitelib must set above with something like this:
>%define python_sitelib %(%{__python} -c 'from distutils import sysconfig; print
>sysconfig.get_python_lib()')
It is very useful, thanks!
Fixed.

>vendor option in desktop-file-install should set to vendor ""
>set it to fedora is not required any more.
Fixed.

>in %post policy:
>touch %{_datadir}/share/seedit/sepolicy/need-init 
>--> %{_datadir}/%{name}/sepolicy/need-init
>same thing for the other below.
Fixed.

>%doc should be place just after %defattr macros and on the same line.
Fixed.

>missing argument:
>%files -f %{name}.lang
>see Handling Locale Files section in
>http://fedoraproject.org/wiki/Packaging/Guidelines
Fixed, but there is no locale file in seedit package.
locale file exists only in gui subpackage.




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