-
Notifications
You must be signed in to change notification settings - Fork 23
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
diff-kernel-config: fix for core-kits #32
diff-kernel-config: fix for core-kits #32
Conversation
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! Looks great, and it sounds like you got the right answer when you tried it.
tools/diff-kernel-config
Outdated
@@ -114,6 +101,7 @@ readonly output_dir | |||
# checkout. | |||
[[ -n ${gitrev_before_arg} ]] || usage_error 'require -b|--before' | |||
[[ -n ${gitrev_after_arg} ]] || usage_error 'require -a|--after' | |||
[[ -n ${kernel_versions[@]} ]] || usage_error 'require -k|--kernel' |
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.
Builds are failing with this error:
diff-kernel-config:104:7: error: Arrays implicitly concatenate in [[ ]]. Use a loop (or explicit * instead of @). [SC2199]
I believe this line is the culprit. Ways to fix this: https://www.shellcheck.net/wiki/SC2199
The script referenced variants, which aren't a thing in the core-kit world Signed-off-by: Arnaldo Garcia Rincon <[email protected]>
1a38550
to
08e37cb
Compare
Forced push includes fixes for shellchecks |
Issue number:
N / A
Description of changes:
The script referenced variants, which aren't a thing in the core-kit world
Testing done:
I ran the script with the changes in: #29 , and the report was generated:
Terms of contribution:
By submitting this pull request, I agree that this contribution is dual-licensed under the terms of both the Apache License, version 2.0, and the MIT license.