You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@artemis.apache.org by "jbertram (via GitHub)" <gi...@apache.org> on 2026/01/07 17:35:12 UTC
[PR] ARTEMIS-5839 Bump org.apache:apache from 35 to 36 [artemis]
jbertram opened a new pull request, #6169:
URL: https://github.com/apache/artemis/pull/6169
Note: this requires Maven 3.9.0 minimum (up from 3.6.3)
--
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-unsubscribe@artemis.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@artemis.apache.org
For additional commands, e-mail: issues-help@artemis.apache.org
Re: [PR] ARTEMIS-5839 Bump org.apache:apache from 35 to 37 [artemis]
Posted by "gemmellr (via GitHub)" <gi...@apache.org>.
gemmellr commented on PR #6169:
URL: https://github.com/apache/artemis/pull/6169#issuecomment-3836524498
I'd consider dropping most of our maven/java version enforcer executions (but not the release profile one) in favour the parent poms. I did that in https://github.com/apache/qpid-jms/commit/9cbf7b715b82c92afa8793439833620ba10c8732 for example.
--
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-unsubscribe@artemis.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@artemis.apache.org
For additional commands, e-mail: issues-help@artemis.apache.org
Re: [PR] ARTEMIS-5839 Bump org.apache:apache from 35 to 36 [artemis]
Posted by "gemmellr (via GitHub)" <gi...@apache.org>.
gemmellr commented on PR #6169:
URL: https://github.com/apache/artemis/pull/6169#issuecomment-3749724425
We could look at removing some (we also override the java version check in the release profile, which needs to stay) of our overridden enforcement in favour of the parents. Though personally I'm not a fan of their new singular 'javaVersion' property, I much prefer the component poms having the maven compiler properties themselves (and I know of things looking for them currently) so I'd probably leave ours in place even if we use the parents enforcement checks.
--
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-unsubscribe@artemis.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@artemis.apache.org
For additional commands, e-mail: issues-help@artemis.apache.org
Re: [PR] ARTEMIS-5839 Bump org.apache:apache from 35 to 37 [artemis]
Posted by "jbertram (via GitHub)" <gi...@apache.org>.
jbertram commented on PR #6169:
URL: https://github.com/apache/artemis/pull/6169#issuecomment-3836449509
@gemmellr, I think this is good to go.
--
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-unsubscribe@artemis.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@artemis.apache.org
For additional commands, e-mail: issues-help@artemis.apache.org
Re: [PR] ARTEMIS-5839 Bump org.apache:apache from 35 to 36 [artemis]
Posted by "gemmellr (via GitHub)" <gi...@apache.org>.
gemmellr commented on PR #6169:
URL: https://github.com/apache/artemis/pull/6169#issuecomment-3749697677
I made the minimal update on our existing setup in https://github.com/apache/artemis/commit/14b34424eb to bump the Maven requirement to 3.9+
--
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-unsubscribe@artemis.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@artemis.apache.org
For additional commands, e-mail: issues-help@artemis.apache.org
Re: [PR] ARTEMIS-5839 Bump org.apache:apache from 35 to 36 [artemis]
Posted by "gemmellr (via GitHub)" <gi...@apache.org>.
gemmellr commented on PR #6169:
URL: https://github.com/apache/artemis/pull/6169#issuecomment-3748861160
A newer 37 version has been released (#6178) since changes in 36 caused issues with releasing. As the overall changes are quite numerous [1], but nothing in it seems truly needed right now (or we already overrode where needed, as you have unwound where appropriate), I think we should hold off on the change until after we release 2.50.0.
I dont think this will actually require Maven 3.9.0 as we have our own enforcer checks for Java and Maven versions overriding the parents, as its tended to lag what was wanted, and it also kept changing how theirs worked (just has again..). I do think requiring 3.9.0 makes sense though, and its now ours that is lagging, I'll have a look at updating that.
[1] https://github.com/apache/maven-apache-parent/compare/apache-35...apache-37
--
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-unsubscribe@artemis.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@artemis.apache.org
For additional commands, e-mail: issues-help@artemis.apache.org
Re: [PR] ARTEMIS-5839 Bump org.apache:apache from 35 to 37 [artemis]
Posted by "jbertram (via GitHub)" <gi...@apache.org>.
jbertram commented on PR #6169:
URL: https://github.com/apache/artemis/pull/6169#issuecomment-3836586504
@gemmellr, good idea. Check the latest push.
--
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-unsubscribe@artemis.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@artemis.apache.org
For additional commands, e-mail: issues-help@artemis.apache.org
Re: [PR] ARTEMIS-5839 Bump org.apache:apache from 35 to 37 [artemis]
Posted by "gemmellr (via GitHub)" <gi...@apache.org>.
gemmellr merged PR #6169:
URL: https://github.com/apache/artemis/pull/6169
--
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-unsubscribe@artemis.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@artemis.apache.org
For additional commands, e-mail: issues-help@artemis.apache.org