-
Notifications
You must be signed in to change notification settings - Fork 153
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
Track upstream tpm2-tools versions and handling #187
Comments
Issue-Label Bot is automatically applying the label Links: app homepage, dashboard and code for this bot. |
Would it be better to target the tpm2-tools I expect |
Hi @jetwhiz , would 4.0 be the same as 4.x? These are the current versions:
I am happy to go with whatever you feel is the better approach and gives us good coverage. Please go ahead and make recommendations as you have a better handle on this then me. 4.0 is the version now in Fedora rawhide / 32 which is where Keylime packaging will initially land. I have some code now to really allow us to home in on specific releases or using |
It looks like they don't even have a |
🎉 This issue has been resolved in version 4.0.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
So we are now often hitting issues with non backward compat changes upstream.
At the moment we check for certain commands being available, but we are going to need to be more specific, for example
keylime/keylime/tpm2.py
Lines 428 to 432 in 1cf9add
The above will break on 4.0 now, as
-o
is no longer present as an arg:I recommend we instead grab the version string, as follows:
We can then build a matrix of command syntax to use, and hopefull upstream will stop renaming args and depreciating commands so frequently.
We also need to have a table showing tools support (like we did for the OS versions for the installer).
The text was updated successfully, but these errors were encountered: