Skip to content

Conversation

@dennisameling
Copy link
Contributor

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

@dennisameling
Copy link
Contributor Author

The failing check-for-missing-dlls jobs are unrelated to this PR, but rather to yesterday's commit to git-sdk-64: git-for-windows/git-sdk-64@b980ccf

Comment on lines -4477 to +3099
bitness=auto
architecture=
bitness=
Copy link
Contributor Author

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.

Copy link
Member

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.

@rimrul
Copy link
Member

rimrul commented Dec 30, 2022

The failing check-for-missing-dlls jobs are unrelated to this PR, but rather to yesterday's commit to git-sdk-64: git-for-windows/git-sdk-64@b980ccf

That issue was fixed in git-for-windows/git-sdk-64@f0c95e9.
I re-ran the failed checks, they're looking good now.

@dennisameling dennisameling force-pushed the sdk-artifact-arm64 branch 2 times, most recently from 0bef871 to d267364 Compare January 29, 2023 15:06
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
@dennisameling
Copy link
Contributor Author

Now that git-for-windows/git-sdk-arm64#6 has been merged, I can confirm that this also works as expected on git-sdk-arm64:

$ /usr/src/build-extra/please.sh create-sdk-artifact --architecture=aarch64 --out=/sdk-build-installers build-installers
Preparing worktree (detached HEAD 4f9bda7791)
Updating index flags: 100% (89057/89057), done.
Updating files: 100% (993/993), done.
grep: /clangarm64/bin/git.exe: No such file or directory
Updating index flags: 100% (89057/89057), done.
Updating files: 100% (16948/16948), done.
Output written to 'C:/git-sdk-arm64/sdk-build-installers'

Small note: the no such file or directory warning already pops up on this script today without the changes in this PR:

$ /usr/src/build-extra/please.sh create-sdk-artifact --bitness=64 --out=/sdk-build-installers build-installers
Preparing worktree (detached HEAD 71754d7913)
Updating index flags: 100% (118700/118700), done.
Updating files: 100% (1201/1201), done.
grep: /mingw64/bin/git.exe: No such file or directory
Updating index flags: 100% (118700/118700), done.
Updating files: 100% (16094/16094), done.
Output written to 'C:/git-sdk-64/sdk-build-installers'

CC @dscho

@dscho
Copy link
Member

dscho commented Feb 6, 2023

Small note: the no such file or directory warning already pops up on this script today without the changes in this PR:

I believe that the problem is solved via 50deb2b for i686, and may need something similar for x86_64.

Comment on lines -4477 to +3099
bitness=auto
architecture=
bitness=
Copy link
Member

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.

@dscho dscho merged commit dc86a16 into git-for-windows:main Feb 6, 2023
@dennisameling dennisameling deleted the sdk-artifact-arm64 branch February 6, 2023 17:05
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.

3 participants