ornladios / ADIOS

The old ADIOS 1.x code repository. Look for ADIOS2 for new repo
https://csmd.ornl.gov/adios
Other
54 stars 41 forks source link

--without-mpi not working #74

Closed ax3l closed 8 years ago

ax3l commented 8 years ago

To build simple serial tools, such as the no-mpi versions of the numpy wrappers, MPI is not required. Unfurtunately, configure --without-mpi is not working.

ax3l commented 8 years ago

@pnorbert needed for simple packaging, distributing and CI of @yyalli 's numpy wrapper.

ax3l commented 8 years ago

In case you use docker, here is how to reproduce the problem with a minimal Ubuntu without MPI:

docker run -it ubuntu
apt-get update
apt-get install -y wget unzip libmxml-dev make autoconf automake libtool python

wget https://github.com/ornladios/ADIOS/archive/master.zip 
unzip master.zip
cd ADIOS-master

./autogen.sh
CFLAGS="-fPIC" ./configure --with-mxml=/usr --with-mxml=/usr --without-mpi
# configure: error: could not find mpi library for C
#make

of course, installing MPI will fix this such as

apt-get install libmpich-dev
CFLAGS="-fPIC" LDFLAGS="-pthread" ./configure --with-mxml=/usr --with-mxml=/usr --with-mpi=/usr/lib/mpich/
make

but if all that one wants is the libadios_nompi.a files (and includes) this makes dependencies in continuous integration, package maintaining and development on non-mpi projects depending on ADIOS libraries unnecessarily cumbersome.

pnorbert commented 8 years ago

We never supported sequential-only build. I agree it could be practical to implement --without-mpi On Jul 13, 2016 5:12 AM, "Axel Huebl" notifications@github.com wrote:

In case you use docker, here is how to reproduce the problem with a default Ubuntu without MPI:

docker run -it ubuntu apt-get update apt-get install -y libmxml-dev wget unzip autoconf automake libtool

wget https://github.com/ornladios/ADIOS/archive/master.zip unzip master.zipcd ADIOS-master

./autogen.sh CFLAGS="-fPIC" ./configure --with-mxml=/usr --with-mxml=/usr --without-mpi# configure: error: could not find mpi library for C

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/ornladios/ADIOS/issues/74#issuecomment-232300875, or mute the thread https://github.com/notifications/unsubscribe/ADGMLd9cgvV_itAQMmPaBBFpXKxLRMsjks5qVKvvgaJpZM4JHSaV .

pnorbert commented 8 years ago

ADIOS now builds --without-mpi since commit 6c393e2665147

ax3l commented 8 years ago

@pnorbert thank you so much, that feature is wonderful! :sparkles: I tested it and it works.