Skip to content

Set packagename to containerd.io in version, and make it configurable#84

Merged
seemethere merged 1 commit intodocker:masterfrom
thaJeztah:fix_package_name
Apr 9, 2019
Merged

Set packagename to containerd.io in version, and make it configurable#84
seemethere merged 1 commit intodocker:masterfrom
thaJeztah:fix_package_name

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

@thaJeztah thaJeztah commented Mar 14, 2019

depends on

The default is containerd.io

make deb
containerd --version
containerd containerd.io 20190314.103813~b858cfb4 b858cfb41b4f49d93990380faed2af5dd9269ffe

But can be overridden using the PACKAGE variable:

make PACKAGE=containerd.dev deb
containerd --version
containerd containerd.dev 20190314.103813~b858cfb4 b858cfb41b4f49d93990380faed2af5dd9269ffe

Signed-off-by: Sebastiaan van Stijn github@gone.nl

@thaJeztah
Copy link
Copy Markdown
Member Author

@seemethere

seemethere
seemethere previously approved these changes Mar 14, 2019
@thaJeztah
Copy link
Copy Markdown
Member Author

I opened a backport PR for the 1.2 release branch, so once that's merged, we can merge this one for the next (1.2.6) release containerd/containerd#3098

The default is containerd.io

    make deb
    containerd --version
    containerd containerd.io 20190314.103813~b858cfb4 b858cfb41b4f49d93990380faed2af5dd9269ffe

But can be overridden using the PACKAGE variable:

    make PACKAGE=containerd.dev deb
    containerd --version
    containerd containerd.dev 20190314.103813~b858cfb4 b858cfb41b4f49d93990380faed2af5dd9269ffe

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah changed the title [WIP] Set packagename to containerd.io in version, and make it configurable Set packagename to containerd.io in version, and make it configurable Apr 9, 2019
@thaJeztah
Copy link
Copy Markdown
Member Author

removing "WIP" from this one; this should now work 👍

@seemethere seemethere merged commit 2466bd8 into docker:master Apr 9, 2019
@thaJeztah thaJeztah deleted the fix_package_name branch April 9, 2019 22:22
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.

2 participants