[Merge] lp:~mterry/webbrowser-app/snap-appid into lp:webbrowser-app
Michael Terry
michael.terry at canonical.com
Mon Feb 13 13:42:56 UTC 2017
Change implemented.
Diff comments:
> === modified file 'snap/webbrowser-app.launcher'
> --- snap/webbrowser-app.launcher 2016-10-07 14:42:55 +0000
> +++ snap/webbrowser-app.launcher 2017-02-09 22:21:05 +0000
> @@ -5,6 +5,6 @@
> export OXIDE_NO_SANDBOX=1
>
> # Explicitly set APP_ID.
> -export APP_ID=webbrowser-app
> +export APP_ID=webbrowser-app_webbrowser-app_${SNAP_REVISION}
Because nothing else about that script seemed very dynamic to me, and we didn't have an environment variable for the command name yet. So some part of the APP_ID would be hardcoded regardless.
But it's easy enough to change (will in one second), and I could indeed imagine someone forking the snap for testing purposes. So sensible enough of a change.
>
> exec "$SNAP/bin/desktop-launch" "webbrowser-app" --desktop_file_hint=unity8 "$@"
--
https://code.launchpad.net/~mterry/webbrowser-app/snap-appid/+merge/316909
Your team Ubuntu Phablet Team is requested to review the proposed merge of lp:~mterry/webbrowser-app/snap-appid into lp:webbrowser-app.
More information about the Ubuntu-reviews
mailing list