Skip to content

parallel-netcdf: add v1.13.0, mark conflict on macOS#46811

Closed
adamjstewart wants to merge 2 commits intospack:developfrom
adamjstewart:packages/parallel-netcdf
Closed

parallel-netcdf: add v1.13.0, mark conflict on macOS#46811
adamjstewart wants to merge 2 commits intospack:developfrom
adamjstewart:packages/parallel-netcdf

Conversation

@adamjstewart
Copy link
Copy Markdown
Member

Can't get this to build on macOS with Xcode 16, and possibly with Xcode 15. Opened an issue upstream. Let's add a conflict for now.

@skosukhin
Copy link
Copy Markdown
Member

#45919

@adamjstewart
Copy link
Copy Markdown
Member Author

@Chrismarsh

May be superseded by #46264, but we should merge one of these 2 PRs soon.

@Chrismarsh
Copy link
Copy Markdown
Contributor

In my testing, @tgamblin's integration of my 'cc' PR here #46598 lets me build parallel-netcdf without issue. I think getting #46598 merged is the priority as my PR needs to be rebased on all the new cc changes anyway.

Copy link
Copy Markdown
Contributor

@tldahlgren tldahlgren left a comment

Choose a reason for hiding this comment

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

Confirmed the version sha256.

@tldahlgren tldahlgren self-assigned this Oct 7, 2024
@tldahlgren tldahlgren changed the title parallel-netcdf: mark conflict on macOS parallel-netcdf: add v1.13.0, mark conflict on macOS Oct 7, 2024
@Chrismarsh
Copy link
Copy Markdown
Contributor

I'll admit I don't love this being conflicted as it immediately breaks much of the macOS workflow. Is 46598 not close to being merged?

@tldahlgren
Copy link
Copy Markdown
Contributor

I'll admit I don't love this being conflicted as it immediately breaks much of the macOS workflow. Is 46598 not close to being merged?

@tgamblin Thoughts regarding @46598 status?

@skosukhin
Copy link
Copy Markdown
Member

To be clear, the conflict must be removed. I have no problem building the package on macOS because there are no extra rpaths in my compiler configuration file.

@adamjstewart
Copy link
Copy Markdown
Member Author

Closing in favor of the other PR.

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.

4 participants