Code review comment for lp://staging/~ted/unity8/no-home-applications

On Tue, 2016-04-26 at 12:10 +0000, Michael Terry wrote:
> - The old code looked in ~/.local/share/applications, but this now
> looks just in .cache/unity8/click-desktops.
To be clear, it doesn't "look in", it gets called when that directory
is updated. The actual existence of the directory isn't important to
this. So the person calling it is click itself, there is no file watch.
> (A) I don't have that directory on my phone, who makes that? and
Click will make the directory when the hook runs. If you build a deb
and install it on your device you'll have to log in and out to make it
appear. But for anyone updating with images it should be when the
update.
> (B) the old location covered more than just clicks. Though I guess
> that would be sufficient for the Touch use case.
Yes, that's what I was targeting, but if there are use-cases for the
home directory we can talk about it.
> - The script's dbus check could more nicely look like:
>
> set -a
>
> if [ -z "$DBUS_SESSION_BUS_ADDRESS" -a -n "$XDG_RUNTIME_DIR" ] ; then
> . "$XDG_RUNTIME_DIR/dbus-session"
> fi
>

I'm not sure if the click user hooks always end up with XDG_RUNTIME_DIR
set. They're basically run through Python code that changes user in
cases of sideloading. It seems if XDG_RUNTIME_DIR is set, it is likely
that the dbus variables are set.