CCFLAGS override?

[saper]

We suggest in the instructions to override CCFLAGS like this:

./bin/package make \
    SHELL=/bin/bash CCFLAGS="-xc99 -D_XPG6 -m64 -xO4" LDFLAGS="-m64"

However, there are some places like for example lib/ast/libast/Makefile:

ksh/src/lib/libast/Makefile

Line 13 in e7f2542

CCFLAGS = $(CC.SUFFIX.DEBUG:+$(CC.DEBUG)) $(CC.OPTIMIZE) $(CC.DLL)

where CCFLAGS gets overwritten anyway. If we override that, and CC.DLL is needed to be to set -fPIC and things like that we may end up in problems with linking.

I don't have a clear reproduction case for this issue right now, but I have noticed builds breaking when trying to set CCFLAGS to quell clang warnings.

[jghub]

@saper
thank you for this heads up. duly noted. as you might imagine, there are still things to organise/adjust in ksh-community layout/setup so far. therefore, it might take some time before someone looks into this more closely.
I also presume that there will not be a "one size fits" all recommendation for CCFLAGS settings anyway. e.g AFAICS ubuntu uses

    ./bin/package make CCFLAGS="-g -O2\
       -fdebug-prefix-map=${PWD}=.\
       -fstack-protector-strong\
       -Wformat -Werror=format-security\
       -Wdate-time -D_FORTIFY_SOURCE=2" \
        LDFLAGS="-Wl,-Bsymbolic-functions -Wl,-z,relro"

for building...

new FAQs and HOWTOs (or, generally, additional documentation) is one of the medium-term goals of ksh-community as well. so comprehensive, platform-dependent build instructions might be integrated there too if need be.

[saper]

Sure, that's fine. I just posted it because this is what struck me reading project's README.
I'll try to post some more details here so feel free to treat it as a "note to self" bug if I may...

[jghub]

sure. please do. and if it is FreeBSD specific, flag it as such ;)

[saper]

No, it is not. Can I edit issue labels?

[jghub]

seemingly not ... what do you want? "documentation"?

[jghub]

I am open to suggestions...