-
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Only compare OS versions if dig
was successful
#5050
Conversation
Signed-off-by: Christian König <ckoenig@posteo.de>
Signed-off-by: RD WebDesign <github@rdwebdesign.com.br>
also use INFO instead of CROSS, because it's not an error. Signed-off-by: RD WebDesign <github@rdwebdesign.com.br>
As your working on this part of the code, I think the order of the output should be changed in general
I would print Distro/Version first (only with an [i]) then the dig code/return and a new additional line with the result of the comparison.
|
- add red color to dig errors - reorder some output lines - add "Distro and version supported" on success Signed-off-by: RD WebDesign <github@rdwebdesign.com.br>
Signed-off-by: RD WebDesign <github@rdwebdesign.com.br>
Current output
|
Signed-off-by: RD WebDesign <github@rdwebdesign.com.br>
Bumps [actions/setup-python](https://github.com/actions/setup-python) from 4.3.0 to 4.3.1. - [Release notes](https://github.com/actions/setup-python/releases) - [Commits](actions/setup-python@v4.3.0...v4.3.1) --- updated-dependencies: - dependency-name: actions/setup-python dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: yubiuser <ckoenig@posteo.de> Signed-off-by: Adam Warner <github@adamwarner.co.uk>
For people here in the future - some sort of oddity happened here in that PR was merged (check dev, the commits are there!) but for some reason it got marked as closed. Initially it was saying there were merge conflicts that needed to be adjusted on the command line.. and then I just followed the instructions:
Then github got confused and closed the PR. |
What does this PR aim to accomplish?:
On the current Debug Log code, if the
dig
command fails it is impossible to check if the OS is supported, but the log prints an error message saying the OS is unsupported (even when that's not true), like this:How does this PR accomplish the above?:
Modifying the
os_check()
function to compare versions only ifdig
returns without error.Also, adding a new error message for this case:
Link documentation PRs if any are needed to support this PR:
none
By submitting this pull request, I confirm the following:
git rebase
)