-
Notifications
You must be signed in to change notification settings - Fork 909
fix(linuxkm): Fix shell syntax error on RHEL8/9 #9415
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
fix(linuxkm): Fix shell syntax error on RHEL8/9 #9415
Conversation
The change fixes Bash syntax error manifesting itself on RHEL8/9 (Rocky
Linux 8/9, to be precise):
...
make -C linuxkm libwolfssl.ko
make[1]: Entering directory '/wolfssl/linuxkm'
bash: -c: line 1: syntax error near unexpected token `}'
bash: -c: line 1: `function resolved_link_is_equal() { [[ -L "$1" && ("$(readlink -f "$1")" == "$(readlink -f "$2")") ]] }'
make[1]: *** [Makefile:145: libwolfssl.ko] Error 2
...
Best reproduced with `rockylinux/rockylinux:8` and
`rockylinux/rockylinux:9` Docker images (Bash versions 4.4.x and 5.1.x,
respecively) - the `configure` options don't seem to matter, since the
function in question doesn't use any configuration-dependent logic
|
Can one of the admins verify this patch? |
|
Testing evidence on Rocky Linux 9: |
|
Testing evidence on Rocky Linux 8: |
|
On the support ticket you opened I asked you to review the contributor agreement. Once that has been approved, this PR can be evaluated. Thanks, |
Thanks, Eric - I'll have it reviewed with my management, will post an update then. |
|
Hi @iliasotnikov -- this is a good fix, and in fact we had developed a similar solution simultaneously and independently. It's fixed in #9438, which should merge early in the coming week. It seems to be a |
Description
The change fixes Bash syntax error manifesting itself on RHEL8/9 (Rocky Linux 8/9, to be precise):
Best reproduced with
rockylinux/rockylinux:8androckylinux/rockylinux:9Docker images (Bash versions 4.4.x and 5.1.x, respectively) - theconfigureoptions don't seem to matter, since the function in question doesn't use any configuration-dependent logicPlease describe the scope of the fix or feature addition.
Fixes zd# N/A
Testing
Local Docker-based environment (
rockylinux/rockylinux:8androckylinux/rockylinux:9)Checklist