Skip to content

Should the installer default to enabling symlinks when running on the Windows 10 Creators Update? #1189

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

Closed
toolness opened this issue May 31, 2017 · 3 comments

Comments

@toolness
Copy link

toolness commented May 31, 2017

Hello! Apologies if this is the wrong place to post this, but I just noticed that support for creating symlinks outside of elevated sessions was just added in #1188. (Thanks for that, by the way!)

I also noticed when installing Git for Windows 2.13.0.windows.1--which I realize does not contain #1188--that the "Enable symbolic links" setting is disabled by default. For reference, I'm referring to this dialog:

2017-05-31_8-35-04

(It's checked in the above screenshot because I manually checked it, but it was originally unchecked.)

I'm curious if, in the next release of Git for Windows which includes #1188, this checkbox will default to being checked if the user is running the installer on the Windows 10 Creators Update. This seems like a reasonable default and might make life easier for Windows, but I'm not sure.

Anyways, thanks again for adding symlink support!

@dscho
Copy link
Member

dscho commented May 31, 2017

@toolness excellent find. Could I ask you to try your hand at patching in the 0x02 flag into the installer when GetVersion() returns at least 14972 << 16, just like #1188 does for Git itself?

You need to build the installer first and then edit /usr/src/build-extra/installer/install.iss (look for CreateSymbolicLink) before making another installer that you can then test.

@dscho
Copy link
Member

dscho commented Jun 19, 2017

@toolness I am afraid that this feature will need your help, as I am busy elsewhere.

@dscho
Copy link
Member

dscho commented Aug 22, 2017

Sigh.

@dscho dscho closed this as completed Aug 22, 2017
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

No branches or pull requests

2 participants