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.

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

Wheels: 0.13.4.post0 #1002

Merged
merged 9 commits into from
Jun 4, 2021
Merged

Wheels: 0.13.4.post0 #1002

merged 9 commits into from
Jun 4, 2021

Conversation

ax3l
Copy link
Member

@ax3l ax3l commented Jun 4, 2021

ADIOS: 2.6.0 -> 2.7.1 (Windows: we patch in ornladios/ADIOS2#2746)

Add support for improved blosc support: ornladios/ADIOS2#2592

@ax3l ax3l force-pushed the wheels-0.13.4.post0 branch from 319e75d to 8adb4c3 Compare June 4, 2021 17:54
ax3l added 4 commits June 4, 2021 11:17
```
/usr/local/lib/libblosc.a(blosc.c.o): In function `blosc_init.part.9':
blosc.c:(.text+0x43e): undefined reference to `pthread_atfork'
```
Better an older ADIOS than no BLOSC support at all.
@ax3l ax3l force-pushed the wheels-0.13.4.post0 branch from 5070104 to f20956e Compare June 4, 2021 19:11
@ax3l ax3l merged commit d25b977 into openPMD:wheels Jun 4, 2021
@ax3l ax3l deleted the wheels-0.13.4.post0 branch June 4, 2021 21:51
@@ -91,21 +92,26 @@ function build_adios2 {
if [ "$(uname -s)" = "Linux" ]
then
EVPATH_ZPL="ON"
# ADIOS 2.7.1 & Blosc 1.20.1/1.21.0
# /usr/local/lib/libblosc.a(blosc.c.o): In function `blosc_init.part.9':
# blosc.c:(.text+0x43e): undefined reference to `pthread_atfork'
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same problem appears again in my env on summit at import time...

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we can try (here and for blosc below):

export LDFLAGS="-lpthread -lpthread_nonshared"

https://gitlab.kitware.com/cmake/cmake/-/issues/20709#note_760439

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this line in blosc is outdated:
https://github.com/Blosc/c-blosc/blob/7435f28dd08606bd51ab42b49b0e654547becac4/blosc/CMakeLists.txt#L68
Is: CMAKE_THREAD_PREFER_PTHREAD (docs)
Should be: THREADS_PREFER_PTHREAD_FLAG (docs)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

started a draft in #1013

@ax3l ax3l mentioned this pull request Jun 16, 2021
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant