Skip to content

Don't dirty the tree when building in CI#2175

Merged
sijie merged 1 commit intoapache:masterfrom
ivankelly:git-sha-dirty
Jul 17, 2018
Merged

Don't dirty the tree when building in CI#2175
sijie merged 1 commit intoapache:masterfrom
ivankelly:git-sha-dirty

Conversation

@ivankelly
Copy link
Copy Markdown
Contributor

@ivankelly ivankelly commented Jul 16, 2018

When we build a package, mvn inserts the sha of the commit we are
building from into the tarball. If there have been any changes to the
work tree, it will add the suffix "(dirty)".

CI makes a couple of modifications to the worktree. It creates its own
.repository for maven artifacts. And it create a docker debugging
logfile.

This patch adds these to .gitignore so that they won't be picked up by
git status and won't cause the tree to be 'dirty'.

When we build a package, mvn inserts the sha of the commit we are
building from into the tarball. If there have been any changes to the
work tree, it will add the suffix "(dirty)".

CI makes a couple of modifications to the worktree. It creates its own
.repository for maven artifacts. And it create a docker debugging
logfile.

This patch adds these to .gitignore so that they won't be picked up by
git status and won't cause the tree to be 'dirty'.
@ivankelly ivankelly changed the title [WIP] figuring out what's happening to the sha in CI Don't dirty the tree when building in CI Jul 17, 2018
@ivankelly ivankelly requested review from jiazhai and sijie July 17, 2018 09:24
@ivankelly ivankelly self-assigned this Jul 17, 2018
@sijie
Copy link
Copy Markdown
Member

sijie commented Jul 17, 2018

retest this please

@sijie sijie added type/enhancement The enhancements for the existing features or docs. e.g. reduce memory usage of the delayed messages area/build labels Jul 17, 2018
@sijie sijie added this to the 2.1.0-incubating milestone Jul 17, 2018
@sijie sijie merged commit 3dfc217 into apache:master Jul 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/build type/enhancement The enhancements for the existing features or docs. e.g. reduce memory usage of the delayed messages

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants