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

AppVeyor: Fix creation of symblic links for good #31

Merged
merged 1 commit into from
Oct 4, 2018
Merged

AppVeyor: Fix creation of symblic links for good #31

merged 1 commit into from
Oct 4, 2018

Conversation

sschuberth
Copy link
Contributor

Description of changes:

The main repository contains symbolic links to directories in the
submodule. As the submodule is initialized after the main repository, it
is unknown to Git whether the links point to files or directories. On
Windows, this is important to know as symbolic links can point to
either files or directories, and the type has to be known at the point of
their creation. The default is to create file symbolic links, which is
wrong in this case. So delete the symbolic links and check them out
again after the submodule has been initialized to get them recreated as
directory symbolic links.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

The main repository contains symbolic links to directories in the
submodule. As the submodule is initialized after the main repository, it
is unknown to Git whether the links point to files or directories. On
Windows, this is important to know as symbolic links can point to
either files or directories, and the type has to be known at the point of
their creation. The default is to create file symbolic links, which is
wrong in this case. So delete the symbolic links and check them out
again after the submodule has been initialized to get them recreated as
directory symbolic links.
@jpeddicord
Copy link
Owner

Cool -- looks like AppVeyor is reporting a successful status here too. Thanks for the change!

@jpeddicord jpeddicord merged commit ac60ec2 into jpeddicord:master Oct 4, 2018
@sschuberth sschuberth deleted the appveyor-symlinks branch October 4, 2018 16:34
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.

2 participants