Merge lp://qastaging/~mitya57/indicator-datetime/lp1502480 into lp://qastaging/indicator-datetime/15.10

It looks fine to me.

In shell, to avoid the loop, I do:
if echo ":$XDG_CURRENT_DESKTOP:" | grep -q ":Unity:"; then
    echo yes
else
    echo no
fi

Performance-wise it's fine, it needs only one malloc (for strcat) instead of many (for split), but I'm not sure how readable it is.

thanks

it might be a bit nicer to move the is_unity in a function which is what is done usually...

+1 on seb128's to move it into a standalone is_unity() function.

Not a dealbreaker, but I'd probably cache the flag so that we don't have to re-evaluate it every time is_unity() is called.

Added a standalone function and caching, please re-review.

LGTM. Thanks Dmitry :)