Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

BelosSolverManager.hpp update broke build. #199

Closed
bathmatt opened this issue Mar 9, 2016 · 3 comments
Closed

BelosSolverManager.hpp update broke build. #199

bathmatt opened this issue Mar 9, 2016 · 3 comments

Comments

@bathmatt
Copy link
Contributor

bathmatt commented Mar 9, 2016

@amklinv
Your latest push broke the build

[ 65%] Building CXX object packages/belos/src/CMakeFiles/belos.dir/Belos_Details_registerLinearSolverFactory.cpp.o
[ 68%] Built target ml
In file included from https://jenkins-srn.sandia.gov:8443/job/Drekar-opt/ws/Trilinos/packages/belos/src/BelosSolverFactory.hpp:47:0,
from https://jenkins-srn.sandia.gov:8443/job/Drekar-opt/ws/Trilinos/packages/belos/src/Belos_Details_LinearSolverFactory.hpp:48,
from https://jenkins-srn.sandia.gov:8443/job/Drekar-opt/ws/Trilinos/packages/belos/src/Belos_Details_registerLinearSolverFactory.cpp:43:
https://jenkins-srn.sandia.gov:8443/job/Drekar-opt/ws/Trilinos/packages/belos/src/BelosSolverManager.hpp:143:20: error: expected nested-name-specifier before ‘StatusTestCombo’
const typename StatusTestCombo<ScalarType,MV,OP>::ComboType &comboType =
^
https://jenkins-srn.sandia.gov:8443/job/Drekar-opt/ws/Trilinos/packages/belos/src/BelosSolverManager.hpp:143:35: error: expected ‘,’ or ‘...’ before ‘<’ token
const typename StatusTestCombo<ScalarType,MV,OP>::ComboType &comboType =
^
make[2]: *** [packages/belos/src/CMakeFiles/belos.dir/Belos_Details_registerLinearSolverFactory.cpp.o] Error 1
make[1]: *** [packages/belos/src/CMakeFiles/belos.dir/all] Error 2
make: *** [all] Error 2
@amklinv
Copy link

amklinv commented Mar 9, 2016

Sorry! I'm pushing the fix right now.

@amklinv
Copy link

amklinv commented Mar 9, 2016

There was a missing include file; it should be fixed now. Sorry for the inconvenience. Can you please try it again and close out the issue if it works for you?

@bathmatt
Copy link
Contributor Author

bathmatt commented Mar 9, 2016

fixed

@bathmatt bathmatt closed this as completed Mar 9, 2016
bartlettroscoe added a commit that referenced this issue Jun 27, 2017
@bartlettroscoe
Automatic snapshot commit from tribits at 29607f6
55074a3 
Origin repo remote tracking branch: 'github/master'
Origin repo remote repo URL: 'github = [email protected]:TriBITSPub/TriBITS.git'

At commit:

commit 29607f6b5b95fd9ab298135de86855e75bf74343
Author:  Roscoe A. Bartlett <[email protected]>
Date:    Tue Jun 27 09:18:00 2017 -0600
Summary: Change name of var 'result' to 'eti_result' to avoid clash (#199)
@trilinos-autotester trilinos-autotester mentioned this issue Nov 5, 2021
Closed
@trilinos-autotester trilinos-autotester mentioned this issue Dec 15, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@bathmatt @amklinv