-
Notifications
You must be signed in to change notification settings - Fork 89
One of setGitDir or setWorkTree must be called #228
Comments
Yeah, that seems pretty likely. I'll mark this a bug and double-check that. Thanks for reporting. |
I'll make a PR :) |
The Exception seems to come from here: org.ajoberstar.grgit.operation.OpenOp#call |
Maybe there should be a final |
So, this seems to be a grgit issue, not gradle-git, I'll move there :) |
Reopening since I need to depend on the fixed grgit version. |
I'm still encountering the same issue as criztovyl on 11/20/16. I'm confused about what the consensus for resolution was..? |
Sorry for my mess of issues and pr's. |
This version: https://github.com/ajoberstar/gradle-git |
This fix was added in v1.7.2 of grgit (which is a dependency of gradle-git), so it should be fixed in gradle-git 1.7.0 or higher. If you're using classic syntax plugins: buildscript {
// ... repositories ...
dependencies {
classpath 'org.ajoberstar:gradle-git:1.7.0' // or higher
}
} If you're using plugin block: plugins {
id 'org.ajoberstar.grgit' version '1.7.0' // or higher
} Though I recommend getting to the latest version you can. |
@ajoberstar , I bumped my version to |
The grgit 2.x and beyond plugin is hosted in grgit. Could you open an issue over there and include the full stacktrace you're seeing? |
Ok, will do @ajoberstar . Thanks |
I re-rerun with
--stacktrack
, which leads to org.ajoberstar.gradle.git.base.GrgitPlugin#applyI think jgit changed the exception thrown if the no Git repo is found.
Will attach full stacktrace soon.Done.The text was updated successfully, but these errors were encountered: