Skip to content

GIT_ASK_YESNO/GIT_ASKPASS patches from Git for Windows#3

Closed
dscho wants to merge 4 commits intoprati0100:masterfrom
dscho:git-gui-askpass
Closed

GIT_ASK_YESNO/GIT_ASKPASS patches from Git for Windows#3
dscho wants to merge 4 commits intoprati0100:masterfrom
dscho:git-gui-askpass

Conversation

@dscho
Copy link
Contributor

@dscho dscho commented Sep 19, 2019

This is another set of patches from Git for Windows' fork that have been sitting there since 2010, providing cross-platform GUI helpers to ask the user a question or allow typing in a password.

hvoigt and others added 4 commits September 19, 2019 13:39
Make use of the new environment variable GIT_ASK_YESNO to support the
recently implemented fallback in case unlink, rename or rmdir fail for
files in use on Windows. The added dialog will present a yes/no question
to the the user which will currently be used by the windows compat layer
to let the user retry a failed file operation.

Signed-off-by: Heiko Voigt <hvoigt@hvoigt.net>
Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
"Question?" is maybe not the most informative thing to ask. In the
absence of better information, it is the best we can do, of course.

However, Git for Windows' auto updater just learned the trick to use
git-gui--askyesno to ask the user whether to update now or not. And in
this scripted scenario, we can easily pass a command-line option to
change the window title.

So let's support that with the new `--title <title>` option.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
For additional GUI goodness.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
@dscho
Copy link
Contributor Author

dscho commented Sep 19, 2019

I should have added: this supersedes patthoyts/git-gui#5.

@prati0100
Copy link
Owner

@dscho,

Please send this PR and your other 2 PRs via email. Also, Cc the Git mailing list so other people interested in git-gui can take a look.

I will close these 3 pull requests here and wait for your patches via email. Thanks.

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