Skip to content

Commit ee51ee8

Browse files
committed
Use --symlink-type=sys for Cygwin setup
Cygwin setup 2.912 has a bug in native Windows symlink support and writes some corrupt symlinks; in particular /etc/ssl/certs is corrupt which breaks git and curl. Depexts will still install using native Windows symlinks, but this should hopefully be less of an issue as the bug affects absolute symlinks, which are less common outside of the base packages.
1 parent 2e737a1 commit ee51ee8

File tree

4 files changed

+6
-0
lines changed

4 files changed

+6
-0
lines changed

CHANGELOG.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,9 @@ and this project adheres to
1212

1313
- Fallback to the version in which the assets exist if no assets exist in the
1414
latest opam release.
15+
- Instruct Cygwin setup to use "sys" symlinks during setup (partial workaround
16+
for bug with native symlinks in Cygwin setup - some depexts may still be
17+
affected)
1518

1619
## [2.0.0-beta11]
1720

dist/index.js

Lines changed: 1 addition & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/post/index.js

Lines changed: 1 addition & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/setup-ocaml/opam.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -202,6 +202,7 @@ async function setupCygwin() {
202202
site,
203203
"--packages",
204204
packages,
205+
"--symlink-type=sys",
205206
]);
206207
const setupExePath = await io.which("setup-x86_64.exe");
207208
await io.cp(setupExePath, CYGWIN_ROOT);

0 commit comments

Comments
 (0)