Skip to content

[release/1.1 backport] Makefile: allow overriding package name#3097

Merged
estesp merged 1 commit intocontainerd:release/1.1from
thaJeztah:1.1_backport_override_package_name
Mar 15, 2019
Merged

[release/1.1 backport] Makefile: allow overriding package name#3097
estesp merged 1 commit intocontainerd:release/1.1from
thaJeztah:1.1_backport_override_package_name

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

backport of #3096 for the 1.1 release branch

With this patch applied, the package-name in the --version output can be overridden;

make PACKAGE=containerd.io binaries

./bin/containerd --version
containerd containerd.io v1.2.0-329-ga15b6e20.m a15b6e2097c48b632dbdc63254bad4c62b69e709.m

Signed-off-by: Sebastiaan van Stijn [email protected]
(cherry picked from commit b858cfb)
Signed-off-by: Sebastiaan van Stijn [email protected]

With this patch applied, the package-name in the `--version` output can be overridden;

    make PACKAGE=containerd.io binaries

    ./bin/containerd --version
    containerd containerd.io v1.2.0-329-ga15b6e20.m a15b6e2.m

Signed-off-by: Sebastiaan van Stijn <[email protected]>
(cherry picked from commit b858cfb)
Signed-off-by: Sebastiaan van Stijn <[email protected]>
@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #3097 into release/1.1 will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##           release/1.1    #3097   +/-   ##
============================================
  Coverage        49.07%   49.07%           
============================================
  Files               85       85           
  Lines             7598     7598           
============================================
  Hits              3729     3729           
  Misses            3194     3194           
  Partials           675      675
Flag Coverage Δ
#linux 49.07% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8ecb055...d28f4aa. Read the comment docs.

Copy link
Copy Markdown
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Member

@fuweid fuweid left a comment

Choose a reason for hiding this comment

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

LGTM

@estesp estesp merged commit 9a0a5fd into containerd:release/1.1 Mar 15, 2019
@thaJeztah thaJeztah deleted the 1.1_backport_override_package_name branch March 15, 2019 01:53
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.

4 participants