Hey Christian,

On Sun, Jan 19, 2014 at 08:24:49PM +0100, Christian Boltz wrote:
> the following patch makes sure all profiles have a line
>   #include <local/...>
> 
> BTW: All profiles pass this test :-)  but having an additional check 
> can never hurt.

FYI, this patch doesn't apply, because something (your email client?)
converted all the tabs to 8 spaces. After reverting that, the patch
applies and looks good. Thanks!

Acked-by: Steve Beattie <[email protected]>

> === modified file 'profiles/Makefile'
> --- profiles/Makefile   2013-01-05 00:33:41 +0000
> +++ profiles/Makefile   2014-01-19 19:22:56 +0000
> @@ -44,6 +44,7 @@
>         for profile in ${TOPLEVEL_PROFILES}; do \
>                 fn=$$(basename $$profile); \
>                 echo "# Site-specific additions and overrides for '$$fn'" > 
> ${PROFILES_SOURCE}/local/$$fn; \
> +               grep "include\\s\\s*<local/$$fn>" "$$profile" >/dev/null || { 
> echo "$$profile doesn't contain #include <local/$$fn>" ; exit 1; } ; \
>         done; \
>  
>  .PHONY: install

-- 
Steve Beattie
<[email protected]>
http://NxNW.org/~steve/

Attachment: signature.asc
Description: Digital signature

-- 
AppArmor mailing list
[email protected]
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/apparmor

Reply via email to