[Buildroot] [PATCH 1/1] Add libselinux and libsepol dependency for SELinux support.

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Fri Jun 24 07:46:24 UTC 2016


Hello,

The title should always contain the package being affected by the
change. I.e, it should be of the form:

	<package>: <description>

In your case:

	busybox: ...

On Thu, 23 Jun 2016 22:43:16 -0400, Adam Duskett wrote:

>  config BR2_PACKAGE_BUSYBOX_SELINUX
> -	select BR2_PACKAGE_LIBSELINUX
> +	depends on BR2_PACKAGE_LIBSELINUX

Why are you doing this? A select is what's appropriate.

> +	depends on BR2_PACKAGE_LIBSEPOL

Why are you adding this? libselinux already selects libsepol, so when
you have libselinux, you have libsepol.

Therefore, this patch looks bogus :-/

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



More information about the buildroot mailing list