Skip to content
This repository has been archived by the owner on Oct 22, 2024. It is now read-only.

Dockerfile: Redefine SWUPD_UPDATE_ARG in all build stages. #385

Merged
merged 1 commit into from
Aug 16, 2019

Conversation

avalluri
Copy link
Contributor

An ARG instruction goes out of scope at the end of the build stage where it was
defined. To use an arg in multiple stages, each stage must include the ARG
instruction.

Refer to:
https://github.com/docker/cli/blob/master/docs/reference/builder.md#understand-how-arg-and-from-interact

FIXES: #382

An ARG instruction goes out of scope at the end of the build stage where it was
defined. To use an arg in multiple stages, each stage must include the ARG
instruction.

Refer to:
https://github.com/docker/cli/blob/master/docs/reference/builder.md#understand-how-arg-and-from-interact

FIXES: intel#382
@avalluri avalluri requested a review from mythi August 13, 2019 12:41
Copy link

@mythi mythi left a comment

Choose a reason for hiding this comment

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

Did update-clear-linux-base.sh also work as expected with this change?

@avalluri
Copy link
Contributor Author

Did update-clear-linux-base.sh also work as expected with this change?

Yes. This script uses = suffix(ARG SWUPD_UPDATE_ARG=) in regex express.

@avalluri avalluri merged commit 60e33f1 into intel:devel Aug 16, 2019
@mythi
Copy link

mythi commented Aug 16, 2019

@avalluri how about updating release-0.5 branch too? update-clear-linux-base.sh run is also needed but that should probably wait until the swupd error 255 is fixed.

@pohly
Copy link
Contributor

pohly commented Aug 26, 2019 via email

@avalluri
Copy link
Contributor Author

Yes, this needs to be backported. @avalluri: are you tracking that?

@pohly No, I was not. Do we need a new release with this fix?

@pohly
Copy link
Contributor

pohly commented Aug 26, 2019 via email

@avalluri
Copy link
Contributor Author

Done. New release v0.5.2 with change is published.

@avalluri avalluri deleted the issue-382 branch October 22, 2019 13:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

release images are not reproducible
3 participants