Skip to content

Fix MacOS Static build without FFI#4820

Closed
eisenhauer wants to merge 4 commits intoornladios:release_211from
eisenhauer:MacOSStaticFix
Closed

Fix MacOS Static build without FFI#4820
eisenhauer wants to merge 4 commits intoornladios:release_211from
eisenhauer:MacOSStaticFix

Conversation

@eisenhauer
Copy link
Copy Markdown
Member

No description provided.

@eisenhauer eisenhauer requested a review from a team as a code owner January 26, 2026 20:56
@ax3l ax3l mentioned this pull request Jan 26, 2026
5 tasks
ax3l added a commit to ax3l/openPMD-api that referenced this pull request Jan 26, 2026
@ax3l
Copy link
Copy Markdown
Contributor

ax3l commented Jan 26, 2026

@eisenhauer unfortunately still the same issue with this patch.
Doesn´t the error look like we need to expand the CMAKE_PREFIX_PATH / set the dill_DIR to the download location of DILL?

ax3l added a commit to ax3l/openPMD-api that referenced this pull request Jan 26, 2026
ax3l added a commit to ax3l/openPMD-api that referenced this pull request Jan 26, 2026
@eisenhauer
Copy link
Copy Markdown
Member Author

@eisenhauer unfortunately still the same issue with this patch. Doesn´t the error look like we need to expand the CMAKE_PREFIX_PATH / set the dill_DIR to the download location of DILL?

The deal is that this version DILL wont/cant build build on macos without libffi. It can't do dynamic code generation (it's main job) so it falls back to emulation of DCG, but to do that it needs libffi. It doesn't have that, so it errors out and doesn't build. FFS is OK with that, it has it's own substitute if it can't get DCG, but something is going wrong with the logic somehow. But anyhow, there's no appropriate path to set for DILL because it doesn't build here. Looking at this again, it's still not getting past ffs, so the EVPath mods are inoperative. I've added some diagnostic output to the ADIOS PR. Can you retrigger?

@ax3l
Copy link
Copy Markdown
Contributor

ax3l commented Jan 26, 2026

Thank you, refreshed the patch: openPMD/openPMD-api#1833

@eisenhauer
Copy link
Copy Markdown
Member Author

I'm not seeing the output that I should see from the diagnostics. After it says: " -- ADIOS2 ThirdParty: Configuring ffs", the next line should be either: "ADIOS2 FFS: DID NOT FIND dill::dill target, setting FFS_USE_DILL to OFF" or "ADIOS2 FFS: found dill::dill target, setting FFS_USE_DILL to ON". Neither appears. Possible that something is going wrong with the patching? Or CI caching?

@ax3l
Copy link
Copy Markdown
Contributor

ax3l commented Jan 27, 2026

Def had an issue with applying the patch, but now it works 🎉

Your patch overcomes the issue! Sorry for fumbling the patch initially.

@eisenhauer
Copy link
Copy Markdown
Member Author

Excellent!

@ax3l
Copy link
Copy Markdown
Contributor

ax3l commented Jan 27, 2026

Sadly, it looks like it needs the EVPath changes, too to find DILL.

@eisenhauer
Copy link
Copy Markdown
Member Author

eisenhauer commented Jan 27, 2026 via email

ax3l added a commit to openPMD/openPMD-api that referenced this pull request Jan 27, 2026
* ADIOS2: 2.11.0 & Campaigns

Update ADIOS2 to v2.11.0 and enable campaign management.

* openPMD-api: 0.17.0

* macOS 13 -> 14/15

GH runners EOL and removed

* Build SQLite3

- [x] Linux/macOS
- [x] Windows

* Windows h_errno patch

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* GTKorvo patch

* Revert "[pre-commit.ci] auto fixes from pre-commit.com hooks"

This reverts commit 3ef0b01.

* Exclude patches from pre-commit

* Wrong path..

* Use a better patch for GTKorvo upstream

* Fix Windows paths

* Debug error

* tmp: Use master branch of ADIOS2

* more paths

* Try Greg's suggestion

* Unix: `-DFFS_USE_DILL=FALSE`

* Ouch

* Ouch

* Try fixing paths again..

* use downloaded patch..

* [Patch] ADIOS macOS DILL

Trying ornladios/ADIOS2#4820

* Refresh ADIOS Patch w/ Diagnostics

* Patch?

* Patch?

* Patchy patch

* MACOSX_DEPLOYMENT_TARGET: 14.0/15.0

libcrypto.3.dylib and libssl.3.dylib pull this up

* "git am" committer

* Patch Cleanup/Simplify

* macOS: keep master commit w/ EVPath Change

* remove stray change

---------

Co-authored-by: Franz Pöschel <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
@eisenhauer
Copy link
Copy Markdown
Member Author

Closing as redundant - the changes in this PR are included in #4817

@eisenhauer eisenhauer closed this Feb 4, 2026
@eisenhauer eisenhauer deleted the MacOSStaticFix branch February 5, 2026 01:32
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.

2 participants