[libvirt] [PATCH] Fix disabling of apparmor security driver

Christophe Fergeau cfergeau at redhat.com
Sun Oct 21 19:06:55 UTC 2012


self NACK, a similar issue seems to be present a bit further in
configure.ac, I'll have to look some more and send a v2

Christophe

On Sun, Oct 21, 2012 at 08:38:49PM +0200, Christophe Fergeau wrote:
> When using --without-apparmor --without-secdriver-apparmor, configure
> will fail saying that AppArmor development package must be installed.
> This is caused by a small bug in --with-secdriver-apparmor handling in
> configure.ac which treats --without-secdriver-apparmor as if the user
> had requested to enable apparmor.
> ---
>  configure.ac | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/configure.ac b/configure.ac
> index 13967e9..6ffcb03 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -1562,7 +1562,8 @@ AC_ARG_WITH([secdriver-apparmor],
>  if test "$with_apparmor" != "yes" ; then
>    if test "$with_secdriver_apparmor" = "check" ; then
>      with_secdriver_apparmor=no
> -  else
> +  fi
> +  if test "$with_secdriver_apparmor" != "no" ; then
>      AC_MSG_ERROR([You must install the AppArmor development package in order to compile libvirt])
>    fi
>  else
> -- 
> 1.7.12.1
> 
> --
> libvir-list mailing list
> libvir-list at redhat.com
> https://www.redhat.com/mailman/listinfo/libvir-list
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20121021/60e85e3a/attachment-0001.sig>


More information about the libvir-list mailing list