Skip to content

Conversation

@chrishiestand
Copy link
Contributor

trying to install qcow.0.10.3 yields a dependency problem:

The following dependencies couldn't be met:

  • qcow-tool → qcow >= 0.10.4
  • qcow-tool → sha = 1.9 → ocaml < 4.06.0
    base of this switch (use `--unlock-base' to force)
    Your request can't be satisfied:
  • No available version of qcow satisfies the constraints

No solution found, exiting

But qcow.0.10.4 works

@GordonTheTurtle
Copy link

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:

$ git clone -b "patch-1" git@github.com:chrishiestand/hyperkit.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

trying to install qcow.0.10.3 yields a dependency problem:

> The following dependencies couldn't be met:
>   - qcow-tool → qcow >= 0.10.4
>   - qcow-tool → sha = 1.9 → ocaml < 4.06.0
>       base of this switch (use `--unlock-base' to force)
> Your request can't be satisfied:
>   - No available version of qcow satisfies the constraints
>
> No solution found, exiting

But qcow.0.10.4 works

Signed-off-by: Chris Hiestand <chrishiestand@gmail.com>
@ijc
Copy link
Collaborator

ijc commented Jan 30, 2019

@djs55 this seems sensible to me but doesn't seem to match what is in repo/darwin/packages/upstream/, perhaps they should be in sync? WDYT?

@djs55
Copy link
Collaborator

djs55 commented Jan 30, 2019

Ideally the README constraints would match those in repo/darwin/packages/upstream since those are tested by the CI and the README isn't.

I'll see if I can successfully update the repo/ constraints in a separate PR and report back.

djs55 added a commit to djs55/hyperkit that referenced this pull request Jan 30, 2019
This is the CI counterpart of moby#244

Signed-off-by: David Scott <dave.scott@docker.com>
djs55 added a commit to djs55/hyperkit that referenced this pull request Jan 31, 2019
This is the CI counterpart of moby#244

Signed-off-by: David Scott <dave.scott@docker.com>
@ijc
Copy link
Collaborator

ijc commented Feb 1, 2019

#245 was merged and I think this can just be merged on top, no need to rebase. So I shall do that.

@ijc ijc merged commit 64bbfbe 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.

4 participants