jbonofre commented on a change in pull request #662:
URL: https://github.com/apache/activemq/pull/662#discussion_r781757330
##########
File path: pom.xml
##########
@@ -82,7 +80,8 @@
<junit-version>4.13.2</junit-version>
<hamcrest-version>1.3</hamcrest-version>
<karaf-version>4.2.10</karaf-version>
- <log4j-version>1.2.17</log4j-version>
+ <slf4j-version>1.7.30</slf4j-version>
+ <log4j-version>2.14.1</log4j-version>
Review comment:
Note:
1. ActiveMQ is not EOL, ActiveMQ 5.17.0 with at least log4j 2.17.1 will be
available soon
2. log4j 1.x is NOT EOL, there's a discussion to restart the project and cut
new releases. Reason is simply a bunch of projects (at Apache and elsewhere)
still uses log4j 1.x
3. I'm sure you have a bunch of projects/products that use EOL dependencies
without problem ;)
4. From an Apache standpoint, EOL doesn't exist, anyone can contribute even
on old branches/projects and do new releases (see log4j 1.x example).
5. Please, stop to "pollute" the PR, use the mailing list or direct email,
thanks !
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
