Re: [PR] Use Maven Wrapper to build [maven]

2024-06-03 Thread via GitHub
slawekjaranowski merged PR #1553: URL: https://github.com/apache/maven/pull/1553 -- 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...@mave

Re: [PR] Use Maven Wrapper to build [maven]

2024-06-03 Thread via GitHub
slawekjaranowski commented on PR #1548: URL: https://github.com/apache/maven/pull/1548#issuecomment-2145825719 Done in #1553 -- 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.

[PR] Use Maven Wrapper to build [maven]

2024-06-03 Thread via GitHub
slawekjaranowski opened a new pull request, #1553: URL: https://github.com/apache/maven/pull/1553 (no comment) -- 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 unsubscrib

Re: [PR] Use Maven Wrapper to build [maven]

2024-06-03 Thread via GitHub
slawekjaranowski commented on PR #1548: URL: https://github.com/apache/maven/pull/1548#issuecomment-2145810928 I will create a new one for 3.x 😄 -- 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 t

Re: [PR] Use Maven Wrapper to build [maven]

2024-06-03 Thread via GitHub
slawekjaranowski commented on PR #1548: URL: https://github.com/apache/maven/pull/1548#issuecomment-2145809544 #1550 is for master ... this one was for 3.x branch -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] Use Maven Wrapper to build [maven]

2024-06-03 Thread via GitHub
slawekjaranowski merged PR #1550: URL: https://github.com/apache/maven/pull/1550 -- 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...@mave

Re: [PR] Use Maven Wrapper to build [maven]

2024-06-03 Thread via GitHub
cstamas commented on PR #1548: URL: https://github.com/apache/maven/pull/1548#issuecomment-2145782407 Superseded by https://github.com/apache/maven/pull/1550 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abov

Re: [PR] Use Maven Wrapper to build [maven]

2024-06-03 Thread via GitHub
cstamas closed pull request #1548: Use Maven Wrapper to build URL: https://github.com/apache/maven/pull/1548 -- 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-m

Re: [PR] Use Maven Wrapper to build [maven]

2024-06-03 Thread via GitHub
slawekjaranowski commented on PR #1548: URL: https://github.com/apache/maven/pull/1548#issuecomment-2145748141 I hope it is ready to merge @cstamas @gnodet what do you think -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

Re: [PR] Use Maven Wrapper to build [maven]

2024-06-01 Thread via GitHub
slawekjaranowski commented on PR #1550: URL: https://github.com/apache/maven/pull/1550#issuecomment-2143604023 This one is needed https://github.com/apache/maven-integration-testing/pull/345 -- This is an automated message from the Apache Git Service. To respond to the message, please log

[PR] Use Maven Wrapper to build [maven]

2024-06-01 Thread via GitHub
slawekjaranowski opened a new pull request, #1550: URL: https://github.com/apache/maven/pull/1550 We should control Maven version used for builds -- 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 t

Re: [PR] Use Maven Wrapper to build [maven]

2024-06-01 Thread via GitHub
slawekjaranowski commented on code in PR #1548: URL: https://github.com/apache/maven/pull/1548#discussion_r1623245316 ## .github/workflows/maven.yml: ## @@ -47,7 +47,7 @@ jobs: cache: 'maven' - name: Build with Maven -run: mvn verify -e -B -V -Ddistri

Re: [PR] Use Maven Wrapper to build [maven]

2024-06-01 Thread via GitHub
gnodet commented on PR #1548: URL: https://github.com/apache/maven/pull/1548#issuecomment-2143461937 Actually, we're not using it... I thought, but the matrix is used with wrapper https://github.com/apache/maven-gh-actions-shared/blob/v4/.github/workflows/maven-verify.yml#L196-L198

Re: [PR] Use Maven Wrapper to build [maven]

