test/conf.do: Only use c99 if CC is undefined v1 REJECTED

Haelwenn (lanodan) Monnier: 1
 test/conf.do: Only use c99 if CC is undefined

 1 files changed, 1 insertions(+), 1 deletions(-)

[PATCH] test/conf.do: Only use c99 if CC is undefined Export this patch

Gentoo has host-architecture symlinks like c99 disabled for some Q&A tests
to make sure cross-compiling works.
---
 test/conf.do | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/test/conf.do b/test/conf.do
index a4b28a9..3956529 100644
--- a/test/conf.do
+++ b/test/conf.do
@@ -11,4 +11,4 @@ SRCALL=$(echo $OBJ | sed 's/\.o/\.c/g')
 
 CFLAGS="$CPPFLAGS -Wall -Wextra -std=c99 -g -Wpedantic"
 
-CC=c99
+CC=${CC:-c99}
-- 
2.39.1

Autumn! <autumnull@posteo.net>

1 year, 8 months ago

Thanks for flagging this!

On my machine, this patch causes a lot of compile errors when running
`haredo test`.

Could you tell me if changing the last few lines of test/conf.do to this:

CFLAGS="$CPPFLAGS -Wall -Wextra -g -Wpedantic"
CC="$CC -std=c99"

fixes your issue, and if so, send another patch with that change?

~Autumn