[jira] [Created] (MPMD-385) Support for --enable-preview

2023-08-07 Thread Gili (Jira)
Gili created MPMD-385:
-

 Summary: Support for --enable-preview
 Key: MPMD-385
 URL: https://issues.apache.org/jira/browse/MPMD-385
 Project: Maven PMD Plugin
  Issue Type: Improvement
  Components: PMD
Affects Versions: 3.21.0
Reporter: Gili


When running PMD against classes compiled using --enable-preview I get:

[WARNING] Error during type resolution of field  in class  
due to: java.lang.UnsupportedClassVersionError: Preview features are not 
enabled for gg/soulbound/optimus/server/Server (class file version 64.65535). 
Try running with '--enable-preview'

It seems that PMD expects us to pass --enable-preview, but the Maven plugin 
does not provide a mechanism for doing so. I tried setting targetJdk to 
"20-preview" but that didn't help either.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [maven] kun-lu20 commented on a diff in pull request #1207: [MNG-7848] Add s390x pipeline to Jenkins CI

2023-08-07 Thread via GitHub


kun-lu20 commented on code in PR #1207:
URL: https://github.com/apache/maven/pull/1207#discussion_r1286254437


##
Jenkinsfile.s390x:
##
@@ -0,0 +1,173 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
+properties([buildDiscarder(logRotator(artifactNumToKeepStr: '5', numToKeepStr: 
env.BRANCH_NAME=='master'?'5':'1'))])
+
+def buildOs = 'linux'
+def buildJdk = '8'
+def buildMvn = '3.8.x'
+def runITsOses = ['linux']
+def runITsJdks = ['11', '17']
+def runITsMvn = '3.8.x'
+def runITscommand = "mvn clean install -Prun-its,embedded -B -U -V" // 
-DmavenDistro=... -Dmaven.test.failure.ignore=true
+def tests
+
+try {
+
+def osNode = jenkinsEnv.labelForOS(buildOs)
+node('s390x') {
+dir('build') {
+stage('Checkout') {
+checkout scm
+}
+
+def WORK_DIR=pwd()
+def MAVEN_GOAL='verify'
+
+stage('Configure deploy') {

Review Comment:
   Sure, thanks for pointing out! Will open a new PR to remove it.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Commented] (MNG-7848) Add s390x pipeline to Jenkins CI

2023-08-07 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MNG-7848?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17751770#comment-17751770
 ] 

ASF GitHub Bot commented on MNG-7848:
-

kun-lu20 commented on code in PR #1207:
URL: https://github.com/apache/maven/pull/1207#discussion_r1286254437


##
Jenkinsfile.s390x:
##
@@ -0,0 +1,173 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
+properties([buildDiscarder(logRotator(artifactNumToKeepStr: '5', numToKeepStr: 
env.BRANCH_NAME=='master'?'5':'1'))])
+
+def buildOs = 'linux'
+def buildJdk = '8'
+def buildMvn = '3.8.x'
+def runITsOses = ['linux']
+def runITsJdks = ['11', '17']
+def runITsMvn = '3.8.x'
+def runITscommand = "mvn clean install -Prun-its,embedded -B -U -V" // 
-DmavenDistro=... -Dmaven.test.failure.ignore=true
+def tests
+
+try {
+
+def osNode = jenkinsEnv.labelForOS(buildOs)
+node('s390x') {
+dir('build') {
+stage('Checkout') {
+checkout scm
+}
+
+def WORK_DIR=pwd()
+def MAVEN_GOAL='verify'
+
+stage('Configure deploy') {

Review Comment:
   Sure, thanks for pointing out! Will open a new PR to remove it.





> Add s390x pipeline to Jenkins CI
> 
>
> Key: MNG-7848
> URL: https://issues.apache.org/jira/browse/MNG-7848
> Project: Maven
>  Issue Type: New Feature
>  Components: Bootstrap  Build, Integration Tests
>Reporter: Kun Lu
>Priority: Major
>  Labels: pull-request-available
>
> Apache INFRA team has installed necessary JDK flavours on all s390x nodes 
> (Please check issue 
> [https://issues.apache.org/jira/projects/INFRA/issues/INFRA-24781]). We’d 
> like to add the s390x pipeline to Jenkins CI by raising a PR to add ` 
> Jenkinsfile.s390x` to the Maven code base.
> Can anyone from Maven team help us review the PR and create the s390x 
> pipeline on Jenkins? Thanks!



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (MNG-7848) Add s390x pipeline to Jenkins CI

2023-08-07 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MNG-7848?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17751765#comment-17751765
 ] 

ASF GitHub Bot commented on MNG-7848:
-

kun-lu20 commented on PR #1213:
URL: https://github.com/apache/maven/pull/1213#issuecomment-1668362161

   Hi @olamy ,
   
   Thanks for pointing out! I'll raise a new PR to address it.




> Add s390x pipeline to Jenkins CI
> 
>
> Key: MNG-7848
> URL: https://issues.apache.org/jira/browse/MNG-7848
> Project: Maven
>  Issue Type: New Feature
>  Components: Bootstrap  Build, Integration Tests
>Reporter: Kun Lu
>Priority: Major
>  Labels: pull-request-available
>
> Apache INFRA team has installed necessary JDK flavours on all s390x nodes 
> (Please check issue 
> [https://issues.apache.org/jira/projects/INFRA/issues/INFRA-24781]). We’d 
> like to add the s390x pipeline to Jenkins CI by raising a PR to add ` 
> Jenkinsfile.s390x` to the Maven code base.
> Can anyone from Maven team help us review the PR and create the s390x 
> pipeline on Jenkins? Thanks!



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [maven] kun-lu20 commented on pull request #1213: [MNG-7848] Change the notification recipients in s390x pipeline

2023-08-07 Thread via GitHub


kun-lu20 commented on PR #1213:
URL: https://github.com/apache/maven/pull/1213#issuecomment-1668362161

   Hi @olamy ,
   
   Thanks for pointing out! I'll raise a new PR to address it.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Commented] (MBUILDCACHE-64) Apply global exclusions to folder names

2023-08-07 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MBUILDCACHE-64?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17751751#comment-17751751
 ] 

ASF GitHub Bot commented on MBUILDCACHE-64:
---

kbuntrock commented on PR #91:
URL: 
https://github.com/apache/maven-build-cache-extension/pull/91#issuecomment-1668279033

   Changed the exclude mechanism to use path matchers. A bit more than a bugfix 
now, feedbacks appreciated if you see any lack, implementation detail you 
disagree with or extra test case to add.




> Apply global exclusions to folder names
> ---
>
> Key: MBUILDCACHE-64
> URL: https://issues.apache.org/jira/browse/MBUILDCACHE-64
> Project: Maven Build Cache Extension
>  Issue Type: Bug
>Affects Versions: 1.0.1
>Reporter: Frank Wagner
>Assignee: Olivier Lamy
>Priority: Major
>  Labels: pull-request-available
> Fix For: 1.1.0
>
>
> It is currently not possible to exclude folders by their name, like 
> {quote}
> 
> 
> node_modules
> dist
> build
> 
> 
> ...
> {quote}
> That's because isFilteredOutSubpath(), 
> [https://github.com/apache/maven-build-cache-extension/blob/master/src/main/java/org/apache/maven/buildcache/checksum/MavenProjectInput.java#L638,]
>  uses startWith on normalized absolute paths.
> That function could be enhanced with an additional criterion like in 
> [https://github.com/apache/maven-build-cache-extension/blob/master/src/main/java/org/apache/maven/buildcache/checksum/MavenProjectInput.java#L510]
> {{filteredOutPaths.stream().anyMatch(it -> 
> it.getFileName().equals(entry.getFileName()))}}
>  
>  



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [maven-build-cache-extension] kbuntrock commented on pull request #91: [MBUILDCACHE-64] Exclusion mechanism bugfix

2023-08-07 Thread via GitHub


kbuntrock commented on PR #91:
URL: 
https://github.com/apache/maven-build-cache-extension/pull/91#issuecomment-1668279033

   Changed the exclude mechanism to use path matchers. A bit more than a bugfix 
now, feedbacks appreciated if you see any lack, implementation detail you 
disagree with or extra test case to add.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Commented] (MBUILDCACHE-64) Apply global exclusions to folder names

2023-08-07 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MBUILDCACHE-64?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17751749#comment-17751749
 ] 

ASF GitHub Bot commented on MBUILDCACHE-64:
---

kbuntrock commented on code in PR #91:
URL: 
https://github.com/apache/maven-build-cache-extension/pull/91#discussion_r1286165121


##
src/main/java/org/apache/maven/buildcache/checksum/MavenProjectInput.java:
##
@@ -108,11 +107,6 @@ public class MavenProjectInput {
  * property name to pass glob value. The glob to be used to list directory 
files in plugins scanning
  */
 private static final String CACHE_INPUT_GLOB_NAME = 
"maven.build.cache.input.glob";
-/**
- * default glob, bbsdk/abfx specific
- */
-public static final String DEFAULT_GLOB = 
"{*.java,*.groovy,*.yaml,*.svcd,*.proto,*assembly.xml,assembly"

Review Comment:
   Was not used except in a UT.



##
src/test/java/org/apache/maven/buildcache/checksum/MavenProjectInputTest.java:
##
@@ -47,52 +47,23 @@
 
 public class MavenProjectInputTest {
 
-private static final String GLOB = "{*-pom.xml}";
+private static final String DEFAULT_GLOB = "*";
 
 @Test
-public void testAddInputsRelativePath() {
-//MavenProjectInput inputs = new MavenProjectInput(config, new 
ArrayList(),
-// Paths.get("src\\test\\resources\\org"), GLOB);
-//ArrayList files = new ArrayList<>();
-//inputs.listDirOrFile("../../resources", inputs.dirGlob, 
files, new HashSet());
-//assertEquals(4, files.size());
-}
-
-@Test
-public void testAddInputsAbsolutePath() {
-//Path baseDirPath = Paths.get("src\\test\\resources\\org");
-//MavenProjectInput inputs = new MavenProjectInput(config, new 
ArrayList(), baseDirPath, GLOB);
-//ArrayList files = new ArrayList<>();
-//Path candidatePath = 
baseDirPath.resolve("../../resources").normalize().toAbsolutePath();
-//inputs.listDirOrFile(candidatePath.toString(), 
inputs.dirGlob, files, new HashSet());
-//assertEquals(4, files.size()); // pom is filtered out by 
hardcoded if
-}
-
-@Test
-public void testGlobInput() {
-//Path baseDirPath = Paths.get("src\\test\\resources");
-//MavenProjectInput inputs = new MavenProjectInput(config, new 
ArrayList(), baseDirPath, GLOB);
-//ArrayList files = new ArrayList<>();
-//inputs.tryAddInputs("*.java", files, new HashSet());
-//assertEquals(0, files.size()); // pom is filtered out by 
hardcoded if
-}
-
-@Test
-public void testGetDirectoryFiles() {
-List directoryFiles = new ArrayList<>();
+public void testCollectFilteredFiles() {

Review Comment:
   Cleaned dead code and updated a test with a wrong assumption. Not the most 
useful test but better than before.



##
src/main/java/org/apache/maven/buildcache/checksum/MavenProjectInput.java:
##
@@ -165,45 +184,94 @@ public MavenProjectInput(
 this.repoSystem = repoSystem;
 this.remoteCache = remoteCache;
 Properties properties = project.getProperties();
-this.dirGlob = properties.getProperty(CACHE_INPUT_GLOB_NAME, 
config.getDefaultGlob());
+this.defaultFilenameGlob = 
properties.getProperty(CACHE_INPUT_GLOB_NAME, config.getDefaultGlob());
 this.processPlugins =
 
Boolean.parseBoolean(properties.getProperty(CACHE_PROCESS_PLUGINS, 
config.isProcessPlugins()));
 this.tmpDir = System.getProperty("java.io.tmpdir");
 
+this.baseDirectoryGlob = baseDirPath.toString().replace("\\", "/") + 
"/";
+
 org.apache.maven.model.Build build = project.getBuild();
-filteredOutPaths = new ArrayList<>(Arrays.asList(
-normalizedPath(build.getDirectory()), // target by default
-normalizedPath(build.getOutputDirectory()),
-normalizedPath(build.getTestOutputDirectory(;
+addToExcludedSection(
+convertToPathMatcherFileSeperator(
+
normalizedPath(build.getDirectory()).toString())
++ GLOB_SX_ALL_SUB_FILES,
+false); // target by default
+addToExcludedSection(
+convertToPathMatcherFileSeperator(
+
normalizedPath(build.getOutputDirectory()).toString())
++ GLOB_SX_ALL_SUB_FILES,
+false); // target/classes by default
+addToExcludedSection(
+convertToPathMatcherFileSeperator(
+
normalizedPath(build.getTestOutputDirectory()).toString())
++ GLOB_SX_ALL_SUB_FILES,
+false); // target/test-classes by default
 
 List excludes = 

[GitHub] [maven-build-cache-extension] kbuntrock commented on a diff in pull request #91: [MBUILDCACHE-64] Exclusion mechanism bugfix

2023-08-07 Thread via GitHub


kbuntrock commented on code in PR #91:
URL: 
https://github.com/apache/maven-build-cache-extension/pull/91#discussion_r1286165121


##
src/main/java/org/apache/maven/buildcache/checksum/MavenProjectInput.java:
##
@@ -108,11 +107,6 @@ public class MavenProjectInput {
  * property name to pass glob value. The glob to be used to list directory 
files in plugins scanning
  */
 private static final String CACHE_INPUT_GLOB_NAME = 
"maven.build.cache.input.glob";
-/**
- * default glob, bbsdk/abfx specific
- */
-public static final String DEFAULT_GLOB = 
"{*.java,*.groovy,*.yaml,*.svcd,*.proto,*assembly.xml,assembly"

Review Comment:
   Was not used except in a UT.



##
src/test/java/org/apache/maven/buildcache/checksum/MavenProjectInputTest.java:
##
@@ -47,52 +47,23 @@
 
 public class MavenProjectInputTest {
 
-private static final String GLOB = "{*-pom.xml}";
+private static final String DEFAULT_GLOB = "*";
 
 @Test
-public void testAddInputsRelativePath() {
-//MavenProjectInput inputs = new MavenProjectInput(config, new 
ArrayList(),
-// Paths.get("src\\test\\resources\\org"), GLOB);
-//ArrayList files = new ArrayList<>();
-//inputs.listDirOrFile("../../resources", inputs.dirGlob, 
files, new HashSet());
-//assertEquals(4, files.size());
-}
-
-@Test
-public void testAddInputsAbsolutePath() {
-//Path baseDirPath = Paths.get("src\\test\\resources\\org");
-//MavenProjectInput inputs = new MavenProjectInput(config, new 
ArrayList(), baseDirPath, GLOB);
-//ArrayList files = new ArrayList<>();
-//Path candidatePath = 
baseDirPath.resolve("../../resources").normalize().toAbsolutePath();
-//inputs.listDirOrFile(candidatePath.toString(), 
inputs.dirGlob, files, new HashSet());
-//assertEquals(4, files.size()); // pom is filtered out by 
hardcoded if
-}
-
-@Test
-public void testGlobInput() {
-//Path baseDirPath = Paths.get("src\\test\\resources");
-//MavenProjectInput inputs = new MavenProjectInput(config, new 
ArrayList(), baseDirPath, GLOB);
-//ArrayList files = new ArrayList<>();
-//inputs.tryAddInputs("*.java", files, new HashSet());
-//assertEquals(0, files.size()); // pom is filtered out by 
hardcoded if
-}
-
-@Test
-public void testGetDirectoryFiles() {
-List directoryFiles = new ArrayList<>();
+public void testCollectFilteredFiles() {

Review Comment:
   Cleaned dead code and updated a test with a wrong assumption. Not the most 
useful test but better than before.



##
src/main/java/org/apache/maven/buildcache/checksum/MavenProjectInput.java:
##
@@ -165,45 +184,94 @@ public MavenProjectInput(
 this.repoSystem = repoSystem;
 this.remoteCache = remoteCache;
 Properties properties = project.getProperties();
-this.dirGlob = properties.getProperty(CACHE_INPUT_GLOB_NAME, 
config.getDefaultGlob());
+this.defaultFilenameGlob = 
properties.getProperty(CACHE_INPUT_GLOB_NAME, config.getDefaultGlob());
 this.processPlugins =
 
Boolean.parseBoolean(properties.getProperty(CACHE_PROCESS_PLUGINS, 
config.isProcessPlugins()));
 this.tmpDir = System.getProperty("java.io.tmpdir");
 
+this.baseDirectoryGlob = baseDirPath.toString().replace("\\", "/") + 
"/";
+
 org.apache.maven.model.Build build = project.getBuild();
-filteredOutPaths = new ArrayList<>(Arrays.asList(
-normalizedPath(build.getDirectory()), // target by default
-normalizedPath(build.getOutputDirectory()),
-normalizedPath(build.getTestOutputDirectory(;
+addToExcludedSection(
+convertToPathMatcherFileSeperator(
+
normalizedPath(build.getDirectory()).toString())
++ GLOB_SX_ALL_SUB_FILES,
+false); // target by default
+addToExcludedSection(
+convertToPathMatcherFileSeperator(
+
normalizedPath(build.getOutputDirectory()).toString())
++ GLOB_SX_ALL_SUB_FILES,
+false); // target/classes by default
+addToExcludedSection(
+convertToPathMatcherFileSeperator(
+
normalizedPath(build.getTestOutputDirectory()).toString())
++ GLOB_SX_ALL_SUB_FILES,
+false); // target/test-classes by default
 
 List excludes = config.getGlobalExcludePaths();
 for (Exclude excludePath : excludes) {
-filteredOutPaths.add(Paths.get(excludePath.getValue()));
+addToExcludedSection(excludePath.getValue(), true);
 }
 
 for (String 

[jira] [Commented] (MJAR-275) outputTimestamp not applied to module-info; breaks reproducible builds

2023-08-07 Thread Herve Boutemy (Jira)


[ 
https://issues.apache.org/jira/browse/MJAR-275?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17751541#comment-17751541
 ] 

Herve Boutemy commented on MJAR-275:


MJAR-295 created to track this JDK patch version in module-info.class, even if 
not sure that the. workaround is best done in m-jar-p. or m-compiler-p

> outputTimestamp not applied to module-info; breaks reproducible builds
> --
>
> Key: MJAR-275
> URL: https://issues.apache.org/jira/browse/MJAR-275
> Project: Maven JAR Plugin
>  Issue Type: Bug
>Affects Versions: 3.2.0
> Environment: Mac OS X 10.14.6
> JDK 15 (build 15+36)
> JDK 11 (build 11.0.8+10)
>Reporter: Anand Beh
>Assignee: Slawomir Jaranowski
>Priority: Minor
> Fix For: 3.3.0
>
> Attachments: MCOMPILER-439.zip, Screenshot 2020-10-25 at 2.35.59 
> PM.png
>
>
> Setting {{project.build.outputTimestamp}} to a fixed value allows creating 
> reproducible builds per this guide: 
> [https://maven.apache.org/guides/mini/guide-reproducible-builds.html 
> |https://maven.apache.org/guides/mini/guide-reproducible-builds.html]However, 
> if one adds a module-info file to the project, reproducible builds break.
> This is caused by module-info.class using the latest timestamp and not 
> {{project.build.outputTimestamp}}. I was able to identify the problem using 
> diffoscope: [https://diffoscope.org/.|https://diffoscope.org/] With it I 
> determined the timestamp across 2 builds was constant for all but the 
> module-info.class:
>  
> {code:java}
>   -rw 2.0 fat  862 bl defN 20-Oct-17 00:40 
> space/arim/libertybans/api/select/SelectionOrder.class
> │  -rw 2.0 fat 1113 bl defN 20-Oct-17 00:40 
> space/arim/libertybans/api/select/SelectionOrderBuilder.class
> │  -rw 2.0 fat 2285 bl defN 20-Oct-17 00:40 
> META-INF/maven/space.arim.libertybans/bans-api/pom.xml
> │  -rw 2.0 fat   74 bl defN 20-Oct-17 00:40 
> META-INF/maven/space.arim.libertybans/bans-api/pom.properties
> │ --rw 2.0 fat  557 bl defN 20-Oct-25 12:39 module-info.class
> │ +-rw 2.0 fat  557 bl defN 20-Oct-25 12:41 module-info.class
> {code}
>  
> Note the + and - which are diffoscope's way of indicating the difference 
> between the .jar files. Here the {{project.build.outputTimestamp}} is on 17 
> October. As shown, module-info has a "rebellious" timestamp.
>  
> *EDIT:*
> Example project to reproduce the bug:
> [https://github.com/A248/MJAR-275|https://github.com/A248/MCOMPILER-439] 
> (Renamed from [https://github.com/A248/MCOMPILER-439])
> Source code is also provided as an attachment below



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Created] (MJAR-295) clean JDK patch version in module-info.class

2023-08-07 Thread Herve Boutemy (Jira)
Herve Boutemy created MJAR-295:
--

 Summary: clean JDK patch version in module-info.class
 Key: MJAR-295
 URL: https://issues.apache.org/jira/browse/MJAR-295
 Project: Maven JAR Plugin
  Issue Type: Improvement
Affects Versions: 3.3.0
Reporter: Herve Boutemy


as seen in MJAR-275, JDK patch version in module-info.class is not included 
only when building with a newer JDK release using "--release" flag, see 
https://github.com/jvm-repo-rebuild/module-info

we need an issue in JDK to get a proper fix

and waiting for. that, we need a workaround to drop the JDK patch version in 
other cases: we'll need to define were is the best place -- 
m-jar-p?m-compiler-p? other?



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [maven-build-cache-extension] dependabot[bot] opened a new pull request, #95: Bump ch.qos.logback:logback-classic from 1.3.7 to 1.3.9

2023-08-07 Thread via GitHub


dependabot[bot] opened a new pull request, #95:
URL: https://github.com/apache/maven-build-cache-extension/pull/95

   Bumps [ch.qos.logback:logback-classic](https://github.com/qos-ch/logback) 
from 1.3.7 to 1.3.9.
   
   Commits
   
   https://github.com/qos-ch/logback/commit/73f99846f27b4093bb285bedb1e578fd45771ff2;>73f9984
 prepare release 1.3.9
   https://github.com/qos-ch/logback/commit/9499401a0dfced4c9b70ce6af493173f00e1a66f;>9499401
 fix TimeBasedRollingWithConfigFileTest.timeAndSize
   https://github.com/qos-ch/logback/commit/54c23fe2591bbb921bcafb736009984b4d4731ce;>54c23fe
 Merge branch 'branch_1.3.x' of github.com:qos-ch/logback into branch_1.3.x
   https://github.com/qos-ch/logback/commit/fbe45c6eb5d08d0bc662aee17bde0f1301a1bf12;>fbe45c6
 LOGABCK-1717 configuirators shipping with logback-classic are instantiated 
vi...
   https://github.com/qos-ch/logback/commit/e69ed4da27f829e6cae07995196a0bd8a3d3e401;>e69ed4d
 fix typo in property name
   https://github.com/qos-ch/logback/commit/45c140d795f7a456b36771fd6c29d2466ccbf078;>45c140d
 enhance provides directive in module-info
   https://github.com/qos-ch/logback/commit/e2f6ef59e0ef0cbd17dd5b321401a9378df578d5;>e2f6ef5
 ongoing work on GraalVM support
   https://github.com/qos-ch/logback/commit/0d16784f5f659f638cd9cea615dc2fe98f2733cb;>0d16784
 Change LoggerContet to use MDCAdapter instead of LogbackMDCAdapter , partial 
...
   https://github.com/qos-ch/logback/commit/950cbdfeaa37479891fe7fc24beb213faee8d061;>950cbdf
 [LOGBACK-1754] fixed use of CountDownLatch
   https://github.com/qos-ch/logback/commit/42e0355d1c2e95e3f03b98ed6ffe7594ccd8c74f;>42e0355
 Change LoggerContet to use MDCAdapter instead of LogbackMDCAdapter , partial 
...
   Additional commits viewable in https://github.com/qos-ch/logback/compare/v_1.3.7...v_1.3.9;>compare 
view
   
   
   
   
   
   [![Dependabot compatibility 
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=ch.qos.logback:logback-classic=maven=1.3.7=1.3.9)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)
   
   Dependabot will resolve any conflicts with this PR as long as you don't 
alter it yourself. You can also trigger a rebase manually by commenting 
`@dependabot rebase`.
   
   [//]: # (dependabot-automerge-start)
   [//]: # (dependabot-automerge-end)
   
   ---
   
   
   Dependabot commands and options
   
   
   You can trigger Dependabot actions by commenting on this PR:
   - `@dependabot rebase` will rebase this PR
   - `@dependabot recreate` will recreate this PR, overwriting any edits that 
have been made to it
   - `@dependabot merge` will merge this PR after your CI passes on it
   - `@dependabot squash and merge` will squash and merge this PR after your CI 
passes on it
   - `@dependabot cancel merge` will cancel a previously requested merge and 
block automerging
   - `@dependabot reopen` will reopen this PR if it is closed
   - `@dependabot close` will close this PR and stop Dependabot recreating it. 
You can achieve the same result by closing it manually
   - `@dependabot ignore this major version` will close this PR and stop 
Dependabot creating any more for this major version (unless you reopen the PR 
or upgrade to it yourself)
   - `@dependabot ignore this minor version` will close this PR and stop 
Dependabot creating any more for this minor version (unless you reopen the PR 
or upgrade to it yourself)
   - `@dependabot ignore this dependency` will close this PR and stop 
Dependabot creating any more for this dependency (unless you reopen the PR or 
upgrade to it yourself)
   
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Commented] (MRESOLVER-324) Make the BF algorithm as the default option to speed up maven dependency resolution and downloading

2023-08-07 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MRESOLVER-324?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17751484#comment-17751484
 ] 

ASF GitHub Bot commented on MRESOLVER-324:
--

cstamas commented on PR #325:
URL: https://github.com/apache/maven-resolver/pull/325#issuecomment-1667284980

   We need to branch off 1.9.x first, then master made into 2.0.0...




> Make the BF algorithm as the default option to speed up maven dependency 
> resolution and downloading
> ---
>
> Key: MRESOLVER-324
> URL: https://issues.apache.org/jira/browse/MRESOLVER-324
> Project: Maven Resolver
>  Issue Type: Wish
>  Components: Resolver
>Affects Versions: 2.0.0
>Reporter: wei cai
>Priority: Major
> Fix For: 2.0.0
>
>
> This is a WISH type Jira :D. So far, Maven 3.9.0 has released a BF algorithm 
> which includes changes from below JIRAs along with the DF option (default).
>  * MRESOLVER-228
>  * MRESOLVER-247
>  * MRESOLVER-7
> The BF algorithm solves:
>  * Cache missing issue when comes to resolve heavy dependencies with 
> different exclusions, it can help speed up dependency resolution especially 
> for large scale enterprise level projects.
>  * Improve download speed by:
>  ** Skip downloading poms for conflict losers.
>  ** Download poms in parallel.
> Below is the introduction in 
> [https://maven.apache.org/docs/3.9.0/release-notes.html:]
> {noformat}
> Choice of resolver collectors: a new BF collector (with parallel POM 
> downloads) has been added along the existing DF one.{noformat}
> The solution is already widely adopted at eBay. You can simply enable it by 
> putting below config item into file: ${maven.projectBasedir}/.mvn/maven.config
> {code:java}
> aether.dependencyCollector.impl=bf {code}
> More introduction about this option: 
> [https://maven.apache.org/resolver/configuration.html]
> Please try and tells us how much it helps by running:
> {code:java}
> 1. mvn clean install -DskipTests -Dmaven.repo.local=bf 
> -Daether.dependencyCollector.impl=bf 
> 2. mvn clean install -DskipTests -Dmaven.repo.local=df{code}
> With more feedback collected, we would be able to propose the changes to 
> Maven team to make BF as the default.
>  



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [maven-resolver] cstamas commented on pull request #325: [MRESOLVER-324] Make bf option as default

2023-08-07 Thread via GitHub


cstamas commented on PR #325:
URL: https://github.com/apache/maven-resolver/pull/325#issuecomment-1667284980

   We need to branch off 1.9.x first, then master made into 2.0.0...


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Commented] (MRESOLVER-324) Make the BF algorithm as the default option to speed up maven dependency resolution and downloading

2023-08-07 Thread ASF GitHub Bot (Jira)


[ 
https://issues.apache.org/jira/browse/MRESOLVER-324?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17751461#comment-17751461
 ] 

ASF GitHub Bot commented on MRESOLVER-324:
--

caiwei-ebay opened a new pull request, #325:
URL: https://github.com/apache/maven-resolver/pull/325

   (no comment)




> Make the BF algorithm as the default option to speed up maven dependency 
> resolution and downloading
> ---
>
> Key: MRESOLVER-324
> URL: https://issues.apache.org/jira/browse/MRESOLVER-324
> Project: Maven Resolver
>  Issue Type: Wish
>  Components: Resolver
>Affects Versions: 2.0.0
>Reporter: wei cai
>Priority: Major
> Fix For: 2.0.0
>
>
> This is a WISH type Jira :D. So far, Maven 3.9.0 has released a BF algorithm 
> which includes changes from below JIRAs along with the DF option (default).
>  * MRESOLVER-228
>  * MRESOLVER-247
>  * MRESOLVER-7
> The BF algorithm solves:
>  * Cache missing issue when comes to resolve heavy dependencies with 
> different exclusions, it can help speed up dependency resolution especially 
> for large scale enterprise level projects.
>  * Improve download speed by:
>  ** Skip downloading poms for conflict losers.
>  ** Download poms in parallel.
> Below is the introduction in 
> [https://maven.apache.org/docs/3.9.0/release-notes.html:]
> {noformat}
> Choice of resolver collectors: a new BF collector (with parallel POM 
> downloads) has been added along the existing DF one.{noformat}
> The solution is already widely adopted at eBay. You can simply enable it by 
> putting below config item into file: ${maven.projectBasedir}/.mvn/maven.config
> {code:java}
> aether.dependencyCollector.impl=bf {code}
> More introduction about this option: 
> [https://maven.apache.org/resolver/configuration.html]
> Please try and tells us how much it helps by running:
> {code:java}
> 1. mvn clean install -DskipTests -Dmaven.repo.local=bf 
> -Daether.dependencyCollector.impl=bf 
> 2. mvn clean install -DskipTests -Dmaven.repo.local=df{code}
> With more feedback collected, we would be able to propose the changes to 
> Maven team to make BF as the default.
>  



--
This message was sent by Atlassian Jira
(v8.20.10#820010)