Hello,

Am Freitag, 31. Juli 2015 schrieb Steve Beattie:
> Signed-off-by: Steve Beattie <[email protected]>
> ---
>  utils/aa-status |   14 ++++++++++++--
>  1 file changed, 12 insertions(+), 2 deletions(-)
> 
> Index: b/utils/aa-status
> ===================================================================
> --- a/utils/aa-status
> +++ b/utils/aa-status
> @@ -12,9 +12,19 @@
> 
>  import re, os, sys, errno
> 
> +# PLEASE NOTE: we try to keep aa-status as minimal as possible, for
> +# environments where installing all of the python utils and python
> +# apparmor module may not make sense. Please think carefully before
> +# importing anything from apparmor; see how the apparmor.fail import
> is +# handled below.
> +
>  # setup exception handling
> -from apparmor.fail import enable_aa_exception_handler
> -enable_aa_exception_handler()
> +try:
> +    from apparmor.fail import enable_aa_exception_handler
> +    enable_aa_exception_handler()
> +except ImportError:
> +    # just let normal python exceptions happen (LP: #1480492)
> +    pass
> 
>  def cmd_enabled():
>      '''Returns error code if AppArmor is not enabled'''

This patch was never commited.

Assuming you still want it, please commit to 2.10 and trunk.

My archive says it's
Acked-by: Seth Arnold <[email protected]>
Acked-by: Christian Boltz <[email protected]> (at least until we need
    more apparmor.* stuff in aa-status)


Regards,

Christian Boltz
-- 
I'll not give more detailed instructions to help you shoot yourself
in your foot :-)  [Stefan Seyfried in opensuse-factory]


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

Reply via email to