Created attachment 765815 [details] dev-java/commons-logging-1.2-r4[log4j] build.log Reverse dependencies of dev-java/commons-logging[log4j] will have a "cannot find symbol" compiler error when they use the org.apache.commons.logging.impl.Log4JLogger class and are built against commons-logging-1.2-r4. The Log4JLogger class is present in the JARs built by commons-logging-1.2-r3[log4j], so this is a regression. This is because, when the -r4 revision is being built, build.xml will emit a warning saying "Log4j 1.2 not found: Cannot Build Log4JLogger", even if the 'log4j' USE flag is enabled, and log4j-12-api is added to the classpath: prepare: [echo] [echo] Log4j12: /usr/share/log4j-12-api-2/lib/log4j-12-api.jar:/usr/share/log4j-core-2/lib/log4j-core.jar [echo] [echo] [echo] LogKit: /usr/share/avalon-logkit-2.0/lib/avalon-logkit.jar [echo] Avalon-Framework: /usr/share/avalon-framework-4.2/lib/avalon-framework.jar [echo] [mkdir] Created dir: /var/tmp/portage/dev-java/commons-logging-1.2-r4/work/commons-logging-1.2-src/target [mkdir] Created dir: /var/tmp/portage/dev-java/commons-logging-1.2-r4/work/commons-logging-1.2-src/target/classes [mkdir] Created dir: /var/tmp/portage/dev-java/commons-logging-1.2-r4/work/commons-logging-1.2-src/target/conf [mkdir] Created dir: /var/tmp/portage/dev-java/commons-logging-1.2-r4/work/commons-logging-1.2-src/target/tests ... log4j12-warning: [echo] [echo] *** WARNING *** [echo] Log4j 1.2 not found: Cannot Build Log4JLogger [echo] ... show-lib-presence: [echo] jdk.1.4.present=true [echo] log4j12.present=${log4j12.present} [echo] logkit.present=true [echo] avalon-framework.present=true Note the 'log4j12.present=${log4j12.present}' printed in the 'show-lib-presence' task's output. With -r3, the value of this property is 'true': prepare: [echo] [echo] Log4j12: /usr/share/log4j/lib/log4j.jar [echo] [echo] [echo] LogKit: /usr/share/avalon-logkit-2.0/lib/avalon-logkit.jar [echo] Avalon-Framework: /usr/share/avalon-framework-4.2/lib/avalon-framework.jar [echo] [mkdir] Created dir: /var/tmp/portage/dev-java/commons-logging-1.2-r3/work/commons-logging-1.2-src/target [mkdir] Created dir: /var/tmp/portage/dev-java/commons-logging-1.2-r3/work/commons-logging-1.2-src/target/classes [mkdir] Created dir: /var/tmp/portage/dev-java/commons-logging-1.2-r3/work/commons-logging-1.2-src/target/conf [mkdir] Created dir: /var/tmp/portage/dev-java/commons-logging-1.2-r3/work/commons-logging-1.2-src/target/tests ... show-lib-presence: [echo] jdk.1.4.present=true [echo] log4j12.present=true [echo] logkit.present=true [echo] avalon-framework.present=true
Created attachment 765816 [details] dev-java/commons-logging-1.2-r3[log4j] build.log
The bug has been closed via the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=de2db80524e6e55868462ac22d89c48ff37e341a commit de2db80524e6e55868462ac22d89c48ff37e341a Author: Yuan Liao <liaoyuan@gmail.com> AuthorDate: 2022-02-26 02:59:49 +0000 Commit: Miroslav Šulc <fordfrog@gentoo.org> CommitDate: 2022-02-26 08:48:25 +0000 dev-java/commons-logging: Let build.xml accept log4j-12-api as Log4j 1.2 Closes: https://bugs.gentoo.org/834036 Signed-off-by: Yuan Liao <liaoyuan@gmail.com> Closes: https://github.com/gentoo/gentoo/pull/24352 Signed-off-by: Miroslav Šulc <fordfrog@gentoo.org> ...1.2-r4.ebuild => commons-logging-1.2-r5.ebuild} | 48 ++++++++++++++++++---- 1 file changed, 41 insertions(+), 7 deletions(-)
