[apparmor] [patch 4/3] libapparmor: adjust %pure-parser directive

Seth Arnold seth.arnold at canonical.com
Thu Sep 4 01:04:59 UTC 2014


On Wed, Sep 03, 2014 at 07:39:39AM -0700, Steve Beattie wrote:
> [Sorry, meant this patch to go out with the others.]
> 
> This patch adjusts libapparmor's aalogparse bison grammer to use the
> %pure-parser keyword instead of the deprecated %pure_parser keyword.
> Bison had been warning about this:
> 
>   libraries/libapparmor/src/grammar.y:71.1-12: warning: deprecated directive, use ‘%pure-parser’ [-Wdeprecated]
>    %pure_parser
>     ^^^^^^^^^^^^

Heh, the webpage says %pure-parser is also deprecated and the replacement
is:

%define api.pure

http://www.gnu.org/software/bison/manual/bison.html#g_t_0025define-Summary

(It says the 'full' option was introduced in 2.7 but doesn't say how far
back the 'true' option (same as no option) works; Ubuntu's oldest is 2.3,
which probably works since other %define api.* things were introduced in
2.3 but there's no mention of 'true'...)

This patch is probably fine as-is but we might be back here again soon.

Acked-by: Seth Arnold <seth.arnold at canonical.com>

Thanks

> 
> Signed-off-by: Steve Beattie <steve at nxnw.org>
> ---
>  libraries/libapparmor/src/grammar.y |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> Index: b/libraries/libapparmor/src/grammar.y
> ===================================================================
> --- a/libraries/libapparmor/src/grammar.y
> +++ b/libraries/libapparmor/src/grammar.y
> @@ -68,7 +68,7 @@ aa_record_event_type lookup_aa_event(uns
>  %}
>  
>  %defines
> -%pure_parser
> +%pure-parser
>  %lex-param{void *scanner}
>  %parse-param{void *scanner}
>  
> 
> -- 
> Steve Beattie
> <sbeattie at ubuntu.com>
> http://NxNW.org/~steve/



> -- 
> AppArmor mailing list
> AppArmor at lists.ubuntu.com
> Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/apparmor

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: Digital signature
URL: <https://lists.ubuntu.com/archives/apparmor/attachments/20140903/e304d1d0/attachment.pgp>


More information about the AppArmor mailing list