-
Notifications
You must be signed in to change notification settings - Fork 49
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
Package updates #411
Package updates #411
Conversation
e3526cb
to
4237783
Compare
if [ "$OSTYPE" != cygwin ] | ||
then | ||
# Windows: stack overflow |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is probably not the right place to discuss this (maybe open an issue on fiat crypto or here?), but fiat crypto does build with opam on Windows / cygwin, according to our CI.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't really know much about this issue. Maybe @MSoegtropIMC can discuss this with you ?
26d3fd4
to
f3db40d
Compare
if [[ "$OSTYPE" != cygwin ]] | ||
then | ||
PACKAGES="${PACKAGES} coq-mtac2.1.4+8.19" # build issues on Windows | ||
fi | ||
======= |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rtetley : This file has merge conflicts in it. You need to resolve this before pushing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
huh ? weird, I rebased and fixed.. Must have slipped my attention...
Co-authored-by: Jason Gross <[email protected]>
d6b26db
to
7529da6
Compare
Bumping: