add upstream patches for GNOME 3.18
Updates may require up to 24 hours to propagate to mirrors. If the following command doesn't work, please retry later:
sudo dnf upgrade --refresh --advisory=FEDORA-2016-7d556fdafa
Please login to add feedback.
This update has been submitted for testing by raphgro.
This update resolve bug #1307253 for me, but there are new problems with this new version.
python doesn't crash anymore, but when running hamster from console, I get:
I'd like to retract my previous comment: Without changing anything, hamster now works. Maybe this was an artifact of a service of the previous version that was still running.
Thanks for testing. Please let me know when you can reproduce the above exception with ZeroDivisionError and what you've done. The PyGIWarning is quite normal (I get it also) and just indicate bad coding style of upstream. Your comment about the need to restart services is somehow useful to know, should we recommend users to logout and login?
I'm not sure if hamster-service and hamster-windows-service are restarted on upgrade. If they aren't, it would probably be helpful to restart them.
Both background services are not restarted. I'm not sure if upstream provides a clean way to restart them.
Btw: The warning started to appear only since Gnome 3.18 so it's more a sign of "recently deprecated" coding patterns rather than "bad coding style".
hamster-service stops itself after it is changed (but I think not when any other file that is used by the service is updated, so I'm not sure it is sufficient): https://github.com/projecthamster/hamster/blob/master/src/hamster-service#L87
However, it has a Quit method that is callable through DBUS: https://github.com/projecthamster/hamster/blob/master/src/hamster-service#L119
hamster-windows-service doesn't stop after a change, but also has a Quit method: https://github.com/projecthamster/hamster/blob/master/src/hamster-windows-service#L28
These could be called after an update to shut down the old version. The next use of the services should respawn them (at least this is what the source says; I have no experience with DBUS).
This update has been pushed to testing.
This update has been submitted for stable by bodhi.
Works well here.
raphgro edited this update.
Thank you for the fix! Functionnal for me.
I can't tell about relaunching the deamon, I tried installing it from sources, and I killed both before reinstalling from testing.
karma: +1
This update has been pushed to stable.