microsoft.git not respecting --disable-interactivity flag in winget about git HOT 5 CLOSED

Hi @shalinikhare27, thanks for the report!

This definitely seems like a case where the installer is not behaving the way that you expect. The microsoft/git fork inherits its installer from git-for-windows/git, so any change would need to happen in that project (and probably specifically in git-for-windows/build-extra).

One thing that would help us investigate this concern is how winget communicates the --disable-interactivity flag down to the installer. Somehow we know to not display the wizard, so some amount of integration already exists. It just hasn't been wired down to these kind of pop-up error messages.

It is important to note is that this prompt is communicating a failed installation procedure, so any integration to prevent the pop-up will still need to write an error to stderr and return a failure.

from git.

@derrickstolee - yes, ideally we want it to respect the --disable-interactivity flag and write an error to stderr. This is important when we run certain installations in non-user context where prompts are meaningless. We want to just exit with a failure. Please let me know if I can provide any other data to debug this.

from git.

I think I found the way to do it, but it will require some manual testing.

from git.

@derrickstolee How can I test this on my end?

from git.

@shalinikhare27: Sorry for the delay in updating this issue, but the git-for-windows/build-extra fix ended up not being needed since there is a way to fix it via updating the winget manifest.

There are instructions in the PR description of how to test this locally, without needing any changes to the microsoft/git release assets. I've tested it myself and as soon as that PR is merged by the winget team this issue will be fixed.

from git.