Skip to content

adding MPI flags for travis#171

Merged
bendudson merged 1 commit intoboutproject:nextfrom
dschwoerer:next
Mar 30, 2016
Merged

adding MPI flags for travis#171
bendudson merged 1 commit intoboutproject:nextfrom
dschwoerer:next

Conversation

@dschwoerer
Copy link
Contributor

No description provided.

@bendudson bendudson merged commit 1cba2ea into boutproject:next Mar 30, 2016
@dschwoerer
Copy link
Contributor Author

Unfortunatly it still does not compile ...

I also tried to compile it with gcc-5.3.1

but got errors:

$ make

----- Compiling BOUT++ -----
CXX = mpicxx
CFLAGS = -std=c++11 -g -O0 -Og -DCHECK=3
-DREVISION=93381d55137e58ce3e7d89c65aa3d9b528d7abee -DNCDF -DHDF5
-DBOUT_HAS_PVODE
CHECKSUM = c593fd55412eae888168a2b9740ac2fb
INCLUDE = -I../include -I/home/dave/local/include -I/usr/include
Compiling field2d.cxx
In file included from ../../include/field2d.hxx:36:0,
from ../../include/bout/mesh.hxx:48,
from ../../include/globals.hxx:30,
from field2d.cxx:29:
../../include/fieldperp.hxx: In member function ‘const BoutReal&
FieldPerp::operator()(int, int) const’:
../../include/fieldperp.hxx:128:8: error: no match for ‘operator!’
(operand type is ‘const Array’)
if(!data)
^
../../include/fieldperp.hxx:128:8: note: candidate: operator!(bool)

../../include/fieldperp.hxx:128:8: note: no known conversion for
argument 1 from ‘const Array’ to ‘bool’
In file included from ../../include/bout/mesh.hxx:48:0,
from ../../include/globals.hxx:30,
from field2d.cxx:29:
../../include/field2d.hxx: In member function ‘BoutReal&
Field2D::operator()(int, int)’:
../../include/field2d.hxx:103:27: error: ‘mesh’ was not declared in this
scope
if((jx < 0) || (jx >= mesh->ngx) || (jy < 0) || (jy >= mesh->ngy) )
^
../../include/field2d.hxx: In member function ‘const BoutReal&
Field2D::operator()(int, int) const’:
../../include/field2d.hxx:114:27: error: ‘mesh’ was not declared in this
scope
if((jx < 0) || (jx >= mesh->ngx) || (jy < 0) || (jy >= mesh->ngy) )
^
../../make.config:177: recipe for target 'field2d.o' failed
make[2]: *** [field2d.o] Error 1
../make.config:86: recipe for target 'field' failed
make[1]: *** [field] Error 2
make.config:86: recipe for target 'src' failed
make: *** [src] Error 2

$ mpicxx --version
g++ (GCC) 5.3.1 20151207 (Red Hat 5.3.1-2)
Copyright (C) 2015 Free Software Foundation, Inc.
This is free software; see the source for copying conditions. There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

I will try gcc 4.8.5 - what compiler are you using?

On 30/03/16 20:56, Ben Dudson wrote:

Merged #171 #171.


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#171 (comment)

@bendudson
Copy link
Contributor

Hey David,

Sorry, those were my fault. I don't think I'd tried it with CHECK > 2, so
some of the old code in those sections hadn't been updated. Should be fixed
now.

Thanks very much for fixing the compiler issue. Travis seems to now fail to
compile PVODE, which is a bit odd.

Best

Ben

On 30 March 2016 at 21:08, dschwoerer notifications@github.com wrote:

Unfortunatly it still does not compile ...

I also tried to compile it with gcc-5.3.1

but got errors:

$ make

----- Compiling BOUT++ -----
CXX = mpicxx
CFLAGS = -std=c++11 -g -O0 -Og -DCHECK=3
-DREVISION=93381d55137e58ce3e7d89c65aa3d9b528d7abee -DNCDF -DHDF5
-DBOUT_HAS_PVODE
CHECKSUM = c593fd55412eae888168a2b9740ac2fb
INCLUDE = -I../include -I/home/dave/local/include -I/usr/include
Compiling field2d.cxx
In file included from ../../include/field2d.hxx:36:0,
from ../../include/bout/mesh.hxx:48,
from ../../include/globals.hxx:30,
from field2d.cxx:29:
../../include/fieldperp.hxx: In member function ‘const BoutReal&
FieldPerp::operator()(int, int) const’:
../../include/fieldperp.hxx:128:8: error: no match for ‘operator!’
(operand type is ‘const Array’)
if(!data)
^
../../include/fieldperp.hxx:128:8: note: candidate: operator!(bool)

../../include/fieldperp.hxx:128:8: note: no known conversion for
argument 1 from ‘const Array’ to ‘bool’
In file included from ../../include/bout/mesh.hxx:48:0,
from ../../include/globals.hxx:30,
from field2d.cxx:29:
../../include/field2d.hxx: In member function ‘BoutReal&
Field2D::operator()(int, int)’:
../../include/field2d.hxx:103:27: error: ‘mesh’ was not declared in this
scope
if((jx < 0) || (jx >= mesh->ngx) || (jy < 0) || (jy >= mesh->ngy) )
^
../../include/field2d.hxx: In member function ‘const BoutReal&
Field2D::operator()(int, int) const’:
../../include/field2d.hxx:114:27: error: ‘mesh’ was not declared in this
scope
if((jx < 0) || (jx >= mesh->ngx) || (jy < 0) || (jy >= mesh->ngy) )
^
../../make.config:177: recipe for target 'field2d.o' failed
make[2]: *** [field2d.o] Error 1
../make.config:86: recipe for target 'field' failed
make[1]: *** [field] Error 2
make.config:86: recipe for target 'src' failed
make: *** [src] Error 2

$ mpicxx --version
g++ (GCC) 5.3.1 20151207 (Red Hat 5.3.1-2)
Copyright (C) 2015 Free Software Foundation, Inc.
This is free software; see the source for copying conditions. There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

I will try gcc 4.8.5 - what compiler are you using?

On 30/03/16 20:56, Ben Dudson wrote:

Merged #171 #171.


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#171 (comment)


You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub
#171 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants