Fix compilation on embedded platforms

[AndreasFuchsTPM]

Fixes: #2692

[wxleong]

Could you please update the configure.ac and Makefile.am so that it's possible to exclude libutilio from the build? This is essential because embedded platforms are unable to build libutilio.

I have tested the following on your fork (AndreasFuchsTPM:fix_formatstrings) and it builds ok:

diff --git a/Makefile.am b/Makefile.am
index 03c5340d..12de494f 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -249,9 +249,11 @@ libutil = libutil.la
 noinst_LTLIBRARIES += $(libutil)
 libutil_la_SOURCES = $(UTIL_SRC)
 
+if ENABLE_UTIL_IO
 libutilio = libutil-io.la
 noinst_LTLIBRARIES += $(libutilio)
 libutil_io_la_SOURCES = $(UTIL_IO_SRC)
+endif
 
 ### TCG TSS Marshaling/Unmarshaling spec library ###
 libtss2_mu = src/tss2-mu/libtss2-mu.la
diff --git a/configure.ac b/configure.ac
index f198bb96..717dcef8 100644
--- a/configure.ac
+++ b/configure.ac
@@ -226,6 +226,13 @@ AC_CHECK_LIB([curl], [curl_url_strerror], [AC_DEFINE([HAVE_CURL_URL_STRERROR],[1
 AS_IF([test "x$enable_fapi" = xyes -o "x$enable_policy" = xyes ],
       [PKG_CHECK_MODULES([UUID], [uuid])])
 
+AC_ARG_ENABLE([util-io],
+              [AS_HELP_STRING([--disable-util-io],
+              [don't build the util-io])],,
+              [enable_util_io=yes])
+
+AM_CONDITIONAL(ENABLE_UTIL_IO, test "x$enable_util_io" = "xyes")
+
 AC_ARG_WITH([tctidefaultmodule],
             [AS_HELP_STRING([--with-tctidefaultmodule],
 [The default TCTI module for ESYS. (Default: libtss2-tcti-default.so)])],

[codecov]

Codecov Report

Merging #2676 (80bfebf) into master (86949f7) will decrease coverage by 0.01%.
Report is 4 commits behind head on master.
The diff coverage is 92.30%.

❗ Current head 80bfebf differs from pull request most recent head 8276939. Consider uploading reports for the commit 8276939 to get more accurate results

@@            Coverage Diff             @@
##           master    #2676      +/-   ##
==========================================
- Coverage   82.54%   82.54%   -0.01%     
==========================================
  Files         368      368              
  Lines       42941    42941              
==========================================
- Hits        35447    35445       -2     
- Misses       7494     7496       +2     

Files	Coverage Δ
src/tss2-esys/api/Esys_StartAuthSession.c	95.28% <ø> (ø)
src/tss2-esys/esys_iutil.c	87.01% <100.00%> (ø)
src/tss2-esys/esys_tr.c	89.25% <ø> (ø)
src/tss2-mu/tpmu-types.c	74.03% <ø> (ø)
src/tss2-policy/tss2_policy.c	78.91% <ø> (ø)
src/tss2-sys/api/Tss2_Sys_Execute.c	77.14% <ø> (ø)
src/tss2-tcti/tcti-device.c	71.57% <ø> (ø)
src/tss2-tcti/tcti-i2c-ftdi.c	76.66% <ø> (ø)
src/tss2-tcti/tcti-i2c-helper.c	83.28% <100.00%> (ø)
src/tss2-tcti/tcti-pcap-builder.c	88.74% <ø> (ø)
... and 9 more

... and 3 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[AndreasFuchsTPM]

Could you please update the configure.ac and Makefile.am so that it's possible to exclude libutilio from the build? This is essential because embedded platforms are unable to build libutilio.

Updated

[wxleong]

The Cirrus CI is failing due to the new usage limitations imposed on its free tier, read here