Skip to content

build: include instructions for runc build#2460

Merged
crosbymichael merged 1 commit intocontainerd:masterfrom
stevvooe:runc-build-instructions
Jul 13, 2018
Merged

build: include instructions for runc build#2460
crosbymichael merged 1 commit intocontainerd:masterfrom
stevvooe:runc-build-instructions

Conversation

@stevvooe
Copy link
Copy Markdown
Member

Including some build information about runc that has been helpful in my
development environment. This should allow one to build and install runc
next to containerd so that both can be worked on simultaneously.

Signed-off-by: Stephen Day [email protected]

@stevvooe stevvooe force-pushed the runc-build-instructions branch from 55e15ea to dd104de Compare July 12, 2018 21:20
Copy link
Copy Markdown
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

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

one minor wording change; looks good

Comment thread BUILDING.md Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

nit: s/recommend/recommended/

Comment thread BUILDING.md Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

nit: sometimes it is necessary

@codecov-io
Copy link
Copy Markdown

codecov-io commented Jul 12, 2018

Codecov Report

Merging #2460 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2460   +/-   ##
=======================================
  Coverage   45.04%   45.04%           
=======================================
  Files          92       92           
  Lines        9424     9424           
=======================================
  Hits         4245     4245           
  Misses       4496     4496           
  Partials      683      683
Flag Coverage Δ
#linux 49.29% <ø> (ø) ⬆️
#windows 41.33% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cb4bf20...a5e0916. Read the comment docs.

Including some build information about runc that has been helpful in my
development environment. This should allow one to build and install runc
next to containerd so that both can be worked on simultaneously.

Signed-off-by: Stephen Day <[email protected]>
@stevvooe stevvooe force-pushed the runc-build-instructions branch from dd104de to a5e0916 Compare July 12, 2018 22:56
@ehazlett
Copy link
Copy Markdown
Member

LGTM

1 similar comment
@crosbymichael
Copy link
Copy Markdown
Member

LGTM

@crosbymichael crosbymichael merged commit 8baeaff into containerd:master Jul 13, 2018
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.

6 participants