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

podman build: --build-arg: does not fall back to environment #9571

Closed
edsantiago opened this issue Mar 2, 2021 · 2 comments
Closed

podman build: --build-arg: does not fall back to environment #9571

edsantiago opened this issue Mar 2, 2021 · 2 comments
Assignees
Labels
locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments.

Comments

@edsantiago
Copy link
Member

Difference between buildah bud and podman build: buildah defaults a build-arg to an envariable if it's set; podman does not:

$ export foo=bar
$ printf "FROM quay.io/libpod/alpine:latest\nARG foo\n" | ./bin/podman build --quiet --build-arg foo -
WARN[0000] missing "foo" build argument. Try adding "--build-arg foo=<VALUE>" to the command line
41187bb4760c7a6c46b3cf4c7ca2595c7432cf87c06c131fadb6fc9e509136c2

buildah, if it sees --build-arg foo without an explicit =something, will interpolate the environment variable $foo.

This may be a bug, or may be an intentional difference, or it may be a nothing-we-can-do-about-it difference. If it can't be fixed, perhaps it should be documented.

@rhatdan
Copy link
Member

rhatdan commented Mar 2, 2021

Give me a test for this one, as well, and I will give you a fix.

@rhatdan rhatdan self-assigned this Mar 2, 2021
@edsantiago
Copy link
Member Author

build-arg.diff.txt

Reconciling all these diffs is left as an exercise for the reader

rhatdan added a commit to rhatdan/podman that referenced this issue Mar 3, 2021
Fixes: containers#9571

Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
rhatdan added a commit to rhatdan/podman that referenced this issue Mar 3, 2021
Fixes: containers#9571

Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
rhatdan added a commit to rhatdan/podman that referenced this issue Mar 3, 2021
Fixes: containers#9571

Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
rhatdan added a commit to rhatdan/podman that referenced this issue Mar 4, 2021
Fixes: containers#9571

Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
rhatdan added a commit to rhatdan/podman that referenced this issue Mar 4, 2021
Fixes: containers#9571

Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
rhatdan added a commit to rhatdan/podman that referenced this issue Mar 5, 2021
Fixes: containers#9571

Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
edsantiago pushed a commit to edsantiago/libpod that referenced this issue Mar 8, 2021
Fixes: containers#9571

Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
vrothberg pushed a commit to vrothberg/libpod that referenced this issue Mar 9, 2021
Fixes: containers#9571

Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
@github-actions github-actions bot added the locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. label Sep 22, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 22, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments.
Projects
None yet
Development

No branches or pull requests

2 participants