On Fri, Jan 10, 2014 at 06:50:45PM -0600, Tyler Hicks wrote:
> Change uservars.inc subdomain variable to use the in-tree parser by
> default.
> 
> Also, clean up some commented out subdomain values that don't look to be
> in use any longer and add one commented out value pointing to the system
> parser.
> 
> Signed-off-by: Tyler Hicks <[email protected]>

Unfortunately, the longpath.sh test wants an absolute path to the
parser. My solution for testing in-tree parsers has been:

subdomain=${PWD}/../../../parser/apparmor_parser

With this change, the patch gets an ack from me.

> diff --git a/tests/regression/apparmor/uservars.inc 
> b/tests/regression/apparmor/uservars.inc
> index 02fd516..a8647dc 100644
> --- a/tests/regression/apparmor/uservars.inc
> +++ b/tests/regression/apparmor/uservars.inc
> @@ -1,7 +1,6 @@
>  # 1. Path to apparmor parser
> -subdomain=/sbin/apparmor_parser
> -#subdomain=/home/steve/svn/immunix/SubDomain/parser/subdomain_parser
> -#subdomain=/home/tonyj/subdomain_parser
> +subdomain=../../../parser/apparmor_parser
> +#subdomain=/sbin/apparmor_parser
>  
>  # 2. additional arguments to the apparmor parser
>  parser_args="-q -K"

-- 
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