Skip to content

Copy the mpt-crypto static archive and link manifest, not the shared object - #94

Merged
manasip-prog merged 1 commit into
mainfrom
manasip/mpt-crypto-static-archive
Sep 8, 2026
Merged

Copy the mpt-crypto static archive and link manifest, not the shared object#94
manasip-prog merged 1 commit into
mainfrom
manasip/mpt-crypto-static-archive

Conversation

@manasip-prog

Copy link
Copy Markdown
Collaborator

build workload image has not succeeded since #92 moved XRPL_PY_REF from pre-3.3-release-group to main. It was masked until now because every run since was failing earlier — first on the deleted confidential-mpt branch, then on expired credentials.

xrpl-py main changed how the cffi extension links. build_mpt_crypto.py now passes libs/linux/libmpt-crypto.a as an extra_object and reads the system libraries to co-link from mpt-crypto-static.link-libs.txt beside it, producing a self-contained .so. pre-3.3-release-group loaded libmpt-crypto.so at runtime instead. The setup script still copies only the .so, so the link has no archive to resolve against:

WARNING: mpt-crypto static archive not found at .../libs/linux/libmpt-crypto.a
gcc -shared ./_mpt_crypto.o .../libs/linux/libmpt-crypto.a ... → collect2: error: ld returned 1 exit status
cffi.VerificationError: LinkError

Both files ship in the same mpt-crypto-natives-<version>.tar.gz already being downloaded, so this copies them instead of the .so.

Observed on ripple/rippled-antithesis run 33932187051, where build xrpld passes and build workload image fails on the link.

🤖 Generated with Claude Code

@manasip-prog
manasip-prog force-pushed the manasip/mpt-crypto-static-archive branch from 211d955 to 5f485b1 Compare September 8, 2026 14:40
Comment thread workload/scripts/setup-confidential-crypto.sh
@manasip-prog
manasip-prog merged commit 82ceb94 into main Sep 8, 2026
1 check passed
@manasip-prog
manasip-prog deleted the manasip/mpt-crypto-static-archive branch September 8, 2026 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants