Skip to content

.dockerignore, internal/build: Read git information directly from file#15458

Merged
karalabe merged 2 commits into
ethereum:masterfrom
asasmoyo:issue/15346
Nov 12, 2017
Merged

.dockerignore, internal/build: Read git information directly from file#15458
karalabe merged 2 commits into
ethereum:masterfrom
asasmoyo:issue/15346

Conversation

@asasmoyo
Copy link
Copy Markdown
Contributor

This commit changes the way of retrieving git commit and branch for build environment from running git command to reading git files directly.

This commit also adds required git files into Docker build context.

fixes: #15346

@GitCop
Copy link
Copy Markdown

GitCop commented Nov 11, 2017

Thank you for your contribution! Your commits seem to not adhere to the repository coding standards

  • Commit: 0c331ba3755b0812c782e2294bb22e56d49a00a0
  • Your commit message body contains a line that is longer than 80 characters

Please check the contribution guidelines for more details.


This message was auto-generated by https://gitcop.com

This commit changes the way of retrieving git commit and branch for build
environment from running git command to reading git files directly.

This commit also adds required git files into Docker build context.

fixes: ethereum#15346
Copy link
Copy Markdown
Member

@karalabe karalabe left a comment

Choose a reason for hiding this comment

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

LGTM, lets hope it doesn't break anything.

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.

Commit hash missing from Geth version in Docker builds

3 participants