Pick up LDFLAGS and CPPFLAGS from environment

[henrycg]

No description provided.

[rhelmer]

Hm, adding LDFLAGS to CFLAGS doesn't seem to do the trick:

$ scons
scons: Reading SConscript files ...
DEBUG MODE!
scons: done reading SConscript files.
scons: Building targets ...
gcc -o build/browser-test/main.o -c -I/usr/local/opt/nss/include -I/usr/local/opt/nspr/include/nspr -L/usr/local/opt/nss/lib -Wall -Werror -Wextra -O3 -std=c99 -I/usr/include/nspr -DDO_PR_CLEANUP -g -DDEBUG -Iinclude -I. build/browser-test/main.c
clang: error: argument unused during compilation: '-L/usr/local/opt/nss/lib' [-Werror,-Wunused-command-line-argument]
scons: *** [build/browser-test/main.o] Error 1
scons: building terminated because of errors

Looks like you want:

if "LDFLAGS" in os.environ:
  env.Append(LINKFLAGS = SCons.Util.CLVar(os.getenv("LDFLAGS")))

That WFM

[henrycg]

Whoops. That was a copy-paste typo. I just pushed an updated version with this fix.

[rhelmer]

Nice, thanks!