Skip to content

Conversation

@davidhewitt
Copy link
Contributor

Uses the manylinux_2_28 container to run the aarch64 build (on an arm runner natively).

cc @pangjunrong you noted that this had problems building with the manylinux_2_28 container. From the test run on my fork, this worked fine: https://github.com/davidhewitt/connector-x/actions/runs/18595020541

Maybe I missed something?

This allows the aarch64 build to match the x86_64 one of glibc 2.28.

uses: actions/upload-artifact@v4
with:
name: "linux-arm-${{ matrix.python-version }}"
name: "linux-${{matrix.platform.arch}}-${{ matrix.python-version[1] }}"
Copy link
Contributor

@wangxiaoying wangxiaoying Oct 19, 2025

Choose a reason for hiding this comment

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

Since the name is changed from ubuntu-latest-xxx to linux-x86_64-xxx, the named referred in the verify task should also be updated? Otherwise it cannot find the artifact: https://github.com/davidhewitt/connector-x/actions/runs/18595020541/job/53024260726#step:4:16

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, I simplified the verify jobs - new CI run at https://github.com/davidhewitt/connector-x/actions/runs/18651351271

@wangxiaoying wangxiaoying merged commit e687b60 into sfu-db:main Oct 21, 2025
@wangxiaoying
Copy link
Contributor

wangxiaoying commented Oct 21, 2025

Thank you @davidhewitt ! I have merged the PR and will release an alpha version using this new workflow.

@wangxiaoying
Copy link
Contributor

The new alpha version is release, feel free to try out: pip install connectorx==0.4.5a1!

@davidhewitt
Copy link
Contributor Author

Super, thanks very much!

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