-
Notifications
You must be signed in to change notification settings - Fork 62
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
Fix karamel.opam: add ctypes and ctypes-foreign #391
base: master
Are you sure you want to change the base?
Conversation
Build fail, please try to see with @pnmadelaine what's up |
@pnmadelaine doesn't know what's going on -- @tahina-pro do you have a clue? |
Maybe @mtzguido knows? |
I think it's jump OPAM defaulting to |
Sigh I think this is a discrepancy between the standalone build and the hierarchic one. The standalone build already does set OPAMYES=1, so the previous CI job from the original commit should have worked. The ones for my pushes just now are hierarchical (since there's an F* image on the machine), and seem to work even before my change (the merge is green). I think the proper thing to do is removing the two flavors and just have one, but unsure on what that actually looks like if we want it to be fast. Also this should at least work around the problem if it shows up again.
|
@cmovcc can I let you add Guido's patch to this PR? thanks! |
What's the status of this? Should it be merged? |
No description provided.