Skip to content

installation: adjust make option - #626

Open
stefanfrede wants to merge 2 commits into
containers:mainfrom
stefanfrede:install/building-from-source
Open

installation: adjust make option#626
stefanfrede wants to merge 2 commits into
containers:mainfrom
stefanfrede:install/building-from-source

Conversation

@stefanfrede

@stefanfrede stefanfrede commented Aug 3, 2026

Copy link
Copy Markdown

Podman's Makefile declares BUILDTAGS ?= as a computed list that includes $(shell hack/systemd_tag.sh), which emits systemd when the libsystemd headers are present.

Setting BUILDTAGS= on the command line replaces that list rather than adding to it, so systemd is dropped.

Concisely describe the change

Use EXTRA_BUILDTAGS to append to the computed defaults.

BUILDTAGS is replacing them.

Before screenshot / screen recording

After screenshot / screen recording

Checklist

Ensure you have completed the following checklist for your pull request to be reviewed:

  • Certify you wrote the patch or otherwise have the right to pass it on as an open-source patch by signing all
    commits. (git commit -s). (If needed, use git commit -s --amend). The author email must match
    the sign-off email address. See CONTRIBUTING.md
    for more information.
  • Referenced issues using Fixes: #00000 in commit message (if applicable)
  • PR description, commit message, and GitHub comments are human-written, per LLM Policy

Comment thread docs/installation.md Outdated
```

Podman supports optional build tags for compiling support of various features.
To add build tags to the make option the `BUILDTAGS` variable must be set, for example:

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.

I think you want

Suggested change
To add build tags to the make option the `BUILDTAGS` variable must be set, for example:
To add build tags while keeping auto-detected defaults, set EXTRA_BUILDTAGS, for example:

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I added the change.

Comment thread docs/installation.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.

You need EXTRA_BUILDTAGS here too

Suggested change
make EXTRA_BUILDTAGS="btrfs_noversion exclude_graphdriver_btrfs"

Please search for other instances.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I added the change and couldn't find more occurances in the document.

Comment thread docs/installation.md Outdated
git clone https://github.com/opencontainers/runc.git $GOPATH/src/github.com/opencontainers/runc
cd $GOPATH/src/github.com/opencontainers/runc
make BUILDTAGS="selinux seccomp"
make EXTRA_BUILDTAGS="selinux seccomp"

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.

Suggested change
make EXTRA_BUILDTAGS="selinux seccomp"
make RUNC_BUILDTAGS="selinux"

runc default already includes seccomp

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I added the change.

@antedotee

Copy link
Copy Markdown
Contributor

I think the DCO check is failing. Maybe we can look into it😅

Podman's Makefile declares `BUILDTAGS ?=` as a computed list that
includes `$(shell hack/systemd_tag.sh)`, which emits `systemd` when the
libsystemd headers are present.

Setting `BUILDTAGS=` on the command line **replaces** that list rather
than adding to it, so `systemd` is dropped.

Signed-off-by: Stefan Frede <7139+stefanfrede@users.noreply.github.com>
@stefanfrede
stefanfrede force-pushed the install/building-from-source branch from 2ed252a to 43e23a9 Compare August 14, 2026 14:12
Signed-off-by: Stefan Frede <7139+stefanfrede@users.noreply.github.com>
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.

3 participants