Skip to content

aerospike: Add version 19.2.2 - #2310

Merged
luhenry merged 1 commit into
mainfrom
aerospike
Sep 25, 2026
Merged

luhenry merged 1 commit into
mainfrom
aerospike

Conversation

@luhenry

@luhenry luhenry commented Sep 25, 2026 •

Copy link
Copy Markdown
Member

Compiles the Aerospike Python client against its bundled aerospike-client-c submodule (C, Apache-2.0). Upstream publishes no riscv64 wheel.

Mirrors upstream's manylinux build path (build-wheel.bash / _build-wheel-natively.bash).

Differs from upstream

  • dnf installs libyaml-devel/openssl-devel on the stock riscv64 image; upstream's custom OpenSSL-baked image has no riscv64 build.

Testing

  • No live Aerospike server in CI; smoke-tests the compiled extension instead of the integration suite.

License: OK

Patches

  • 0001-as_arch-portable-pause-fallback.patch - To upstream (not yet submitted). as_arch_pause() is undeclared on riscv64 under -Werror. riscv64-only.

Built and imported the wheel on x86_64 locally (sdist -> wheel, no cibuildwheel); extension loads, client() smoke test passes.

Ports the Aerospike Python client (a cibuildwheel build-from-checkout of
aerospike-client-python, which bundles the aerospike-client-c C client SDK
as a git submodule). One patch is needed: as_arch.h's as_arch_pause() spin
hint is only defined for __x86_64__/__aarch64__, and is used unconditionally
from always-compiled spinlock/rwlock inline functions in as_atomic_gcc.h,
so riscv64 hits an implicit-declaration -Werror failure without a portable
no-op fallback.

libyaml-devel (CRB) and openssl-devel (AppStream) both resolve on the
riscv64 manylinux image, matching upstream's own yum install line. No live
Aerospike server in CI; the test command proves the extension compiled and
exercises the real C cluster-init code path (the one the patch touches) via
a fast, deterministic ConnectionError against a closed port.
@github-actions

github-actions Bot commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-09-25 11:26 UTC

luhenry added a commit that referenced this pull request Sep 25, 2026
@luhenry
luhenry merged commit 1ca4642 into main Sep 25, 2026
14 of 15 checks passed
@luhenry
luhenry deleted the aerospike branch September 25, 2026 11:26
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.

1 participant