[Buildroot] [PATCH v10] libsepol: bump to 2.7, setools: bump to 4.1.1

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Wed Oct 18 21:26:58 UTC 2017


Hello,

On Tue, 17 Oct 2017 18:32:16 -0400, Adam Duskett wrote:
> The reason for combining these patches is because the
> old version of setools is not compatible iwth libsepol 2.7.  If a user where
> to do a git pull on a patch that only updates libsepol or setools, the build
> would fail to compile.
> 
> setools has been completely rewritten in python instead of C.
> 
> The current version of setools includes a few programs that require
> python-qt5 or python-networkx to run, however the package does not
> check to see if these exist when compiling, and will install the scripts
> to the target directory even if they don't exist.
> 
> In the case of python-networkx, this package is not available on Buildroot.
> The scripts that require them are: sedta and seinfoflow.
> In the case of python-qt5, qpol is the script that requires it.
> 
> Some setools.mk notes to get the package to compile:
> 
> - Convert the package .mk to use python-package instead of autotools-package.
> 
> - setup.py hard codes base_lib_dirs to point to several host directories.
>   To fix this, sed is used before compiling to point the base_lib_dirs to
>   the staging directory.
> 
> - setup.py also includes the "Werror" flag, however compilers before gcc6
>   cause a few autogenerated variables to not be initialized before use,
>   causing the build to fail.
>   To fix this, a patch is provided that removes the Werror flag.
> 
> - Remove sedta and seinfoflow from the target system after install.  These
>   packages rely on the package python-networkx which is not available in
>   buildroot.
> 
> - Remove the installed apol package and the setoolsgui directory from the
>   target directory if python-qt5 is not selected.
> 
> Other changes:
> - Removed all patches, as they are not compatible with the new version of
>   setools.
> 
> - Add COPYING, COPYING.GPL, and COPYING.LGPL to setools.hash
> 
> Signed-off-by: Adam Duskett <Adamduskett at outlook.com>
> ---
> Changes v1 -> v2:
>   - None

Thanks, I've applied the three remaining patches. In the last patch,
there was some trailing whitespace, or whitespaces instead of tabs in
Config.in files. I fixed them up before applying.

Let's see what the autobuilders have to say now :)

Thanks for this work!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com


More information about the buildroot mailing list