Skip to content

Commit

Permalink
Revert "Fix help for FAPI configure option"
Browse files Browse the repository at this point in the history
--enable-fapi was left as a vestigil to not break packagers and
conflicts with #824

We want folks to use `--with-fapi` everywhere, but keep supporting
enable-fapi for backwards compat (for now, packagers can move over
time).

This reverts commit bb95e3a.
  • Loading branch information
williamcroberts committed Sep 5, 2024
1 parent 10750c8 commit fbe5052
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion configure.ac
Original file line number Diff line number Diff line change
Expand Up @@ -116,7 +116,7 @@ AS_IF([test "x$esapi_sf" = "xyes"],

AC_ARG_WITH(
[fapi],
[AS_HELP_STRING([--enable-fapi],
[AS_HELP_STRING([--with-fapi],
[enable or disable the fapi backend. Default is "auto" to autodetect])],
[enable_fapi=$withval],
[enable_fapi=auto])
Expand Down
6 changes: 3 additions & 3 deletions docs/FAPI.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ code was implemented using the Enhanced System API (ESAPI) and for on disk stora
## Build Time

At the time the package is built, it will detect tss2-fapi library and automatically configure it's inclusion into the tpm2-pkcs11 library. One can *explicitly* configure this
with `--enable-fapi=yes|no`.
with `--with-fapi=yes|no`.

## Run Time

Expand All @@ -23,8 +23,8 @@ You can take a few actions if you run into this issue:
1. Ignore them, and optionally disable FAPI error logging:
- `export TSS2_LOG=fapi+NONE`

2. Reconfigure the package with `--enable-fapi=no`:
- `./configure --enable-fapi=no`
2. Reconfigure the package with `--with-fapi=no`:
- `./configure --with-fapi=no`

3. Provision FAPI using `tss2_provision`. See the tpm2-tools project for more information:
- <https://github.com/tpm2-software/tpm2-tools/blob/master/man/tss2_provision.1.md>
Expand Down

0 comments on commit fbe5052

Please sign in to comment.