[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