Skip to content

Conversation

@djs55
Copy link
Collaborator

@djs55 djs55 commented Jan 30, 2019

Newer OCaml versions require a newer qcow version, see #244.

This PR:

  • uses the 1.2 branch of the upstream opam-repository. When we update to opam 2 we can use the master branch again
  • bumps qcow to 0.10.4 and sha to 1.10

cd ${REPO_DIR_NAME}
git checkout 1.2
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FWIW I think this change is equivalent to

go clone --depth=1 --branch=1.2 ${OPAM_REPO} ${REPO_DIR_NAME}

which retains the shallow clone. Not sure if 1.2 here is a branch or a tag, but --branch is documented to accept either.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok will try that, thanks. (with git rather than go :)

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bah!

I suppose I could edit my previous comment to make you look like a loon but I won't ;-)

djs55 added 5 commits January 31, 2019 14:27
Signed-off-by: David Scott <dave.scott@docker.com>
This is the CI counterpart of moby#244

Signed-off-by: David Scott <dave.scott@docker.com>
Signed-off-by: David Scott <dave.scott@docker.com>
Signed-off-by: David Scott <dave.scott@docker.com>
Signed-off-by: David Scott <dave@recoil.org>
@djs55
Copy link
Collaborator Author

djs55 commented Feb 1, 2019

I think this is good to merge, and then #244 on top.

@ijc ijc merged commit eca0f71 into moby:master Feb 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants