[apparmor] [patch] autodetect VENDOR_PERL directly

Kees Cook kees at ubuntu.com
Wed Sep 14 23:56:40 UTC 2011


On Thu, Sep 15, 2011 at 12:18:03AM +0200, Christian Boltz wrote:
> side effect of creating the openSUSE RPM:
> 
> Make the VENDOR_PERL directoy autodetected in utils/Makefile instead of 
> hardcoding it.
> 
> === modified file 'utils/Makefile'
> --- utils/Makefile      2011-09-13 18:58:24 +0000
> +++ utils/Makefile      2011-09-14 22:12:35 +0000
> @@ -42,7 +42,7 @@
>  DESTDIR=/
>  BINDIR=${DESTDIR}/usr/sbin
>  CONFDIR=${DESTDIR}/etc/apparmor
> -VENDOR_PERL?=/usr/lib/perl5/vendor_perl
> +VENDOR_PERL=$(shell perl -e 'use Config; print $$Config{"vendorlib"};')
>  PERLDIR=${DESTDIR}${VENDOR_PERL}/${MODDIR}
>  
>  po/${NAME}.pot: ${TOOLS}

In Ubuntu that resolves to /usr/share/perl5, but looking at the builds, I
think it already moves it from /usr/lib/perl5/vendor_perl to share, so
yeah, this seems fine to me.

Acked-by: Kees Cook <kees at ubuntu.com>

-Kees

-- 
Kees Cook
Ubuntu Security Team



More information about the AppArmor mailing list