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
Build failure with latest Cygwin (2.11.0-1) #7845
Comments
Comment author: @bryphe This is a list of changes that came in to Cygwin 2.11.0-1: https://cygwin.com/ml/cygwin-announce/2018-08/msg00031.html There were several fixes for path handling, so I suspect that one of those changes may be related to the above error. |
Comment author: @bryphe One other note - if I copy over the 2.10.0-1 version of 'cygwin1.dll', the build works correctly. So that's been our workaround for the time being. I've also started a thread on the Cygwin mailing list: https://cygwin.com/ml/cygwin/2018-09/msg00000.html |
Comment author: @dra27 I agree that this is likely to be a regression in Cygwin 2.11 - I'll attempt to produce a smaller repro case. We should clean up the mixing of forward and backslashes anyhow - it's ugly, even if not technically incorrect. |
Comment author: @dra27 If you're using the graphic setup program, for now you can manually downgrade the Cygwin package to the previous. Unfortunately they don't expose that in the command line, so you can't do that for AppVeyor. However, if this is confirmed as a regression, I expect Corinna will push a fix very quickly. |
Comment author: @bryphe Thanks David for the response and triage! Looks like Corinna created a fix here: https://cygwin.com/snapshots/ I tested it out with a simplified version of our build pipeline: https://github.com/bryphe/test-ocaml-build and the build is now green with the patched dll: https://ci.appveyor.com/project/bryphe/test-ocaml-build Also, regarding a minimal repro, looks like they have a minimal repro here:
|
Comment author: @bryphe I like the idea of cleaning up the mixing of forward and backslashes - standardizing to forward slashes seems more reliable. |
Issue resolved, so closing. |
Original bug ID: 7845
Reporter: @bryphe
Assigned to: @dra27
Status: assigned (set by @dra27 on 2018-09-01T08:03:24Z)
Resolution: open
Priority: normal
Severity: major
Platform: Windows + Cygwin
OS: Windows
OS Version: 10
Version: 4.07.0
Category: configure and build/install
Monitored by: @nojb @hcarty
Bug description
Builds of the OCaml compiler have started to fail for me today, with errors of the form:
Before today, builds were working great in our Cygwin environment!
After some investigation, I narrowed it down to the recent Cygwin release - the cygwin dll was updated to 2.11.0 today (8/31). On an environment with the 2.10.0 version, the build is successful, but on an environment with the 2.11.0 version, the build fails with the above error.
A full build failure is available here:
https://gist.github.com/bryphe/58603ab752ecd988f78ee383fa9c9e78
Steps to reproduce
The failure is reproducible by following the MingW build steps here: https://github.com/ocaml/ocaml/blob/trunk/README.win32.adoc
, if you have the latest Cygwin environment. You can validate that you have the latest environment by checking
cygcheck -c
and looking for the cygwin version - if it is 2.11.0, it should repro.Alternatively, if you have NPM, you can follow these steps to repro:
npm install -g esy-bash@0.1.22
git clone https://github.com/ocaml/ocaml
cd ocaml
git submodule update --init
cp config/m-nt.h runtime/caml/m.h
cp config/s-nt.h runtime/caml/s.h
cp config/Makefile.mingw64 config/Makefile
esy-bash make flexdll
esy-bash make world.opt
esy-bash make flexlink.opt
esy-bash make install
The text was updated successfully, but these errors were encountered: