wafsamba/samba_autoconf: when setting undefined result, use empty tuple
authorAlexander Bokovoy <ab@samba.org>
Thu, 6 Sep 2018 07:51:00 +0000 (07:51 +0000)
committerAndrew Bartlett <abartlet@samba.org>
Fri, 7 Sep 2018 09:37:23 +0000 (11:37 +0200)
commit311e1eb67306d6a639ce80b94a16ea2ef19592f0
tree25d00ec5b8d0d58f07560db5af407dc1bee5e103
parent193fdbf181ad7848cabf56db9780571aeb91aa3a
wafsamba/samba_autoconf: when setting undefined result, use empty tuple

A difference between waf 1.x and 2.x is that we gained 0 as an undefined
variable in the cache file. This does not allow to differentiate unset
and set to 0 defines.

Force to use empty tuple () to signify unset defines.

Also, fix handling of extra cflags in case of 'strict=True': if
extra_cflags were not defined, we'd append None to the cflags list and
it confuses conf.check() later. 'None' is added to the command line of a
tool executed by the conf.check() which, depending on a tool, may be
treated as an error and cause wrong test result.

Signed-off-by: Alexander Bokovoy <ab@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
buildtools/wafsamba/samba_autoconf.py
buildtools/wafsamba/samba_waf18.py