geodynamics / axisem

AxiSEM is a parallel spectral-element method to solve 3D wave propagation in a sphere with axisymmetric or spherically symmetric visco-elastic, acoustic, anisotropic structures.
66 stars 31 forks source link

Should there be. top-level CMakeLists.txt? #63

Open markyoder opened 4 years ago

markyoder commented 4 years ago

Should there be a top level CMakeLIsts.txt, or should MESHER and SOLVER be compiled separately?

Looking back at this fix: https://github.com/geodynamics/axisem/commit/e9f73ba3461e230bd2f29e42ec0b453ef597bce2

seems to indicate that there should be a top-level build. Generally, the build instructions in the documentation are pretty vague. If you can clarify some of the finer points, I can cut a Fork and help with that.

Possibly answering my own question, it looks like, yes, and that this disappearing CMakeLIsts.txt file has been a (probably git promoted problem for a while). It looks like the most recent commit with a top level CMakeLists.txt is: 4e5c7b1a3d5df6fd664e65ad9ab7bef9f2f1f1bd

Is that correct?

... but then I see a later commit where the cmake directory appears to be intentionally removed. I restored both of these from their most recent commits and appear to have compiled the code (still needs a test), but I would certainly appreciate any guidance from the developers. Thanks in advance!

sstaehler commented 3 years ago

Hi @markyoder ,

apologies for the late reply. Correct, cmake was removed deliberately, because we found that it was not beneficial for the relatively easy compilation task we have.

Run $ copy_templates.sh in the main directory to create makefiles and then do $make

Best,

Simon