Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
ExternalProject SUNDIALS 3.2.1 #1960
base: master
Are you sure you want to change the base?
ExternalProject SUNDIALS 3.2.1 #1960
Changes from all commits
4581ab4
0a3692c
d38f489
f36e64a
daf0008
857bd86
152e86f
e613da1
f2c6b4d
5c5ae92
3341256
9a38810
e0b3ef1
5a11a3f
aeb1684
00b2c66
174036f
554b326
56a1134
7e6761c
7176f4a
824732b
a2044cb
5e0162b
c60a3eb
fe5c851
d0bf3e7
65b13f5
12ddfb5
84f05af
ffdb5fb
34062f3
dbfc76f
06f2bc3
ff9bb0f
1a84a72
910d9ac
7e9ddf1
5ac7685
009a914
53683bf
9d6b033
7dbf02f
6f92bd5
6a4e401
0eda725
f9749ca
cfe9ef1
6279e08
858b829
67b677c
3777c05
8f17f69
93be65e
30ee751
8c46a46
9414fb6
7cf79d6
be164eb
b40997a
30c9890
d7c2d10
fe1260f
cf0ccb1
f26fc9d
8368752
ca98ce5
eb5e85a
0d4cc21
21c7826
95b5cca
52cd8b8
97b1089
3277919
02c9d51
8b4d2c2
a66cc8f
6910d0a
932bdff
a8e8b85
50b1163
73c8c79
71a629c
4d4889e
befbb64
083a1c8
4c5905b
ce227b4
e6017d3
4267638
befec84
8099fd1
7c670b7
e0a6bc2
8ab1989
5bdaf5e
9870f3f
f0d233a
710c2c2
45e0fc9
9db5f29
5de9b21
4464390
5614e5d
285dada
f17e9a8
ac12c82
34b82a7
4e12ea1
b398627
68bc030
1c2308c
82c2f25
1204452
11afb3b
c084919
f96ebe3
a5f2b48
7286d43
a89f3e4
0467293
d38be72
091c5c0
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was previously discussed with Alex, but still remember less-than-ideal experiences with
ExternalProject_Add
during 2016-17. I am still wondering if there might be better alternatives to using nested cmake configure steps. :)I'll leave it to @alkino and @1uc to weigh in.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My experience with
ExternalProject_Add
for Sundials hasn't been too bad. At least any change to a build at the nrn level propagates to re-compile and build the sundials *.a files (that link into libnrniv.so). It could be better in terms of having a build tree as a subtree in the nrn build tree. (I often use mulitple nrn/build.../ with different cmake option choices and it would be nice if the latest nrn build did not cause the previous sundial build to be overwritten. I would also prefer to avoidgit status
not showing the untracked folderexternal/sundials
At the moment the sundials repository seems to begin at nrn/external/sundials/src/sundials-external, the build takes place in nrn/external/sundials/src/sundials-external-build, and the install takes place in nrn/external/sundials/(include, lib). I suppose the way we have been handling InterViews and, more recently, fmt, might provide an alternative strategy.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm all in for ExternalProject_Add 🚀 given that we can do custom build/configure commands, it will be definitely useful for upgrading to newer sundials.