Note: This is a beta release of Red Hat Bugzilla 5.0. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Also email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback here.
Bug 226042 - Merge Review: libselinux
Summary: Merge Review: libselinux
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Adel Gadllah
QA Contact: Fedora Package Reviews List
Depends On:
Blocks: F9MergeReviewTarget
TreeView+ depends on / blocked
Reported: 2007-01-31 19:27 UTC by Nobody's working on this, feel free to take it
Modified: 2008-01-21 22:20 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2008-01-21 22:20:40 UTC
adel.gadllah: fedora-review+

Attachments (Terms of Use)
Patch to adress remaining spec file issues (deleted)
2008-01-16 20:19 UTC, Adel Gadllah
no flags Details | Diff

Description Nobody's working on this, feel free to take it 2007-01-31 19:27:17 UTC
Fedora Merge Review: libselinux
Initial Owner:

Comment 1 Adel Gadllah 2008-01-01 21:30:35 UTC
OK taking this one:

First the rpmlint warnings/errors:

> libselinux.x86_64: W: invalid-license Public domain
> libselinux.x86_64: W: invalid-license uncopyrighted
The license tag is indeed invalid it should be 
"Public Domain" 
> libselinux.x86_64: W: no-url-tag
Why? Please add an URL tag pointing to the upstream homepage
> libselinux-python.x86_64: E: script-without-shebang >
As this file is not intended to be executed directly chmod -x it (mode 0644 not
0755) (If it is then please add a shebang)
libselinux-python.x86_64: W: summary-not-capitalized python bindings for libselinux
OK thats an easy one ;)
libselinux-python.x86_64: E: description-line-too-long The libselinux-python
package contains the python bindings for developing SELinux
Please split this into 80 char lines.

Now general comments to the spec file:

>%files devel

Please remove this file or if you want to ship it for some reason place it in a
-static subpackage.


>%files python

Instead of {_libdir}/python*/site-packages do this:
%{!?python_sitelib: %define python_sitelib %(%{__python} -c "from
distutils.sysconfig import get_python_lib; print get_python_lib()")}
to the top of the spec file and use %{python_sitelib} instead.

>make clean
>make CFLAGS="-g %{optflags}" swigify
>make CFLAGS="-g %{optflags}" all pywrap

Please append %{?_smp_mflags} to the make calls (if it does not break the build).

If you want I can attach a patch with the changes mentioned above to help
speeding up the review process.

Comment 2 Daniel Walsh 2008-01-03 20:39:55 UTC
Fixes are in libselinux-2.0.46-3


Comment 3 Adel Gadllah 2008-01-04 00:32:28 UTC
(In reply to comment #2)
> Fixes are in libselinux-2.0.46-3
> Thanks.

OK, the rpmlint output is now:
"libselinux-python.x86_64: W: no-documentation"
which is fine!

The only remaining issue is the python site-packages path.
Once fixed I will approve the package.

Comment 4 Adel Gadllah 2008-01-04 17:24:52 UTC
one more thing:
should be replaced with
(set the correct modes during %install)

Comment 5 Daniel Walsh 2008-01-04 19:19:26 UTC
adel you are replacing one command with the same.

Comment 6 Adel Gadllah 2008-01-15 16:54:38 UTC
(In reply to comment #5)
> adel you are replacing one command with the same.

... stupid copy and paste sorry; 
I meant "%defattr(-,root,root,-)"

Comment 7 Adel Gadllah 2008-01-16 20:19:25 UTC
Created attachment 291883 [details]
Patch to adress remaining spec file issues

Daniel, this patch should fixes the issues I mentioned in this bug. I moved
libselinux.a into a -static package because it seems to be needed.

Comment 8 Daniel Walsh 2008-01-21 20:59:42 UTC
fixed in libselinux-2.0.47-4.fc9

Comment 9 Adel Gadllah 2008-01-21 21:48:26 UTC
(In reply to comment #8)
> fixed in libselinux-2.0.47-4.fc9

Looks fine now.


Note You need to log in before you can comment on or make changes to this bug.