From 46b7cf180a9c10bf27330de804353c9ded792b7f Mon Sep 17 00:00:00 2001 From: Ray Speth Date: Wed, 30 Jan 2019 14:20:12 -0500 Subject: [PATCH] [SCons] Make dependency on copying Eigen headers explicit SCons seems to miss this dependency sometimes, resulting in confusing failures while trying to compile DenseMatrix.cpp. --- ext/SConscript | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/ext/SConscript b/ext/SConscript index 75dbe9d3e..e4cfd9d87 100644 --- a/ext/SConscript +++ b/ext/SConscript @@ -84,8 +84,9 @@ if env['system_sundials'] == 'n': if not env['system_eigen']: license_files.append(('Eigen', 'eigen/COPYING.MPL2')) - build(copyenv.Command('#include/cantera/ext/Eigen', '#ext/eigen/Eigen', - Copy('$TARGET', '$SOURCE'))) + h = build(copyenv.Command('#include/cantera/ext/Eigen', '#ext/eigen/Eigen', + Copy('$TARGET', '$SOURCE'))) + copyenv.Depends(copyenv['config_h_target'], h) # Google Test: Used internally for Cantera unit tests. if env['googletest'] == 'submodule':