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

Fix i386 builds reporting x86_64 in the build process #189

Merged
merged 1 commit into from
Jan 3, 2024

Conversation

bdraco
Copy link
Member

@bdraco bdraco commented Jan 3, 2024

Image builds failed on i386 because the builder tried to install the x86_64 package which doesn't exist on i386 https://github.com/home-assistant/core/actions/runs/7392726377/job/20111619276 because python saw the arch as x86_64 instead of i386

The wheel builder is able to handle this case so the goal is to match https://github.com/home-assistant/wheels/blob/6a18828c159faf31114eb1bf4b2ffae30d7fbc66/rootfs/bin/run-builder.sh#L5

@bdraco bdraco marked this pull request as ready for review January 3, 2024 04:16
@bdraco
Copy link
Member Author

bdraco commented Jan 3, 2024

I'm pretty sure this will solve the issue but I don't have a good way to test it on my local arm system

@frenck
Copy link
Member

frenck commented Jan 3, 2024

This change will not harm, so let's try

@frenck frenck merged commit 1f15d48 into master Jan 3, 2024
10 checks passed
@frenck frenck deleted the mask_cpu_i386 branch January 3, 2024 07:44
@bdraco
Copy link
Member Author

bdraco commented Jan 4, 2024

I guess we need to do a release of this one and bump it in core to fix nightly.

I've never done a release on this repo before so I'm not sure if there is anything more involved that marking the draft as the latest release in github so I've refrained from hitting the button to ensure I don't break anything.

@frenck
Copy link
Member

frenck commented Jan 4, 2024

I will pick up, didn't do yesterday because of the main release.

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