-
Notifications
You must be signed in to change notification settings - Fork 285
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
CA-389241: import-update-key compat with xs8 and xs9 #5483
Conversation
2687420
to
ba237b0
Compare
ba237b0
to
ed46327
Compare
d73816b
to
577a243
Compare
Signed-off-by: Lin Liu <lin.liu@citrix.com>
577a243
to
1ff083a
Compare
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.
No more version detection, this looks good now.
Please run some automated tests on both XS8 and XS9 to check it works on both.
New XenRT:
|
@@ -1,6 +1,9 @@ | |||
#!/bin/sh |
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.
This needs to be /bin/bash
because the script it using bash features and would not work in a Posix shell.
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 merged the PR, will fix it up later.
Could you clarify which feature causing that issue? My testing of sh -x
looks good.
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.
Sorry, I was wrong. I assumed ${#keyhash}
(for string length) requires bash but it is in Posix.
pytype_reporter extracted 50 problem reports from pytype output. You can check the results of the job here |
XS9 3943842
XS8 25618140