Skip to content
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

Updated Update Install Command to improve silent installation #279

Merged
merged 1 commit into from Mar 20, 2020

Conversation

ld0614
Copy link

@ld0614 ld0614 commented Mar 20, 2020

Relates to issue #2584. This PR should stop the new installer showing a initial install screen by using the existing installer /VERYSILENT command.

As I was using Visual Studio to make these changes an update to the .gitignore got included to avoid additional visual studio specific settings being included in the repo, I've left the .gitignore change in because I didn't think it would harm anything but can remove if required.

This is my first contribution to git so I've not managed to build and test the code however considering the minor changes I didn't know if this was required. I'll try and have a got at building the updated installer over the weekend.

Please let me know if there are any other changes that are needed to be made (patch note additions etc) and I'll get them done as well

A friendly reminder for new contributors:

What is DCO sign-off? Why is it required?

DCO stands for Developer's Certificate of Origin.

We require all commits to be DCO signed-off in case we need to track down and handle legal and/or technical issues.

To DCO sign-off your commits, you could run the git-commit command with the -s option or just add a line in your commit message in this format:

        Signed-off-by: Leo D'Arcy leo.darcy@poweronplatforms.com

For more information, check out how PRs are checked for DCO sign-off: https://github.com/probot/dco#how-it-works .

Thank you very much.


This change is Reviewable

The command successfully identifies and downloads the update however when it
triggers the update it causes a license screen to display which requires
user interaction. Therefore the update can't be fully automated.

This is due to the installer only using the /SILENT option on the new
installer. Let's add also the /VERYSILENT option so that the installer
works as expected.

Signed-off-by: Leo D'Arcy <leo.darcy@poweronplatforms.com>
Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
@dscho
Copy link
Member

dscho commented Mar 20, 2020

Thank you for your contribution!

@dscho dscho merged commit 7a5c156 into git-for-windows:master Mar 20, 2020
dscho added a commit that referenced this pull request Mar 20, 2020
`git update-git-for-windows -y` [now is fully
automatable](#279).

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants