-
Notifications
You must be signed in to change notification settings - Fork 63
Fixups #1644
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
Conversation
Signed-off-by: William Douglas <william.douglas@intel.com>
In order to get the content needed for check-update with the --verbose option, root is required as data is put into the statedir. Keep the old behavior when verbose isn't passed. This was change was required because of the new latest file handling for --incremental. Signed-off-by: William Douglas <william.douglas@intel.com>
aeffe42
to
ab242a7
Compare
The current format is very useful information to get from bug reports so add it as default output to swupd update. Signed-off-by: William Douglas <william.douglas@intel.com>
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.
In the updated test content, I found inconsistent escaping of parens in heredocs.
The cases where the parens are escaped are because the test uses a regex vs the other cases where it is using the exact strings so no escape is needed. |
No description provided.