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

Cywing setup hangs #6041

Closed
hhugo opened this issue Jun 21, 2024 · 5 comments · Fixed by #6046
Closed

Cywing setup hangs #6041

hhugo opened this issue Jun 21, 2024 · 5 comments · Fixed by #6046

Comments

@hhugo
Copy link
Contributor

hhugo commented Jun 21, 2024

Trying to use opam 2.2 from ocaml-setup
Setup-ocaml.v3alpha uses a separate cygwin root so that it can cache it separately from opam.
After restoring a cygwin cache and the opam one. cygwin-setup (called by opam) hangs without any logs.

See https://github.com/ocaml/Zarith/actions/runs/9596506376/job/26463523796

@mtelvers
Copy link
Contributor

mtelvers commented Jun 21, 2024

I was going to create a new issue, but instead, I'll add some context here.

When opam finds an existing installation, it tries to run setup-x86_64.exe to update the installation. However, the command it runs (or suggests that you run in another terminal window) does not include --site, so a popup dialog prompts the users to select a mirror. In my case, on Windows 2022 Server Core, that dialog is non-responsive, and I have to abort, and in your case, you can't see the dialog as it's running in GitHub actions.

@hhugo
Copy link
Contributor Author

hhugo commented Jun 21, 2024

Excellent !
The dialog being non-responsive is probably expected with --quiet-mode noinput.

For my use case, it's only after the cache is restored that it hangs. I imagine that cygwin remembers the site if called multiple time but that knowledge is lost after the cache restoration.

@dra27
Copy link
Member

dra27 commented Jun 21, 2024

It's possible to work around this by setting rootdir in HKLM\SOFTWARE\Cygwin\setup to D:\cygwin as part of setup-ocaml. In fact, I'm not so sure it's a workaround, but something it should be doing routinely.

@hhugo
Copy link
Contributor Author

hhugo commented Jun 22, 2024

The setup-ocaml issue was fixed by ocaml/setup-ocaml#817 and release in ocaml/[email protected]

@dra27
Copy link
Member

dra27 commented Jun 24, 2024

Indeed - I think there's no further action required on opam's side for this one

@dra27 dra27 closed this as completed Jun 24, 2024
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 a pull request may close this issue.

3 participants