-
Notifications
You must be signed in to change notification settings - Fork 22
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
Really stick to new nomenclature, plus … #337
Conversation
Open questions:
|
Calling Therefore I conclude that the first line in the usage output is plain wrong. |
Thanks, this is what I failed to find quickly yesterday night. So "Starts patchmanager-daemon" would be a simpler and better explanation of the option
Oh, obviously
So I will change it to "Print this help text.", O.K.? I also think that there should be an option Three more things I am considering:
|
Oh, line 943 should be Actually then it would make sense to replace lines 941-947 with
O.K.? Side note: For me this cries for a |
This is an off-topic minimalistic fix sneaked into MR #337.
src/qml/SettingsPage.qml, line 94
On the topic of It causes a rather radical noninteractive change, (it determines the rpms owning any files changed by patches, downloads them and force-installs them). I guess it's supposed to fix issues caused by modifying files on disk. Ref: |
"There is no
The |
Heh except for |
This is what confused me into guessing that there must be a separate daemon when reading the help text.
… as the lines 941-947 and my code-snippet show. Does your approval also cover this code-snippet? I.e., does it look O.K.? |
That did ring a bell, but I had no clue which, so I looked it up: O.K., that was in 1984! |
Yep, LGTM. |
src/bin/patchmanager-daemon/main.cpp
, lines 51-61.src/bin/patchmanager-daemon/patchmanagerobject.cpp
, lines 941-953.3
to1
minutes for CI runs on MRs to themaster
branch.