You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@logging.apache.org by "vy (via GitHub)" <gi...@apache.org> on 2025/12/16 15:07:58 UTC

[PR] Merge `rel/2.25.3` to `2.x` [logging-log4j2]

vy opened a new pull request, #4004:
URL: https://github.com/apache/logging-log4j2/pull/4004

   (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 unsubscribe, e-mail: notifications-unsubscribe@logging.apache.org

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


Re: [PR] Merge `rel/2.25.3` to `2.x` [logging-log4j2]

Posted by "vy (via GitHub)" <gi...@apache.org>.
vy commented on code in PR #4004:
URL: https://github.com/apache/logging-log4j2/pull/4004#discussion_r2624700281


##########
log4j-1.2-api/src/main/java/org/apache/log4j/helpers/package-info.java:
##########
@@ -18,7 +18,7 @@
  * Log4j 1.x compatibility layer.
  */
 @Export
-@Version("2.26.0")
+@Version("2.25.3")

Review Comment:
   Fixed in dde4d4a.



-- 
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: notifications-unsubscribe@logging.apache.org

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


Re: [PR] Merge `rel/2.25.3` to `2.x` [logging-log4j2]

Posted by "ppkarwasz (via GitHub)" <gi...@apache.org>.
ppkarwasz commented on code in PR #4004:
URL: https://github.com/apache/logging-log4j2/pull/4004#discussion_r2623740808


##########
log4j-1.2-api/src/main/java/org/apache/log4j/helpers/package-info.java:
##########
@@ -18,7 +18,7 @@
  * Log4j 1.x compatibility layer.
  */
 @Export
-@Version("2.26.0")
+@Version("2.25.3")

Review Comment:
   These versions (updated in commit 8d43a991c3068c6ff4c8fc38ee2c73ba5d24299a and e2898a0ec334cd4527b6618b3190edd1d47535f4) are wrong:
   
   - In the release `2.25.2` of `log4j-1.2-api`, the version of this package was `2.20.3`,
   - Since version `2.25.3` only made **patch** level changes to the public interface, the version should have been update to `2.20.4`. The API of the package is identical to what we released in `2.20.0`, with some micro changes to annotations.
   
   Anyway, since Log4j `2.25.3` is already out and package versions need to be **monotonic**, there is nothing we can do right now.



-- 
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: notifications-unsubscribe@logging.apache.org

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


Re: [PR] Merge `rel/2.25.3` to `2.x` [logging-log4j2]

Posted by "vy (via GitHub)" <gi...@apache.org>.
vy merged PR #4004:
URL: https://github.com/apache/logging-log4j2/pull/4004


-- 
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: notifications-unsubscribe@logging.apache.org

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