tpm2-software / tpm2-tss / 1663

36%

push

travis-ci

<a href="https://github.com/tpm2-software/tpm2-tss/commit/<a class=hub.com/tpm2-software/tpm2-tss/commit/74037d3c15b9f9d98f9b50ca4f1c1a99d239d751">74037d3c1<a href="https://github.com/tpm2-software/tpm2-tss/commit/74037d3c15b9f9d98f9b50ca4f1c1a99d239d751">&quot;&gt;build: Fix breakage in generated configure script when CFLAGS is set.

This fixes a bug introduced in </a><a class="double-link" href="https://github.com/tpm2-software/tpm2-tss/commit/<a class="double-link" href="https://github.com/tpm2-software/tpm2-tss/commit/3980bad87fe18ad9d32914e9d21dba145eba973f">3980bad87</a>">3980bad87</a><a href="https://github.com/tpm2-software/tpm2-tss/commit/74037d3c15b9f9d98f9b50ca4f1c1a99d239d751">.
That patch references the documentation for AC_PROG_CC but it misread
the recommended workaround. The leading colon (aka `:`) in the
documentation is significant.

The `:` is a shell &#39;builtin&#39; command that is equivalent to invoking the
&#39;true&#39; command. By placing the conditional substitution of the CFLAGS
after this, the result of the substitution will be ignored and the
script won&#39;t fail. Without this the contents of the CFLAGS variable set
in the environment will be interpreted as a command and since they&#39;re
not commands the configure script will abort.

Signed-off-by: Philip Tricca <flihp@twobit.org>

16675 of 19755 relevant lines covered (84.41%)

548.89 hits per line