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 <[email protected]>

Thanks

> 
> Signed-off-by: Steve Beattie <[email protected]>
> ---
>  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
> <[email protected]>
> http://NxNW.org/~steve/



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

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