Skip to content

Enhancement: add osversion.Build() utility - #569

Merged
John Howard (lowenna) merged 1 commit into
microsoft:masterfrom
thaJeztah:add_build_function
Apr 19, 2019
Merged

Enhancement: add osversion.Build() utility#569
John Howard (lowenna) merged 1 commit into
microsoft:masterfrom
thaJeztah:add_build_function

Conversation

@thaJeztah

Copy link
Copy Markdown
Contributor

Comment thread osversion/osversion.go

// Build gets the build-number on Windows
// The calling application must be manifested to get the correct version information.
func Build() uint16 {

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

FWIW; I was in doubt wether to name this Build() or BuildNumber()

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.

@lowenna John Howard (lowenna) 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.

LGTM

@lowenna

Copy link
Copy Markdown
Contributor

Sebastiaan van Stijn (@thaJeztah) Needs a rebase

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

Copy link
Copy Markdown
Contributor Author

rebased 👍

@lowenna John Howard (lowenna) 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.

LGTM

@lowenna
John Howard (lowenna) merged commit 2226e08 into microsoft:master Apr 19, 2019
@thaJeztah
Sebastiaan van Stijn (thaJeztah) deleted the add_build_function branch April 19, 2019 21:41
Sebastiaan van Stijn (thaJeztah) added a commit to thaJeztah/cli that referenced this pull request Oct 26, 2019
full diff: microsoft/hcsshim@672e52e...2226e08

- microsoft/hcsshim#569 Enhancement: add osversion.Build() utility
    - relates to moby/moby#39100 Use Microsoft/hcsshim constants and deprecate pkg/system.GetOsVersion()

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Jenkins (docker-jenkins) pushed a commit to docker-archive/docker-ce that referenced this pull request Oct 28, 2019
full diff: microsoft/hcsshim@672e52e...2226e08

- microsoft/hcsshim#569 Enhancement: add osversion.Build() utility
    - relates to moby/moby#39100 Use Microsoft/hcsshim constants and deprecate pkg/system.GetOsVersion()

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Upstream-commit: 5e4c7eba448246e90f88c0d319a45a62b2e62c33
Component: cli
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