-
Notifications
You must be signed in to change notification settings - Fork 29
Add version override #260
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
base: main
Are you sure you want to change the base?
Add version override #260
Conversation
4a69ce1 to
bb2916c
Compare
| FROM src-base AS metadata-builder | ||
| ARG PKG_REPO | ||
| ARG PKG_REF | ||
| ARG PKG_VERSION |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you also need to add it to builder-deb, builder-rpm and builder-static stages. Same for other dockerfiles
pkg/compose/Dockerfile
Outdated
| ARG PKG_NAME="docker-compose-plugin" | ||
| ARG PKG_REPO="https://github.com/docker/compose.git" | ||
| ARG PKG_REF="main" | ||
| ARG PKG_VERSION |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not needed to put it there.
Also need to add it to metadata, builder-deb, builder-rpm and builder-static stages.
1d3365c to
9bb37e0
Compare
Allow manual version setting when building. Useful for pre-release testing builds when there's no tag yet. Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
9bb37e0 to
d41fab7
Compare
crazy-max
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs rebase
Allow manual version setting when building. Useful for pre-release testing builds when there's no tag yet.