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

add cmake option: upgrade chk and stats reporting. #1413

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

tsteven4
Copy link
Collaborator

Many linux distributions prefer not to have the upgrade check and statistics reporting.

Add a cmake options to disable them. This removes gui/upgrade.* from the build and the related items from the menus. By default the upgrade check and statistics reporting are ON as they have traditionally been.

cmake -DGPSBABEL_UPGRADE_CHECK=OFF ...

@tsteven4
Copy link
Collaborator Author

One point of view is that users of apps distributed by packagers should check for the existence of an upgrade by the methods used by that system, e.g. apt-get, dnf, ... These report if an updated package is available independent of an upstream release.

@tsteven4
Copy link
Collaborator Author

@jspricke can you comment on the desirability of this? I believe it could replace some of your patches. I don't see a need for you to incorporate it until something else drives a change.

@jspricke
Copy link
Contributor

This would be great, thanks for proposing it.

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 this pull request may close these issues.

2 participants