[apparmor] [patch] aa-notify and $DISPLAY

[Seth Arnold]

On Mon, Oct 10, 2011 at 1:25 PM, Christian Boltz <apparmor at cboltz.de> wrote:
> Hello,
>
> I finally found some time to implement handling of $DISPLAY as discussed
> some days (or weeks?) ago.
>
> The attached patch adds the option --display to set the $DISPLAY
> environment variable.
>
> If $DISPLAY is not set and --display is not used, aa-notify prints a
> warning that notifications won't be shown (exact warning text depends if
> using sudo or not).

Excellent, thanks. (I didn't test it, but it looks good to me. The
nice $sudo_warn_msg tip might be annoying if we ever want to localize
this program but we might as well deal with that later; I'm sure there
are more annoying things that need more attention first.)