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

Commit 8b3ffde causes hdf5 build issues with some option combinations #4448

Closed
uleysky opened this issue Apr 29, 2024 · 4 comments
Closed

Commit 8b3ffde causes hdf5 build issues with some option combinations #4448

uleysky opened this issue Apr 29, 2024 · 4 comments
Assignees
@derobins
Labels
Component - Build CMake, Autotools Component - Fortran Fortran wrappers Priority - 2. Medium ⏹ It would be nice to have this in the next release Type - Bug Please report security issues to [email protected] instead of creating an issue on GitHub

Comments

@uleysky
Copy link

uleysky commented Apr 29, 2024

Hdf-1.14.4.2 does'nt build after ./configure --enable-fortran --enable-cxx --enable-parallel --enable-unsupported. See Gentoo bug https://bugs.gentoo.org/930143. Reverting commit 8b3ffde fixes this.
@jreadey jreadey added the Component - C Library Core C library issues (usually in the src directory) label Apr 30, 2024
@derobins
Copy link
Member

We can put this hack back in, but why is your build system emitting stray -l entries?

@derobins derobins added Priority - 2. Medium ⏹ It would be nice to have this in the next release Component - Fortran Fortran wrappers Component - Build CMake, Autotools Type - Bug Please report security issues to [email protected] instead of creating an issue on GitHub and removed Component - C Library Core C library issues (usually in the src directory) labels Apr 30, 2024
@uleysky
Copy link
Author

uleysky commented Apr 30, 2024

I can’t answer because I haven’t found where and how the postdeps variable is generated. I can only say that it has something to do with the combination of all four options; if you remove any of them, everything works.

@derobins derobins mentioned this issue May 20, 2024
Merged
@derobins
Copy link
Member

Should be fixed w/ #4501, which restores the sed line

@derobins
Copy link
Member

This will appear in the 1.14.4-3 patch release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@derobins derobins

Labels
Component - Build CMake, Autotools Component - Fortran Fortran wrappers Priority - 2. Medium ⏹ It would be nice to have this in the next release Type - Bug Please report security issues to [email protected] instead of creating an issue on GitHub
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@derobins @uleysky @jreadey