Improve the frustrating gtest SConscript 05/20905/5
authorMats Wichmann <mats@linux.com>
Sun, 9 Apr 2017 15:15:31 +0000 (09:15 -0600)
committerPhil Coval <philippe.coval@osg.samsung.com>
Tue, 20 Jun 2017 09:36:38 +0000 (09:36 +0000)
commit68251ab226b837cd5ca66faaa6464cbdd2f84128
tree3daa0bc2b9371bf15da9dcc7c7775ffc946b1634
parent4b8dac558729a303100c2988861f8ba31024a2bb
Improve the frustrating gtest SConscript

Define gtest version in a single variable, for future-proofing
Fix to "python style" (except long lines and constant names) - indents, etc.
Fix redefinition of builtin name 'var' (windows section)
Quit right away if target does not need gtest, and remove extra checks
  for this elsewhere
Add checks for help and clean targets: should not unpack/config/build in
  these cases, and actually clean if clean is called
Split configure and build targets for non-windows:
  may still need to build even if already configured (due to clean target)

Change-Id: Ic9818fdd221eb2d3d7fead8c9585c41db80b3d5a
Signed-off-by: Mats Wichmann <mats@linux.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/18633
Reviewed-by: Phil Coval <philippe.coval@osg.samsung.com>
Tested-by: jenkins-iotivity <jenkins@iotivity.org>
Reviewed-by: Dan Mihai <Daniel.Mihai@microsoft.com>
Origin: master
Reviewed-on: https://gerrit.iotivity.org/gerrit/20905
Reviewed-by: Peter Hatina <phatina@gmail.com>
Reviewed-by: Ziran Sun <ziran.sun@samsung.com>
extlibs/gtest/SConscript