[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.
Cantera · Feb 4, 2019 · 46b7cf1 · 46b7cf1
1 parent dc09040
commit 46b7cf1
Showing 1 changed file with 3 additions and 2 deletions.
diff --git 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':