On Tue, Aug 25, 2015 at 09:30:58PM +0200, Christian Boltz wrote:
> > Isn't LC_COLLATE=C needed as well? IIRC if it's set it can override
> > $LANG for some aspects of sorting.
> 
> Good question, let's test it ;-)

> [ make-rules-sort-capabilities.diff ]
> 
> === modified file 'common/Make.rules'
> --- common/Make.rules   2015-01-30 21:15:53 +0000
> +++ common/Make.rules   2015-08-25 17:00:00 +0000
> @@ -82,7 +82,7 @@
>  # =====================
>  
>  # emits defined capabilities in a simple list, e.g. "CAP_NAME CAP_NAME2"
> -CAPABILITIES=$(shell echo "\#include <linux/capability.h>" | cpp -dM | 
> LC_ALL=C sed -n -e '/CAP_EMPTY_SET/d' -e 's/^\#define[ 
> \t]\+CAP_\([A-Z0-9_]\+\)[ \t]\+\([0-9xa-f]\+\)\(.*\)$$/CAP_\1/p' | sort)
> +CAPABILITIES=$(shell echo "\#include <linux/capability.h>" | cpp -dM | 
> LC_ALL=C sed -n -e '/CAP_EMPTY_SET/d' -e 's/^\#define[ 
> \t]\+CAP_\([A-Z0-9_]\+\)[ \t]\+\([0-9xa-f]\+\)\(.*\)$$/CAP_\1/p' | LC_ALL=C 
> sort)
>  
>  .PHONY: list_capabilities
>  list_capabilities: /usr/include/linux/capability.h

Nice catch :)

Acked-by: Seth Arnold <[email protected]>

Thanks

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