[apparmor] [patch] aa-notify and $DISPLAY
seth.arnold at gmail.com
Tue Oct 11 22:15:12 UTC 2011
On Mon, Oct 10, 2011 at 1:25 PM, Christian Boltz <apparmor at cboltz.de> wrote:
> 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.)
More information about the AppArmor