Skip to content

Conversation

@TheSnoozer
Copy link
Collaborator

As discussed in #149
Using "...-dirty" as default dirty-marker.

@ktoso
Copy link
Collaborator

ktoso commented Dec 18, 2014

Almost :-)
Would you mind to make the tests pass on travis?

Current failure is

java.lang.RuntimeException: Failed to execute git command (`git describe --dirty=-dirty --match=v2.1.8 --abbrev=7` @ /home/travis/build/ktoso/maven-git-commit-id-plugin/target/sandbox/my-pom-project/my-jar-module)!
    at pl.project13.maven.git.NativeGitProvider.runGitCommand(NativeGitProvider.java:277)
    at pl.project13.maven.git.NativeGitProvider.tryToRunGitCommand(NativeGitProvider.java:242)
    at pl.project13.maven.git.NativeGitProvider.getGitDescribe(NativeGitProvider.java:92)
    at pl.project13.maven.git.GitDataProvider.maybePutGitDescribe(GitDataProvider.java:120)
    at pl.project13.maven.git.GitDataProvider.loadGitData(GitDataProvider.java:88)
    at pl.project13.maven.git.GitCommitIdMojo.loadGitDataWithNativeGit(GitCommitIdMojo.java:508)
    at pl.project13.maven.git.GitCommitIdMojo.loadGitData(GitCommitIdMojo.java:491)
    at pl.project13.maven.git.GitCommitIdMojo.execute(GitCommitIdMojo.java:351)
    at pl.project13.maven.git.GitCommitIdMojoIntegrationTest.runGitDescribeWithMatchOption(GitCommitIdMojoIntegrationTest.java:649)
    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
    at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
    at java.lang.reflect.Method.invoke(Method.java:606)

@TheSnoozer
Copy link
Collaborator Author

Sure.
Somehow the travis for the maven-git-commit-id-plugin itself is fail with the same exception....

https://travis-ci.org/ktoso/maven-git-commit-id-plugin/builds

I will review this ASAP (propably tomorrow).
Mighty be again some troubles with travis' git shallow copy...

failing since:
44bc422

@ktoso
Copy link
Collaborator

ktoso commented Dec 19, 2014

Ouch, thanks for noticing. I'll be able to look into it on sunday :)

@TheSnoozer
Copy link
Collaborator Author

@ktoso,
travis works for know.
Not sure what this original exception came from, locally had no problem with it.
It might be that travis makes only a shallow copy of the repository or something...
When it comes back at some time, there should somehow a better fix instead of just using some other commits.

@ktoso ktoso merged commit 7efa3fc into git-commit-id:master Mar 10, 2015
ktoso pushed a commit that referenced this pull request Mar 10, 2015
@ktoso
Copy link
Collaborator

ktoso commented Mar 10, 2015

Merged this in manually.

@ktoso ktoso added this to the 2.1.13 milestone Mar 10, 2015
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