-
Notifications
You must be signed in to change notification settings - Fork 2.7k
Update the installer git uses to 5.5.8 please #647
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
Comments
We are dual signing without the need for the new feature. And we do not use the IDE. |
Oh ok. |
This completely misses that we are using 5.5.6, still, and it is quite possible that our use of
That would be a rather dramatic regression, and it would have been good if you would have tested this before opening this ticket (you have been around long enough to know how to make an installer). |
I did the menial work of testing this and found |
I noticed the `osx-gcc` job failing in git-for-windows#647 so I found this upstream fix from @peff. Merging now to unblock PR builds in `microsoft/git`.
Hi please update Inno Setup to 5.5.8
The change log for changes is http://www.jrsoftware.org/files/is5-whatsnew.htm
The main changes I think are good are
•The [Setup] section may now list multiple SignTool directives which will be executed in order of appearance. This can be used to dual sign (SHA1 & SHA256) Setup and Uninstall. This requires a recent signtool.exe version and a SHA256 (SHA-2) certificate. Inno Setup's own installer is now also dual signed. See the help file for a dual sign example. Without dual signing Windows will no longer trust any of your new installers if they are downloaded. See http://social.technet.microsoft.com/wiki/contents/articles/32288.windows-enforcement-of-authenticode-code-signing-and-timestamping.aspx for more information.
Which we are doing.
•The Compiler IDE's New Script Wizard now offers to create a shortcut to the main executable in the common Start Menu Programs folder instead of creating a new Start Menu folder especially for the application. This option is enabled by default and is recommended by Microsoft unless you install a suite of applications rather than a single application.
The text was updated successfully, but these errors were encountered: