Your gradle.properties looks good, it shouldn't hit the same issue as
the ASF Jenkins. The gradle.properties of the ASF Jenkins looked like
this before I updated it:
```
# Disable Gradle daemon for all builds.
org.gradle.daemon=false
# Lucene specific settings for lucene2 build nodes
systemProp.file.encoding=UTF-8
org.gradle.jvmargs=-Xmx3g
org.gradle.workers.max=4
tests.jvms=4
tests.multiplier=2
```
And now:
```
# Disable Gradle daemon for all builds.
org.gradle.daemon=false
# Lucene specific settings for lucene2 build nodes
systemProp.file.encoding=UTF-8
org.gradle.jvmargs=-Xmx3g \
--add-exports jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED \
--add-exports jdk.compiler/com.sun.tools.javac.file=ALL-UNNAMED \
--add-exports jdk.compiler/com.sun.tools.javac.parser=ALL-UNNAMED \
--add-exports jdk.compiler/com.sun.tools.javac.tree=ALL-UNNAMED \
--add-exports jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED
org.gradle.workers.max=4
tests.jvms=4
tests.multiplier=2
```
I was seeing a few errors in precommit in early versions of the linked
PR, so I wanted to make sure I was using the most up-to-date versions
of dependencies before attempting any fix. But to my knowledge this
new version doesn't address the failure that we saw on the ASF
Jenkins.
On Thu, Jan 6, 2022 at 11:33 AM Uwe Schindler <uwe@thetaphi.de> wrote:
>
> But Jenkins removes its gradle.properties file from the build directory before each build as it starts with a clean checkout.
>
> So it is not understandable to me. It recreated the file: https://jenkins.thetaphi.de/job/Lucene-main-Linux/ws/gradle.properties/*view*/
> See also timestamp: https://jenkins.thetaphi.de/job/Lucene-main-Linux/ws/
>
> I have seen that you updated the format in the PR.
>
> Uwe
>
> -----
> Uwe Schindler
> Achterdiek 19, D-28357 Bremen
> https://www.thetaphi.de
> eMail: uwe@thetaphi.de
>
> > -----Original Message-----
> > From: Adrien Grand <jpountz@gmail.com>
> > Sent: Thursday, January 6, 2022 11:28 AM
> > To: Lucene Dev <dev@lucene.apache.org>
> > Subject: Re: Heads up: main branch will soon require Java 17
> >
> > To clarify, merging the PR wouldn't address the test failures.
> >
> > On Thu, Jan 6, 2022 at 11:27 AM Adrien Grand <jpountz@gmail.com> wrote:
> > >
> > > This seems to be due to the fact that it's using an old
> > > gradle.properties file that doesn't have the module exports that Dawid
> > > added as part of LUCENE-10066.
> > > https://github.com/apache/lucene/pull/259/files#diff-
> > fbdfae634c6898f4a415bc2a278ff771c38436d70b0e64250ab068b4b3e58426R
> > 48-R52
> > >
> > > Let me look into whether I can make the ASF Jenkins recreate its
> > > gradle.properties.
> >
> >
> >
> > --
> > Adrien
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
> > For additional commands, e-mail: dev-help@lucene.apache.org
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
> For additional commands, e-mail: dev-help@lucene.apache.org
>
--
Adrien
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org