-
-
Notifications
You must be signed in to change notification settings - Fork 10.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
lulu.rb: backup and restore settings #56218
Conversation
Has this been reported upstream? |
Good point. Have you done it, @KizzyCode? |
Sorry; I didn't have the time to setup a test setup until now. I'll check it in the next days. |
But have you reported the initial issue upstream, to see if they’re willing to offer an official solution as suggested? |
If you mean working auto-update functionality in LuLu , then yes; there’s already a pull request for it (not by me) which should be merged in the next version. |
That’s not what I meant, I meant an option to uninstall without deleting settings. |
@vitorgalvao Ah, ok – sorry that i misunderstood you there. I've now reported the issue upstream. Your patch sadly does not work as intended; if I run
|
@KizzyCode There was a typo (missing |
Ok, now the upgrade fails with
As far as I can see there are three errors:
And last of all – is it a good idea implementing all this logic in the formula instead of waiting for an upstream fix/auto-updates? |
Mitigate #53078.