hkmoffat / cantera

Automatically exported from code.google.com/p/cantera
0 stars 0 forks source link

Scons installation routine leads to missing header files #138

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
Follow scons installation routine with options prefix=$HOME/cantera2, 
python_package=minimal,matlab_toolbox=n,f90_interface=n,use_snudials=n

What is the expected output? What do you see instead?
CVodeInt.h is missing in $HOME/cantera2/include/cantera/numerics. But it has 
been build.Moving CVodeInt.h into the path solves the problem. I think, if one 
uses use_sundials=y the file CVodesIntegrator.h is missing (?).

Using cantera 2.0.1, scons-2.2, gcc 4.7.2 on linux 3.6.11-1-ARCH.

Original issue reported on code.google.com by hbon...@googlemail.com on 9 Jan 2013 at 12:01

GoogleCodeExporter commented 9 years ago
These header files aren't part of the Cantera "include" directory because 
they're not intended to be part of its public interface. The classes CVodeInt 
and CVodesIntegrator are wrappers that only provide the functionality needed in 
other parts of Cantera.

If you want to use CVODES, you should just use Sundials directly. Also, I would 
strongly recommend against using the old copy of CVODE that's included with 
Cantera as it's much less robust than the more recent releases included with 
Sundials.

Proposed resolution: WontFix

Original comment by yarmond on 9 Jan 2013 at 4:11

GoogleCodeExporter commented 9 years ago
You are right, my fault!

Original comment by hbon...@googlemail.com on 29 Jan 2013 at 1:34

GoogleCodeExporter commented 9 years ago

Original comment by yarmond on 31 Jan 2013 at 8:00