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

BUILDING.md: add osusergo for static build #2476

Merged
merged 1 commit into from
Jul 19, 2018

Conversation

kolyshkin
Copy link
Contributor

Go 1.11 includes a fix to os/user to be working in a static binary
(fixing golang/go#23265). The fix requires
osusergo build tag to be set for static binaries, which is what
this commit documents.

@codecov-io
Copy link

codecov-io commented Jul 19, 2018

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2476   +/-   ##
=======================================
  Coverage   44.73%   44.73%           
=======================================
  Files          93       93           
  Lines        9490     9490           
=======================================
  Hits         4245     4245           
  Misses       4562     4562           
  Partials      683      683
Flag Coverage Δ
#linux 48.93% <ø> (ø) ⬆️
#windows 41.03% <ø> (ø) ⬆️

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 94cfce6...48570b3. Read the comment docs.

Go 1.11 includes a fix to os/user to be working in a static binary
(fixing golang/go#23265). The fix requires
`osusergo` build tag to be set for static binaries, which is what
this commit documents.

[v2: sort tags alphabetically]

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
@kolyshkin
Copy link
Contributor Author

Also related: moby/moby#37500.

@crosbymichael
Copy link
Member

LGTM

1 similar comment
@dmcgowan
Copy link
Member

LGTM

@dmcgowan dmcgowan merged commit 77ffa36 into containerd:master Jul 19, 2018
@kolyshkin kolyshkin deleted the osusergo branch August 14, 2018 16:26
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