-
Notifications
You must be signed in to change notification settings - Fork 14
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
wn_bin pkg: Rewrite using modern cmake #354
base: master
Are you sure you want to change the base?
wn_bin pkg: Rewrite using modern cmake #354
Conversation
3cf9ac0
to
d397069
Compare
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.
Hey @ChristianTackeGSI ,
first off, thanks a lot for the patch.
In general it looks good.
But I wasn't able to test it. It returns an error while building the wn_bin target:
[100%] Generate WN binary package
CMake Error at /Users/anar/tmp/Killme/DDS/cmake/DDSCollectWNPkgFiles.cmake:10 (file):
file Resolved path is not absolute
make[3]: *** [CMakeFiles/wn_bin] Error 1
make[2]: *** [CMakeFiles/wn_bin.dir/all] Error 2
make[1]: *** [CMakeFiles/wn_bin.dir/rule] Error 2
make: *** [wn_bin] Error 2
Hi @AnarManafov, can you please post your precise |
@ChristianTackeGSI , sure thing. Here you go:
Please, let men now if you need any other file or info. |
fix proposed in FairRootGroup/DDS#354
d397069
to
5a03bce
Compare
090c6dc
to
2355a73
Compare
2355a73
to
7d72f18
Compare
7d72f18
to
84135f2
Compare
89737b2
to
92b3c44
Compare
92b3c44
to
68c81e7
Compare
68c81e7
to
a735c03
Compare
Modern cmake has better tools to collect the prereqs of binaries. Let's use them. The original patches are by Dennis Klein. He deserves most of the credit for developing this.
a735c03
to
3877f5a
Compare
Modern cmake has better tools to collect the prereqs of binaries. Let's use them.
We're forward-porting this patch now for a long time in our spack tree.
This patch is required for proper building on various systems.
Please reconsider this.