Page MenuHomeFreeBSD

Update sysutils/docker-freebsd to new version
Needs ReviewPublic

Authored by joneum on Mon, Sep 9, 8:20 AM.

Details

Summary

this review is intended to help integrate dockers into FreeBSD
and make them run.

Help is welcome

TODO: runC, libnetwork and containerd

Test Plan

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 26553
Build 24941: arc lint + arc unit

Event Timeline

joneum created this revision.Mon, Sep 9, 8:20 AM
joneum added a reviewer: bcr.Mon, Sep 9, 8:21 AM
joneum updated this revision to Diff 61822.Mon, Sep 9, 8:24 AM

Add more files

joneum updated this revision to Diff 61823.Mon, Sep 9, 8:27 AM

Fix Makefile

0mp added a subscriber: 0mp.Mon, Sep 9, 11:02 AM
0mp added inline comments.
sysutils/docker-freebsd/Makefile
9

Shouldn't moby be capitalized?

joneum added inline comments.Mon, Sep 9, 11:08 AM
sysutils/docker-freebsd/Makefile
9

We can change that. At the moment we are concentrating on the port building. We will then see what "COMMENT" says at the end ... :-)

linimon retitled this revision from Update docker to new Version to Update sysutils/docker-freebsd to new version and rename.Mon, Sep 9, 8:20 PM
dmgk added a subscriber: dmgk.Tue, Sep 10, 4:12 PM
dmgk added inline comments.
sysutils/docker-freebsd/Makefile
23

Either GH_SUBDIR or GO_PKGNAME is OK, specifying both is not necessary. GO_PKGNAME gets inferred from GH_SUBDIR if omitted.

24

Could be just ./cmd/dockerd

joneum retitled this revision from Update sysutils/docker-freebsd to new version and rename to Update sysutils/docker-freebsd to new version.Thu, Sep 12, 10:24 AM
joneum edited the summary of this revision. (Show Details)
joneum added reviewers: lifanov, dmgk.
joneum edited the test plan for this revision. (Show Details)
lwhsu added a subscriber: lwhsu.Tue, Sep 17, 6:10 PM
joneum updated this revision to Diff 62266.Wed, Sep 18, 12:23 PM

Update Makefile, Reported by dmgk