[Bug 1792604] Re: Installing yaru should try to remove the communitheme snap

2018-09-24 Thread Didier Roche
As we discussed, I'm working on the immediate issue: moving to a separate binary package. This is done on https://bugs.launchpad.net/ubuntu/+source/gnome-session/+bug/1794024 -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu.

[Bug 1792604] Re: Installing yaru should try to remove the communitheme snap

2018-09-17 Thread Olivier Tilloy
Tested this in a PPA, and it works as expected. One minor issue is that when uninstalling the communitheme snap while the theme is in use, icons in the dock change on the fly, and that doesn't look great. That would happen only during upgrade from bionic and to users actually using the

[Bug 1792604] Re: Installing yaru should try to remove the communitheme snap

2018-09-14 Thread Olivier Tilloy
The following should do the trick (will require thorough testing): $ cat debian/yaru-theme-gtk.postinst #!/bin/sh set -e # When first installing yaru-theme-gtk (i.e. when upgrading from bionic), # remove the communitheme snap if present (https://launchpad.net/bugs/1792604). if [ "$1" =

[Bug 1792604] Re: Installing yaru should try to remove the communitheme snap

2018-09-14 Thread Olivier Tilloy
lxd has some logic in the deb package to install the snap in its preinst script (sort of the opposite of what we're trying to achieve here), this can be interesting: https://github.com/lxc/lxd-pkg-ubuntu/blob/dpm- cosmic/debian/lxd.preinst -- You received this bug notification because you are a

[Bug 1792604] Re: Installing yaru should try to remove the communitheme snap

2018-09-14 Thread Olivier Tilloy
This would need to happen on new installs of yaru-theme-gtk only, not updates. The removal should be attempted, but failure to do so for whatever reason (does snapd need network access to remove an installed snap?) should be gracefully ignored. -- You received this bug notification because you