Skip to content
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

Include a method for packagers to disable update check #1103

Closed
limburgher opened this issue Jul 22, 2021 · 9 comments · Fixed by #1132
Closed

Include a method for packagers to disable update check #1103

limburgher opened this issue Jul 22, 2021 · 9 comments · Fixed by #1132

Comments

@limburgher
Copy link

See: https://bugzilla.redhat.com/show_bug.cgi?id=1984726

@tpikonen
Copy link
Contributor

There's the config variable software_update.check_on_startup, which could be set to False by default in src/gpodder/config.py with a one line patch. Would this be enough?

@limburgher
Copy link
Author

That would be good, but they're looking for the menu option to be disabled/removed as well.

@auouymous
Copy link
Member

There is no UI that can easily be disabled, only an "advanced" UI that shows all config options. The best solution would be to have gpodder check if a file exists, such as /etc/gpodder-no-update-check, and not perform the check even if the setting is enabled. The distro can then touch that file during install without needing to patch each release. Are there any conventions for such files?

@limburgher
Copy link
Author

Not that I'm aware of. gpodder as currently packaged has nothing in /etc since everything is configured per-user. What about a build-time options to set that flag? That seems the most portable.

@tpikonen
Copy link
Contributor

There is the 'Software updates' menu item in the app menu. But, I don't see the original bug report complaining about that, rather than about the automatic update, which happens every 5 days by default, and pops up a dialog if (and only if) a new version is found.

@limburgher
Copy link
Author

/me needs to schedule an appointment with her optometrist.

I'll just patch software_update.check_on_startup. Thanks!

@auouymous
Copy link
Member

Patching the default setting to False only works for new users, existing users will still have it set to True in their Settings.json file. The patch must remove the six lines in gtkui/main.py at self.config.software_update.check_on_startup.

@limburgher
Copy link
Author

I'll patch that as well for now, thanks!

@oturpe
Copy link

oturpe commented Aug 17, 2021

Original reporter here.
Thank you everybody for working on this!
Fedora patch looks good (though it is diffucult to test properly)
and #1132 looks even better.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants