Fix compatibility with Gradle `8.1` and Configuration Cache about gradle-git-versioning-plugin

[gmazzo]

@gmazzo maybe you can provide a simple project to reproduce the error?

Here is it:
https://github.com/gmazzo/gradle-git-versioning-plugin-cc-issue

Run ./gradlew --stop && ./gradlew build to get the issue.

Here is a scan of it also:
https://scans.gradle.com/s/bs5gyy43tlebc

[qoomon]

Hey, I got corona :-/ so probably I'll look a this next week.

[aSemy]

I hope you feel better soon @qoomon!

I think what might work is putting every JGit calls inside a ValueSource, to be computed lazily. That should allow Gradle to fingerprint the property.

[qoomon]

Unfortunately I am not able to reproduce the issue :-/

I run ./gradlew --configuration-cache version multiple times, without any error

Gradle Version: 8.1.1
Plugin Version: 6.4.2

[qoomon]

@gmazzo thanks a lot that works or actually it doesn't :-D

[gmazzo]

I don't know how to reset it after it work once.

Kill the daemon, that worked to me every time: ./gradlew --stop && ./gradlew xxx

[gmazzo]

Researching a bit deeper, it seems this is actually caused by JGIT implementation, I am right? I thought JGIT was a pure java implementation, why is running the host git command? 🤔

[gmazzo]

I'd say this issue is caused by JGit and every plugin that relies on it will face the same. It seems it's accessing .gitconfig on ins initialization by a couple of git commands and that is causing to Gradle's Configuration Cache to fail:

[gmazzo]

Yeah no hurries. I was talking a look and it seems there is no way no hook on jgit to delegate the process creation to Gradle's exec.

So one option will be to ask for it, or open a PR, but probably is going to be a long process.

Another one will be to check if you can not depend on it at all, and just make raw git commands with exec.

[gmazzo]

Felling better right? Sorry I missed that comment 😅

Unfortunately I am not able to reproduce the issue :-/

That's strange 🤔, maybe it's not done all the time? I think it's worth wrapping it in a provider anyway, as the GIT case is actually their main example of it

I have too much on my plate right now (between work, personal plugins, and other contributions) but I'll keep it in the agenda for later if no one else can do it.

[qoomon]

@gmazzo maybe you can provide a simple project to reproduce the error?

[qoomon]

It's totally strange to me, it fails the first time i run ./gradlew version the second time it works just as expected.
Even if I remove the ./gradle folder it still works.

[gmazzo]

Same here, I think it should be caused by some static initialization of jgit.

[qoomon]

I don't know how to reset it after it work once.

[qoomon]

even after removing .git and .gradle dir it still works