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

Build ocamlfind in AppVeyor tests #3901

Closed
wants to merge 3 commits into from
Closed

Conversation

dra27
Copy link
Member

@dra27 dra27 commented Jun 28, 2019

Travis (and Cygwin) both test an opam switch by installing ocamlfind. This is the very last part of #3260 bringing parity to the main testsuite (and leaving "only" opam-rt to go).

This cannot be merged until the %{exe}% variable in #3900 is merged and some hygiene patches are made to the ocamlfind package in opam-repository so that a1e0fa8 can be blown away.

@XVilka
Copy link
Contributor

XVilka commented Mar 5, 2020

Could be updated I think? Opam and ocamlfind both had changed since. Also there are warnings still:

The following actions will be performed:
  - install conf-m4   1     [required by ocamlfind]
  - install ocamlfind 1.8.0
===== 2 to install =====
<><> Processing actions <><><><><><><><><><><><><><><><><><><><><><><><><><><><>
-> installed conf-m4.1
-> retrieved ocamlfind.1.8.0  (http://download.camlcity.org/download/findlib-1.8.0.tar.gz)
[WARNING] .install file is missing .exe extension for src/findlib/ocamlfind
[WARNING] Automatically adding .exe to C:\projects\opam\.opam\default\.opam-switch\build\ocamlfind.1.8.0\src\findlib\ocamlfind.exe
[WARNING] .install file is missing .exe extension for src/findlib/ocamlfind_opt
[WARNING] Automatically adding .exe to C:\projects\opam\.opam\default\.opam-switch\build\ocamlfind.1.8.0\src\findlib\ocamlfind_opt.exe
[WARNING] C:\projects\opam\.opam\default\bin\safe_camlp4 is a script; the command won't be available
#=== ERROR while installing ocamlfind.1.8.0 ===================================#
Strict mode is enabled - previous warnings considered fatal
<><> Error report <><><><><><><><><><><><><><><><><><><><><><><><><><><><><><><>
+- The following actions failed
| - install ocamlfind 1.8.0
+- 
+- The following changes have been performed
| - install conf-m4 1
+-

@dra27 dra27 added PR: WIP Not for merge at this stage and removed PR: NEEDS UPDATE labels Mar 26, 2021
@dra27
Copy link
Member Author

dra27 commented May 17, 2022

GitHub actions install the lwt package as a test on a all platforms.

@dra27 dra27 closed this May 17, 2022
Out of release : doc, test, install, etc. automation moved this from PR to done May 17, 2022
@dra27 dra27 deleted the windows-ocamlfind branch May 17, 2022 10:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: WIP Not for merge at this stage
Development

Successfully merging this pull request may close these issues.

None yet

3 participants