Skip to content

fix build on OpenBSD by defining Self() - #37301

Merged
thaJeztah merged 1 commit into
moby:masterfrom
Mischi:fix-define-self
Jun 25, 2018
Merged

thaJeztah merged 1 commit into
moby:masterfrom
Mischi:fix-define-self

Conversation

@Mischi

@Mischi Mischi commented Jun 16, 2018

Copy link
Copy Markdown
Contributor

- What I did
While building all tools from https://github.com/ethereum/go-ethereum on OpenBSD, I got an error because Self was not defined in reexec/command_unsupported.go.

- How I did it
This PR defines the method.

- How to verify it
It builds now under OpenBSD. I did not do any runtime testing though.

- Description for the changelog

fix the build on OpenBSD.

@GordonTheTurtle GordonTheTurtle added dco/no Automatically set by a bot when one of the commits lacks proper signature status/0-triage labels Jun 16, 2018
@thaJeztah

Copy link
Copy Markdown
Member

Thanks for contributing! Can you sign off your commit (you can use the instructions that @GordonTheTurtle provided above) otherwise CI won't pass 😅

Signed-off-by: Fabian Raetz <fabian.raetz@gmail.com>
@GordonTheTurtle GordonTheTurtle removed the dco/no Automatically set by a bot when one of the commits lacks proper signature label Jun 16, 2018
@moby moby deleted a comment from GordonTheTurtle Jun 18, 2018

@vdemeester vdemeester 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 thaJeztah 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

@codecov

codecov Bot commented Jun 25, 2018 •

Copy link
Copy Markdown

Codecov Report

Merging #37301 into master will increase coverage by 0.21%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #37301      +/-   ##
==========================================
+ Coverage   34.84%   35.05%   +0.21%     
==========================================
  Files         610      609       -1     
  Lines       45006    45000       -6     
==========================================
+ Hits        15681    15774      +93     
+ Misses      27210    27112      -98     
+ Partials     2115     2114       -1

@thaJeztah
thaJeztah merged commit cb37987 into moby:master Jun 25, 2018
@Mischi

Mischi commented Jun 25, 2018

Copy link
Copy Markdown
Contributor Author

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants