tdx-attester: make DCAP libs optional #572
Draft
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Still WIP/RFC.
In #434 I had proposed to make use of
tdx-attest-rs
behind a new feature but dropped the idea since the initdata work moved to use it too and that is a valid case event with upstream kernels.However, @fidencio mentioned a use case that building
kbs-client
without having to installlibtdx-attest*
(because of added complexity and the fact that the libs are not yet avail on all OS'es, e.g, 24.04) but still get the basicget_evidence()
working can be useful. So I restored my original code and added here for feedback on how it's best handled.