[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