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

Disable cgo-requiring bits when cgo is not enabled #410

Merged
merged 1 commit into from
Aug 6, 2019

Conversation

nalind
Copy link
Member

@nalind nalind commented Aug 5, 2019

Adjust build tags in drivers and pkg so that builds with CGO_ENABLED=0 won't fail outright. This ends up disabling btrfs (which needs kernel headers), ostree (which uses libostree), overlayfs (which uses C headers to define fs_disk_quota_t), and devicemapper (which uses libdevmapper and loopback) by default.

Adjust build tags in drivers and pkg so that builds with CGO_ENABLED=0
won't fail outright.  This ends up disabling btrfs (which uses kernel
headers), ostree (which uses libostree), overlayfs (which uses C headers
to define fs_disk_quota_t), and devicemapper (which uses libdevmapper
and loopback) by default.

Signed-off-by: Nalin Dahyabhai <nalin@redhat.com>
@rhatdan
Copy link
Member

rhatdan commented Aug 5, 2019

LGTM, although I would love to get overlay to work without cgo.

@rhatdan
Copy link
Member

rhatdan commented Aug 5, 2019

Copy link
Member

@vrothberg vrothberg left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@TomSweeneyRedHat TomSweeneyRedHat left a comment

Choose a reason for hiding this comment

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

LGTM

@rhatdan rhatdan merged commit 161c94f into containers:master Aug 6, 2019
@nalind nalind deleted the no-cgo branch August 6, 2019 14:09
@nalind
Copy link
Member Author

nalind commented Aug 6, 2019

This should help with containers/image#675 (comment).

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