-
-
Notifications
You must be signed in to change notification settings - Fork 662
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
Update windows build documentation #11960
base: development
Are you sure you want to change the base?
Conversation
4b7dbf0
to
ce8edee
Compare
I still haven't tried this locally and I'm scared to do so, but if you say that this is the new way then I'll believe you. Let me know when this is good to merge. |
I think if you check out a new copy of the haxe repository for testing and get the native opam to manage its own separate cygwin install, it should be possible to test this in a way that you can just delete the
I'm wondering if there is a better way to expose make to the build environment other than adding the cygwin bin directory to PATH, but maybe that's for the future. Perhaps we should wait until someone else follows the instructions to double check I didn't miss anything. |
ce8edee
to
fd7daeb
Compare
|
||
```pwsh | ||
& $(opam exec -- cygpath -w "/bin/tar") -C / -xvf path/to/package.tar.xz | ||
``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This fails with:
/usr/bin/tar: Child returned status 127
/usr/bin/tar: Error is not recoverable: exiting now
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the output if you run these commands:
opam exec -- cygpath -w "/bin/tar"
& (opam exec -- cygpath -w "/bin/bash") -c "type tar"
Opam runs natively on Windows now and manages cygwin automatically, so there is no need to install cygwin manually anymore. These instructions explain how to set up opam like this.
A workaround is currently needed for luv, see: aantron/luv#162