Skip to content

[master] Add build target for ubuntu-eoan - #412

Merged
docker-unir[bot] merged 1 commit into
docker:masterfrom
arkodg:build-ubuntu-eoan
Dec 11, 2019
Merged

[master] Add build target for ubuntu-eoan#412
docker-unir[bot] merged 1 commit into
docker:masterfrom
arkodg:build-ubuntu-eoan

Conversation

@arkodg

Copy link
Copy Markdown

Signed-off-by: Arko Dasgupta arko.dasgupta@docker.com

Signed-off-by: Arko Dasgupta <arko.dasgupta@docker.com>
@arkodg

Copy link
Copy Markdown
Author

@thaJeztahthaJeztah left a comment

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.

LGTM

@thaJeztah

Copy link
Copy Markdown
Member

@kolyshkin@seemethere ptal

@seemethereseemethere left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nothing really uses this variable, while adding it here is good the main source of truth of what eventually gets built isn’t in this repository. That should be in release-packaging

@arkodg

Copy link
Copy Markdown
Author

@seemethere thanks for the hint . I'm guessing this PR needs to get merged and another PR similar to https://github.com/docker/release-packaging/pull/318 needs to be raised in release-packaging

@seemethere

Copy link
Copy Markdown
Contributor

I can’t see that PR but yes I’m assuming that’s the correct assumption.

@docker-unir
docker-unirBot merged commit 3463279 into docker:masterDec 11, 2019
@thaJeztahthaJeztah changed the title Add build target for ubuntu-eoan[master] Add build target for ubuntu-eoanMay 1, 2020
Sign up for freeto 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

@arkodg@thaJeztah@seemethere@zelahi