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

Tweaks #4625

Merged
merged 5 commits into from Apr 14, 2021
Merged

Tweaks #4625

merged 5 commits into from Apr 14, 2021

Conversation

dra27
Copy link
Member

@dra27 dra27 commented Apr 1, 2021

Various things found debugging the problem in #4607. The main one is that we should do cygpath -- foo in the wrapper not cygpath foo (I got a surprise running tar --version....!)

@dra27 dra27 added this to PR in Progress in Opam 2.1.x via automation Apr 1, 2021
@dra27 dra27 added this to the 2.1.0~rc milestone Apr 1, 2021
@AltGr AltGr merged commit 123419a into ocaml:master Apr 14, 2021
Opam 2.1.x automation moved this from PR in Progress to Done Apr 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Opam 2.1.x
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants