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

Bug 1536013 - Select win64-aarch64 builds when appropriate. #518

Merged
merged 3 commits into from Apr 3, 2019

Conversation

KwanEsq
Copy link
Collaborator

@KwanEsq KwanEsq commented Mar 19, 2019

Doing this to hopefully get a build for testing.

Copy link
Contributor

@wlach wlach left a comment

Choose a reason for hiding this comment

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

This all looks reasonable to me, I just added you as a collaborator to the mozregression repository, so feel free to merge when ready. I'd like to upload a new revision of mozregression today if possible (there are some other changes which are overdue to go out).

@KwanEsq
Copy link
Collaborator Author

KwanEsq commented Mar 21, 2019

@wlach ah, I was actually thinking that the proper way to do this would be to update mozinfo and push a new version of that first. It should probably be returning 64 bits for ARM64, as it does for x86-64 even when running on 32-bit Python. (It might also want to return aarch64 for processor instead of ARM64, ala x86_64 instead of AMD64)

@KwanEsq
Copy link
Collaborator Author

KwanEsq commented Mar 21, 2019

Thanks a lot for the collaborator add!
Now I just need to be careful I don't hit any new buttons...

@KwanEsq KwanEsq self-assigned this Mar 21, 2019
@wlach
Copy link
Contributor

wlach commented Mar 21, 2019

@KwanEsq oh ok, if there's a more elegant fix involving mozinfo, let's go for that. I probably shouldn't review a patch for that, but can help with pushing that out to pypi once it's landed.

@wlach
Copy link
Contributor

wlach commented Mar 21, 2019

for now i'll do a new release of mozregression with the other fixes that are pending

@KwanEsq
Copy link
Collaborator Author

KwanEsq commented Mar 29, 2019

Whoops, pushed the wrong branch, didn't mean to do that yet.

Due to a too-broad regex trying to mozregress jsshell on win64 will have been
grabbing the wrong build for the architecture ever since aarch64 builds started
being produced.

Adds a test for this.
@KwanEsq KwanEsq marked this pull request as ready for review April 2, 2019 09:22
@coveralls
Copy link

Coverage Status

Coverage increased (+0.04%) to 84.437% when pulling ba78773 on KwanEsq:win64-aarch64 into eac4771 on mozilla:master.

@KwanEsq KwanEsq merged commit 370840f into mozilla:master Apr 3, 2019
@KwanEsq KwanEsq deleted the win64-aarch64 branch April 3, 2019 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants