Skip to content
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

Use libfido2 on builds if present #43224

Merged
merged 2 commits into from
Jun 19, 2024
Merged

Conversation

codingllama
Copy link
Contributor

Link dynamically to libfido2, by default, if it's present in the environment. Affects tsh and tctl builds.

@codingllama codingllama added backport/branch/v14 no-changelog Indicates that a PR does not require a changelog entry backport/branch/v15 backport/branch/v16 labels Jun 18, 2024
@github-actions github-actions bot requested review from kimlisa and rudream June 18, 2024 22:04
@codingllama codingllama requested a review from rosstimothy June 18, 2024 22:04
@codingllama
Copy link
Contributor Author

FYI, I've changed the "OS Compatibility" builds to use static libfido2, as that better reflect our actual prod binaries (commit d0359df).

@codingllama codingllama added this pull request to the merge queue Jun 19, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jun 19, 2024
@codingllama codingllama added this pull request to the merge queue Jun 19, 2024
Merged via the queue into master with commit cbe7d9f Jun 19, 2024
38 checks passed
@codingllama codingllama deleted the codingllama/make-libfido2 branch June 19, 2024 16:47
@public-teleport-github-review-bot

@codingllama See the table below for backport results.

Branch Result
branch/v14 Failed
branch/v15 Failed
branch/v16 Create PR

rosstimothy pushed a commit that referenced this pull request Jun 20, 2024
* Use libfido2 on builds if present

* Use static libfido2 linking on OS compat tests
rosstimothy pushed a commit that referenced this pull request Jun 20, 2024
* Use libfido2 on builds if present

* Use static libfido2 linking on OS compat tests
github-merge-queue bot pushed a commit that referenced this pull request Jun 20, 2024
* Use libfido2 on builds if present

* Use static libfido2 linking on OS compat tests
github-merge-queue bot pushed a commit that referenced this pull request Jun 20, 2024
* Use libfido2 on builds if present

* Use static libfido2 linking on OS compat tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants