-
Notifications
You must be signed in to change notification settings - Fork 633
please.sh: teach create_sdk_artifact to use architecture instead of bitness
#462
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
Conversation
|
The failing |
| bitness=auto | ||
| architecture= | ||
| bitness= |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not seeing any places that are relying on bitness=auto (including setup-git-for-windows-sdk), so I think we're good to explicitly require --bitness or --architecture to be set. Please let me know if that's not the case.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought my manual runs were relying on this, allowing me to be lazy and not specify anything but the <name> ;-) But I guess I broke it long ago. So we're good to leave --bitness=auto behind.
That issue was fixed in git-for-windows/git-sdk-64@f0c95e9. |
0bef871 to
d267364
Compare
d267364 to
f5350bd
Compare
Git for Windows's toolchain is moving away from "bitness" as it doesn't support arm64. See the attached discussion for more details. Signed-off-by: Dennis Ameling <[email protected]> Ref: git-for-windows/git#4203
f5350bd to
e292201
Compare
|
Now that git-for-windows/git-sdk-arm64#6 has been merged, I can confirm that this also works as expected on Small note: the CC @dscho |
I believe that the problem is solved via 50deb2b for i686, and may need something similar for x86_64. |
| bitness=auto | ||
| architecture= | ||
| bitness= |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought my manual runs were relying on this, allowing me to be lazy and not specify anything but the <name> ;-) But I guess I broke it long ago. So we're good to leave --bitness=auto behind.
Git for Windows's toolchain is moving away from "bitness" as it doesn't support arm64. See the attached discussion for more details.
Ref: git-for-windows/git#4203