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.

Sign up for GitHub

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

Jump to bottom

Linking libtss2-{mu, sys, tcti-mssim and tcti-device}.so without -lgcrypt on --with-crypto=gcrypt #1365

Closed
dilyanpalauzov opened this issue Mar 29, 2019 · 0 comments · Fixed by #1614
Closed

Linking libtss2-{mu, sys, tcti-mssim and tcti-device}.so without -lgcrypt on --with-crypto=gcrypt #1365

dilyanpalauzov opened this issue Mar 29, 2019 · 0 comments · Fixed by #1614

Comments

@dilyanpalauzov
Copy link

configure.ac contains:

AS_IF([test "x$enable_esapi" != xno -a "x$with_crypto" = xgcrypt],
      [AC_CHECK_LIB([gcrypt], [gcry_mac_open], [], [AC_MSG_ERROR([Missing required library: gcrypt.])])])

The [] mean that on ./configure --with-crypro=gcrypt puts in the LIBS variable -lgcrypt and then everything is linked with libgcrypt.

  • Do not link libtss2-mu.so, libtss2-sys.so, libtss2-tcti-mssim.so and libtss2-tcti-device.so with libgrypt, as they don’t use any symbols from there.

  • Consider cheching for libgrypt using AM_PATH_LIBGCRYPT.
@dilyanpalauzov dilyanpalauzov mentioned this issue May 23, 2019
Merged
@tstruk tstruk mentioned this issue Jan 29, 2020
Merged
tstruk added a commit that referenced this issue Jan 29, 2020
@tstruk
build: don't link everything against libgcrypt
3c98f73 
Use AM_PATH_LIBGCRYPT instead of AC_CHECK_LIB for checking
libgcrypt, with doesn't add -lgcrypt to the globac $LIBS.

Fixes: #1365

Signed-off-by: Tadeusz Struk <tadeusz.struk@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

build: don't link everything against libgcrypt
1 participant
@dilyanpalauzov