Skip to content
Snippets Groups Projects

Allow compiling against "dev" version of Coq

Merged Ralf Jung requested to merge ralf/opam into master

This matches e.g. Iris allowing a "dev" version of std++: You can install a "dev" version to test stuff, but then you are responsible for making sure that these versions actually work together. We rely on that when testing things against Iris master every night, for which purpose we install Iris master as "dev" version.

Cc @jjourdan who, IIRC, argue for removing the dev version some time ago.

If we accept this here, I will send similar patches to (some of?) our other projects.

Edited by Ralf Jung

Merge request reports

Approval is optional

Merged by Ralf JungRalf Jung 7 years ago (Nov 29, 2017 12:25pm UTC)

Merge details

  • Changes merged into master with 2b35253e.
  • Deleted the source branch.

Pipeline #5647 passed

Pipeline passed for 2b35253e on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Ralf Jung changed the description

    changed the description

  • @janno mentioned on Mattermost that there also was a problem with opam updating coq to the dev version on make build-dep. That infrastructure has changed since then, so this problem may no longer be present.

  • I'm not sure what I said exactly, but here is what I think is important: it should be possible to install the dev version of Iris using a non-dev version of Coq. And, possibly, similarly for a non-dev version of SSR.

  • it should be possible to install the dev version of Iris using a non-dev version of Coq. And, possibly, similarly for a non-dev version of SSR.

    This patch adds more versions of Coq that can be used. So, of course, non-dev versions will keep working.

  • Ralf Jung mentioned in commit 2b35253e

    mentioned in commit 2b35253e

  • merged

  • Merging because JH's requirements are satisfied. (RK ack'ed privately.)

Please register or sign in to reply
Loading