[apparmor] [patch] profiles/Makefile: make sure all profiles have #include <local/...>

Steve Beattie steve at nxnw.org
Mon Jan 20 18:37:52 UTC 2014


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 <steve at nxnw.org>

> === 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
<sbeattie at ubuntu.com>
http://NxNW.org/~steve/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <https://lists.ubuntu.com/archives/apparmor/attachments/20140120/9c6b38a1/attachment.pgp>


More information about the AppArmor mailing list