Skip to content
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

Update docker repo, and mirage-net-psock dependency #8

Open
wants to merge 4 commits into
base: 0.5.3-dev
Choose a base branch
from

Conversation

Cjen1
Copy link

@Cjen1 Cjen1 commented Nov 14, 2019

The name for opam docker image has changed and mirage-net-psock needs to be pinned against an old version of the repo to allow it to build. This PR fixes both of those issues.

(additional -ys to allow docker building)

Dockerfile Outdated Show resolved Hide resolved
Dockerfile Outdated Show resolved Hide resolved
Dockerfile Outdated
FROM ocaml/opam-dev:alpine-3.6_ocaml-4.04.2 as BUILDER
FROM ocaml/opam2:alpine-3.7-ocaml-4.04 as BUILDER

ENV OCAMLYES=true
Copy link

Choose a reason for hiding this comment

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

Should be OPAMYES, no?

Dockerfile-relay Outdated

WORKDIR /core-network
ADD core-network.export core-network.export

ENV OCAMLYES=true
Copy link

Choose a reason for hiding this comment

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

OPAMYES...?

@mor1
Copy link

mor1 commented Dec 27, 2019

Seems to have build failures...?

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