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
Error building with OPAM #64
Comments
I believe that this issue stems from a missing |
This seems related to the changes introduced from |
Someone might want to submit a patch to the opam file to fix this for now. (The Makefile can be patched in the opam build process.) That's probably the fastest way to get past the problem. |
More like But yes, a quick fix would be appreciated because this takes down our entire CI. |
The quickest fix for anyone is to force use of A slightly "better" quick fix is to put the Makefile patch into the opam file and request a pull to the opam repository. |
Note that I've pushed ocaml/opam-repository#12556 to (temporary) fix the issue in opam. |
Can we close this for now, or should we keep it open until the underlying issue is fixed? |
Hi, There is a patch that fixes build against ppx_tools 5.2.1: `--- sedlex-1.99.4/src/syntax/Makefile.orig 2018-10-25 07:55:34.821346628 +0000 OCAMLC=ocamlfind ocamlc all: ppx_sedlex$(EXE) sedlex.cma |
If you would like to submit a patch, please do so via the pull request mechanism. |
Er — is there a reason it's not constrained to ‘below 5.2.1’ in this repository? This issue trickled down to my fork in an unrelated fashion, and I'm confused as to why Happy to PR if upstreaming of this constraint is desired. |
It's intentional. The constraint |
@smolkaj thanks for the explanation! <3 |
I opened #84 to help fix this. I was getting bitten by a "catch 22" situation in nixpkgs trying to build sedlex (<2.0) for ocaml 4.08. |
@matthewess I've merged it. Is the problem gone? |
we're now using ppxlib, should be closed |
Since this morning, I've had the following issue:
does that ring a bell? the travis logs are available here: https://travis-ci.org/FStarLang/kremlin/builds/421260062?utm_source=slack&utm_medium=notification if that helps
The text was updated successfully, but these errors were encountered: