-
Notifications
You must be signed in to change notification settings - Fork 665
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
CMake: Implement WITH_AUTO_UPDATER #9814
Conversation
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
This needs to be documented in the client docs, there is a paragraph about building the client. @mmattel Are there, apart from this switch, ohters that should be documented there? |
They are already kind of documented in code. Lines 80 to 105 in 3ae537b
|
TOKEN_AUTH_ONLY is no longer supported and will be removed one day |
yes, and with a cmake-guru-level above 8 people will find that ;-) |
Those settings are available in the cmake gui and in most IDE's |
Yes, I know. Sorry if I am stubborn: A good "How to build" documentation should mention the most important ones anyway, because packagers (that you want to help ;-) do not starts UIs for every program they are packaging. WHAT the heck is the problem to add that to documentation? At least a hint? |
I don't really have a problem with it, its just an other part of the documentation that can be outdated quite fast |
Fixes: #9082