2024-06-01 Thread via GitHub
laeubi commented on PR #1548: URL: https://github.com/apache/maven/pull/1548#issuecomment-2143449905 > You are mixing things. Again, we do NOT use setup-maven action, hence the readme you point it is... off topic. I mentioned [you need to use setup-maven action](https://github.com/ap

Re: [PR] Use Maven Wrapper to build [maven]

2024-06-01 Thread via GitHub
cstamas commented on PR #1548: URL: https://github.com/apache/maven/pull/1548#issuecomment-2143449297 IMHO we should not use any action to setup Maven but wrapper that's my 5 cents (we even have examples of "maven matrix" that is simply doable with wrapper). -- This is an automated m

Re: [PR] Use Maven Wrapper to build [maven]

2024-06-01 Thread via GitHub
cstamas commented on PR #1548: URL: https://github.com/apache/maven/pull/1548#issuecomment-2143449034 > Well I would call it **strange** to use an action without specify a version ... I mean even [the readme tells you to specify one](https://github.com/stCarolas/setup-maven), i more suspect

Re: [PR] Use Maven Wrapper to build [maven]

2024-06-01 Thread via GitHub
laeubi commented on PR #1548: URL: https://github.com/apache/maven/pull/1548#issuecomment-2143448479 setup-java has nothing to do with installing / "using" maven it just do the following: - generates `toolchain.xml` from installed jdks - uses `pom.xml` files as cache keys be

Re: [PR] Use Maven Wrapper to build [maven]

2024-06-01 Thread via GitHub
rmannibucau commented on PR #1548: URL: https://github.com/apache/maven/pull/1548#issuecomment-2143424345 setup-java uses OS maven version which is likely not what we want. Guess we should reuse our shared action enabling to support setup-maven to have a build matrix (ideally). -- This

Re: [PR] Use Maven Wrapper to build [maven]

2024-06-01 Thread via GitHub
laeubi commented on PR #1548: URL: https://github.com/apache/maven/pull/1548#issuecomment-2143422824 > I can't see the use of this action for ITs. We do use actions/setup-java, but not actions/setup-maven. I would think reusing this action in the context of GH action makes more sense.

Re: [PR] Use Maven Wrapper to build [maven]

2024-06-01 Thread via GitHub
gnodet commented on PR #1548: URL: https://github.com/apache/maven/pull/1548#issuecomment-2143420229 > > There is a setup-maven action: https://github.com/marketplace/actions/setup-maven > > I use it all the time without any issues and you can perfectly control the maven version. >

Re: [PR] Use Maven Wrapper to build [maven]

2024-06-01 Thread via GitHub
laeubi commented on PR #1548: URL: https://github.com/apache/maven/pull/1548#issuecomment-2143352303 > and the last IT failures made us realize how strange it behaves: it set up different Maven versions per > True, we did not ask for specific maven version, but that is not the point.

Re: [PR] Use Maven Wrapper to build [maven]

2024-06-01 Thread via GitHub
cstamas commented on PR #1548: URL: https://github.com/apache/maven/pull/1548#issuecomment-2143346085 > There is a setup-maven action: https://github.com/marketplace/actions/setup-maven > > I use it all the time without any issues and you can perfectly control the maven version.

Re: [PR] Use Maven Wrapper to build [maven]

2024-05-31 Thread via GitHub
laeubi commented on PR #1548: URL: https://github.com/apache/maven/pull/1548#issuecomment-2143285471 There is a setup-maven action: https://github.com/marketplace/actions/setup-maven-action -- This is an automated message from the Apache Git Service. To respond to the message, please lo

Re: [PR] Use Maven Wrapper to build [maven]

2024-05-31 Thread via GitHub
slawekjaranowski commented on PR #1548: URL: https://github.com/apache/maven/pull/1548#issuecomment-2142844431 or we can add: ``` - name: Set up Maven run: mvn --errors --batch-mode --show-version org.apache.maven.plugins:maven-wrapper-plugin:3.3.2:wra