-
Notifications
You must be signed in to change notification settings - Fork 564
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.
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
missing DEPLIBS for stk #451
Conversation
Without this, I'm also getting build errors like
when building tests. @bmpersc, could you take a look? |
I think we are still not allowed to push changes to STK. And it looks like there is little hope that will ever change. Someone on the Sierra side would have to do it. @trilinos/stk, can one of the STK developers please apply this patch to the native STK repo? I think I have access to the native STK repo so I could push these changes if there are no objections? Since these only impact CMakeLists.txt files, this will not affect that native BJAM build of STK in its host application. |
Ah, I had been under the impression that the CMake part of STK is handled on the Trilinos side. In any case, thanks Ross for the reply! |
We are not allowed to make any changes to STK at all on the Trilinos side, even to just the CMakeLists.txt files. Unfortunately, this is unlikely to ever change judging how things are going with a new integration effort with STK. |
That's perfectly fine if there is a way to submit issues/PRs to STK directly. |
I don't think there can be given the way things are going. I think you would need to be an application developer in the Sandia code base where the STK code is embedded in order to do that. You can only do that if your are a Sandian and pay thousands of $$ a year to get access to that code base. |
@trilinos/stk Ping. |
It is not clear to me that any one on the @trilinos/stk team is both paying attention and able to do something about this issue. @trilinos/framework: What can be done to resolve this issue? |
@bmpersc Is there some way to push this through since it only affects CMakelists files, or should we try to file a STK ticket? This is a poster child for why the integration has to be redone in the right way. |
Unfortunately, this hasn't been resolved yet. Is there anything I can do to help? |
Realistically, I think @bmpersc is the only person who can really make this happen. |
@nschloe I am in the middle of fixing other build errors that have cropped up with an updated STK, once I have addressed those I will look into applying your suggested changes as well. |
I have applied this to STK with some minor changes to address warnings in the build. The only substantive change was to remove stating the boost_program_options dependency. That was causing a warning in the build and it should be redundant since the subpackage was correctly optionally depending on the BoostLib TPL. I will be working to get this back into Trilinos next week. |
@bmpersc Have you had the chance to look into this yet? |
@nschloe yes and no. Yes I was working on getting these back into Trilinos when we discovered something in the integration that was completely unacceptable. That forced our hand to instead make the planned move to a new integration system earlier than we'd planned. I'm currently working on that revamp now. |
@bmpersc How is the revamp going? |
@jwillenbring I still see the error. Anything I can do to help getting this merged? |
@nschloe It's been a while since STK got merged into Trilinos. Not sure of the schedule for the next merge. @khpierson may want to know about this. |
@bmpersc Has this been essentially applied at this point through the integration process? Can we close this? |
This commit fixes missing symbols errors revealed by linking with
-Wl,--no-undefined
. Evolved from https://software.sandia.gov/bugzilla/show_bug.cgi?id=5929.@bmpersc @gsjaardema Probably for you to review/merge.