SPECFEM / specfem3d_globe

SPECFEM3D_GLOBE simulates global and regional (continental-scale) seismic wave propagation.
GNU General Public License v3.0
90 stars 95 forks source link

specfem3d_globe compile error with --with-openmp #198

Closed luet closed 10 years ago

luet commented 10 years ago

See build 81. Thanks, David

komatits commented 10 years ago

Hi David,

Great! BuildBot caught another one. I'll fix it in a few minutes.

Great tool indeed.

Thanks, Dimitri.

On 01/08/2014 18:32, David Luet wrote:

See build 81 http://specfem3d-globe-buildbot.princeton.edu/builders/specfem3d_globe_builder/builds/81/steps/--with-openmp_make/logs/stdio. Thanks, David

— Reply to this email directly or view it on GitHub https://github.com/geodynamics/specfem3d_globe/issues/198.

Dimitri Komatitsch CNRS Research Director (DR CNRS), Laboratory of Mechanics and Acoustics, UPR 7051, Marseille, France http://komatitsch.free.fr

komatits commented 10 years ago

It seems --with-openmp is missing from that configure:

komatits-laptop: configure --with-openmp
configure: WARNING: unrecognized options: --with-openmp
Optional Packages:
  --with-PACKAGE[=ARG]    use PACKAGE [ARG=yes]
  --without-PACKAGE       do not use PACKAGE (same as --with-PACKAGE=no)
  --with-cuda             build cuda GPU enabled version [default=no]
  --with-opencl           build OpenCL GPU enabled version [default=no]
  --with-adios            build ADIOS enabled version [default=no]
  --with-vtk              The prefix where VTK is installed [default=/usr]
  --with-vtk-suffix       Suffix to append to VTK's include directory, e.g.,
                          for vtk-5.2/ the suffix is "-5.2"
  --with-vtk-version      Version number of VTK library [default=5.2]

This seems to be related to the fact that there are no #ifdef OPENMP statements in the source code. However, as mentioned yesterday for the SPECFEM3D package it would be better to have a --with-openmp option in configure, and that option would / should add the right FCFLAGS for OpenMP automatically.

komatits commented 10 years ago

Fixed!