sysutils/runj: new port
ClosedPublic
Actions

Authored by dch on Mar 16 2021, 8:26 PM.

Details

Reviewers

dch

Commits

rP568660: sysutils/runj: include phab review feedback
rP568659: sysutils/runj: new port

Summary

runj is an experimental, proof-of-concept OCI-compatible runtime for
FreeBSD jails.

Important: runj is a proof-of-concept and the implementation has not
been evaluated for its security. Do not use runj on a production
system. Do not run workloads inside runj that rely on a secure
configuration. This is a personal project, not backed by the
author's employer.

WWW: https://github.com/samuelkarp/runj

Test Plan

needs poudriere + usage tests still
- waiting on upstream to confirm if they want a port atm

Diff Detail

Repository

rP FreeBSD ports repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

dch created this revision.Mar 16 2021, 8:26 PM

Herald added a subscriber: mat. · View Herald TranscriptMar 16 2021, 8:26 PM

dch requested review of this revision.Mar 16 2021, 8:26 PM

Harbormaster completed remote builds in B37872: Diff 85853.Mar 16 2021, 8:26 PM

tobik added a subscriber: tobik.Mar 17 2021, 6:05 AM

tobik added inline comments.

sysutils/runj/Makefile
18 ↗	(On Diff #85853)	`.include "Makefile.deps"`
20–22 ↗	(On Diff #85853)	The binaries have the same name as the packages and the default install is to `${PREFIX}/bin`. The `:bin/*` bits are redundant. GO_TARGET= ./cmd/runj \ ./cmd/runj-entrypoint \ ./cmd/containerd-shim-runj-v1
28–32 ↗	(On Diff #85853)	Why is this needed? It seems redundant too. USES=go installs the binaries with `INSTALL_PROGRAM` and `go build` is passed `-ldflags=-s`. They should already be stripped by that anyway.
sysutils/runj/pkg-descr
1 ↗	(On Diff #85853)	s/unj/runj/