Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Build and test wheels for x86_64 & arm64 #102

Merged
merged 2 commits into from
Feb 28, 2024
Merged

Conversation

andreas-el
Copy link
Contributor

No description provided.

@andreas-el andreas-el force-pushed the work_on_carolina_build branch 23 times, most recently from 4273109 to 840d9c1 Compare February 28, 2024 13:44
@andreas-el andreas-el changed the title Work on carolina build Build and test wheels for x86_64 & arm64 Feb 28, 2024
@andreas-el andreas-el self-assigned this Feb 28, 2024

jobs:
build_dependencies:
strategy:
fail-fast: true
fail-fast: false

Choose a reason for hiding this comment

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

was this intended for dev or production?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This made most sense when developing, but I see why it would make more sense to do a full stop if something fails in production. Lets flip this to true.

fi
python --version

export PYTHON_EXECUTABLE=$(which python3)
Copy link

@yngve-sk yngve-sk Feb 28, 2024

Choose a reason for hiding this comment

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

I think I wrote this python3 originally for some reason, but does it maybe make sense to just write python instead of python3 here, if it has the same effect? Since we are sourcing the venv which is python3 anyway.

export _PYTHON_HOST_PLATFORM="macosx-11.0-$arch_ver"
export ARCHFLAGS="-arch $arch_ver"

if [ "$arch_ver" == "arm64" ]; then

Choose a reason for hiding this comment

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

Suppose this is necessary to make it work, when/if GH actions change their runner/arch combinations, will this still hold up?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The tools actually update the version number in the host platform, so this hints to the wheel build that we don't want universal wheels, but rather arm64 or x86_64 wheels separate.
I will add a comment to this since the next person won't know these details.

Copy link

@yngve-sk yngve-sk left a comment

Choose a reason for hiding this comment

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

Good job! Some comments/questions to address, feeling pretty confident you can address them and then merge this

@andreas-el andreas-el merged commit e107cf4 into main Feb 28, 2024
27 checks passed
@andreas-el andreas-el deleted the work_on_carolina_build branch February 28, 2024 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants