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 LSB services bnc#846422 #33
Conversation
- use list-units as the base for displaying services - add services from list-unit-files if they are missing - use Service.Status for getting information about the missing services
@@ -71,11 +71,14 @@ def save | |||
success = ServicesManager.save | |||
UI.CloseDialog | |||
if !success | |||
# FIXME if user select to continue the content of the popup is not discarded | |||
# and new error messages will be displayed beneath the old ones |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any advise how to achieve this would be helpful.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't see any reason why it should have this behaviour. Quick look at source code doesn't show anything obvious. We should debug it how it can happen.
- simplify double reject
Include LSB services bnc#846422
success = ! Popup::ContinueCancel( | ||
_("Writing the configuration have failed.\n" + | ||
_("Writing of the configuration have failed.\n" + |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Writing of the configuration
means something like a configuration poem
, writing
is a noun.
On the other hand writing the configuration
uses writing
as doing something
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the notice, it sounded somehow weird to me for some reason.. I will revert it with the next PR.
LSB services were ignored by services manager as they are not 'pure' systemd units. More details in bnc#846422