trilinos / xSDKTrilinos

BSD 3-Clause "New" or "Revised" License
6 stars 6 forks source link

Failed building of examples on Apple with clang compilers #3

Closed BarrySmith closed 8 years ago

BarrySmith commented 8 years ago

ftp://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/04/18/configure_next_arch-osx-xsdk-opt_ipro.log

--with-debugging=0 --download-hypre=1 --download-metis=1 --download-parmetis=1 --download-superlu_dist=1 --download-trilinos=1 --download-xsdktrilinos=1 --download-mpich=1 --with-clanguage=C++ --with-cxx-dialect=C++11 --download-sowing=1 --with-boost-dir=/usr/local --download-cmake=1 --with-no-output -PETSC_ARCH=arch-osx-xsdk-opt -PETSC_DIR=/Users/petsc/petsc.clone-4 --download-hdf5 --download-netcdf --download-exodusii

In file included from /Users/petsc/petsc.clone-4/arch-osx-xsdk-opt/externalpackages/git.xsdktrilinos/petsc/src/BelosPETScSolMgr.cpp:44: /Users/petsc/petsc.clone-4/arch-osx-xsdk-opt/externalpackages/git.xsdktrilinos/petsc/src/BelosPETScSolMgr.hpp:176:34: error: allocation of incomplete type 'Epetra_Vector' { trilinosX = Teuchos::rcp(new Epetra_Vector(View,helper.Map(),x)); } ^~~~~ /Users/petsc/petsc.clone-4/arch-osx-xsdk-opt/include/Epetra_MultiVector.h:53:7: note: forward declaration of 'Epetra_Vector' class Epetra_Vector; ^ 1 error generated. make[2]: * [petsc/src/CMakeFiles/xsdkpetsc.dir/BelosPETScSolMgr.cpp.o] Error 1 make[2]: * Waiting for unfinished jobs.... make[1]: * [petsc/src/CMakeFiles/xsdkpetsc.dir/all] Error 2 make[1]: * Waiting for unfinished jobs.... make: *\ [all] Error 2

I think there is a bug in your code but that the Gnu compilers work anyways while clang is more picky and thus generates the error. Usually "allocation of incomplete type" means that you have not included the right include file needed by C++ that defines the classes "guts" so the compiler does not have enough information to actually create the object.

bartlettroscoe commented 8 years ago

Just needs a simple #include "Epetra_Vector.h"

BarrySmith commented 8 years ago

That's easy for you to say :-)

amklinv commented 8 years ago

Would you mind trying the latest commit, Barry, and closing this if it fixed your issue?

PS: I did not receive an email for this issue. Would you mind mentioning me @amklinv on future issues?

BarrySmith commented 8 years ago

Thanks.

It will be in our tests.

On Apr 19, 2016, at 2:08 PM, amklinv notifications@github.com wrote:

Would you mind trying the latest commit, Barry, and closing this if it fixed your issue?

PS: I did not receive an email for this issue. Would you mind mentioning me @amklinv on future issues?

I think is a way for you to set it so that you are automatically informed when each new issue is posted. (Maybe "watch") Otherwise anyone else who doesn't know to put a @amklinv will get annoyed when they think you are ignoring the issue.

Barry

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub

amklinv commented 8 years ago

Ah. I didn't realize I wasn't listed as a watcher. I just added myself to the list.

Thanks Barry!


From: Barry Smith notifications@github.com Sent: Tuesday, April 19, 2016 1:12 PM To: trilinos/xSDKTrilinos Cc: Klinvex, Alicia Marie; Mention Subject: [EXTERNAL] Re: [trilinos/xSDKTrilinos] Failed building of examples on Apple with clang compilers (#3)

Thanks.

It will be in our tests.

On Apr 19, 2016, at 2:08 PM, amklinv notifications@github.com wrote:

Would you mind trying the latest commit, Barry, and closing this if it fixed your issue?

PS: I did not receive an email for this issue. Would you mind mentioning me @amklinv on future issues?

I think is a way for you to set it so that you are automatically informed when each new issue is posted. (Maybe "watch") Otherwise anyone else who doesn't know to put a @amklinv will get annoyed when they think you are ignoring the issue.

Barry

You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub

You are receiving this because you were mentioned. Reply to this email directly or view it on GitHubhttps://github.com/trilinos/xSDKTrilinos/issues/3#issuecomment-212073409

bartlettroscoe commented 8 years ago

@BarrySmith, did Alicia's commit 5b106b7 fix this issue?

BarrySmith commented 8 years ago

The fix seems to have resolved the issue.