Skip to content

git-xet Windows installer and code signing#519

Merged
seanses merged 10 commits intomainfrom
di/git-xet-windows-sign-install
Oct 2, 2025
Merged

git-xet Windows installer and code signing#519
seanses merged 10 commits intomainfrom
di/git-xet-windows-sign-install

Conversation

@seanses
Copy link
Collaborator

@seanses seanses commented Oct 1, 2025

This PR

  • adds to the git-xet release workflow to code-sign Windows executable "git-xet.exe" using the Microsoft Trusted Signing Service;
  • builds a Windows installer for git-xet to place "git-xet.exe" in the system, modify the system PATH environment variable, and run the command "git-xet install" to configure git-xet; On uninstallation from "Control Panel\Programs\Programs and Features", it first runs "git-xet uninstall --all" so it is deregistered from git-lfs custom transfer.
  • signs the built Windows installer msi file.

A test commit created a pre-release at https://github.com/huggingface/xet-core/releases/tag/git-xet-v0.1.0 with the signed Windows executables.

@seanses seanses changed the title Windows codesign and installer git-xet Windows installer and code signing Oct 1, 2025
@seanses seanses marked this pull request as ready for review October 2, 2025 00:40
Copy link
Collaborator

@rajatarya rajatarya left a comment

Choose a reason for hiding this comment

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

I have a couple minor things - nothing that should block this from getting merged.

@seanses seanses merged commit 6fbde98 into main Oct 2, 2025
6 checks passed
@seanses seanses deleted the di/git-xet-windows-sign-install branch October 2, 2025 19:40
with:
azure-tenant-id: ${{ inputs.azure_tenant_id }}
azure-client-id: ${{ inputs.azure_client_id }}
azure-client-secret: ${{ inputs.azure_client_secret }}

Choose a reason for hiding this comment

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

Not urgent, but could we use the Github Oauth instead of client ID / Secret?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I'll look into it!

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.

5 participants