-
Notifications
You must be signed in to change notification settings - Fork 21
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
Test with AppVeyor #28
Conversation
Note, this fails for me with:
(https://ci.appveyor.com/project/talex5-ci/ocaml-sha) Meanwhile, 0install (which has tested sha with AppVeyor as a dependency for a long time) has just started failing with:
(https://ci.appveyor.com/project/talex5-ci/0install-yp593/build/90#L398) |
Also, Travis is now failing with:
|
(this code is copied from https://raw.githubusercontent.com/ocaml/ocaml-ci-scripts/master/appveyor.yml, which doesn't specify a license - I've filed an issue at ocaml/ocaml-ci-scripts#201, but I doubt it will be a problem) |
I've enabled appveyor on this repo and rebased your branch to see what happens. |
This fails on Windows, with: /usr/include/sys/_timeval.h:40:18: error: conflicting types for ‘time_t’ typedef _TIME_T_ time_t; ^~~~~~ In file included from /usr/x86_64-w64-mingw32/sys-root/mingw/include/stddef.h:7:0, from /usr/lib/gcc/x86_64-w64-mingw32/6.4.0/include/stddef.h:1, from /usr/include/sys/reent.h:14, from /usr/include/string.h:11, from sha1.c:16: /usr/x86_64-w64-mingw32/sys-root/mingw/include/crtdefs.h:138:20: note: previous declaration of ‘time_t’ was here typedef __time64_t time_t;
Same problem. I wonder if the I've pushed a new commit to try removing it... |
Well, that got a few more lines, but:
Probably someone with a Windows machine will have to debug this. |
On Windows we omit this #include. Signed-off-by: David Scott <[email protected]>
This concept doesn't exist on Windows. We can omit this flag. Signed-off-by: David Scott <[email protected]>
Travis and appveyor are green -- I'll merge this for the release. |
Note that this fixes compilation for the |
Just to add to the confusion, I see there's another patch to support Windows (from @fdopen) at fdopen/opam-repository-mingw@cd2ebee#diff-fe866ea31ff1e66dce52f6124adbeacd. |
No description provided.