From f08698352ac6653a61d57f40de8e74502ab917cf Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jo=C3=A3o=20Dinis=20Ferreira?= <hello@joaof.eu>
Date: Wed, 1 Apr 2026 21:10:41 +0200
Subject: [PATCH] fix: add SLF4J NOP binding to target, clean up stale imports
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Add slf4j.nop 2.0.17 from Eclipse Orbit aggregation 4.39.0 to the DDK
target platform. This makes the SLF4J NOP binding available to the OSGi
runtime, though due to classloader isolation in OSGi the startup warning
cannot be fully suppressed — slf4j.nop is a standalone bundle (not a
fragment of slf4j.api), so ServiceLoader-based provider discovery does
not find it before SLF4J's first initialization. This is a known
limitation shared by Eclipse Platform itself.

References:
- https://github.com/qos-ch/slf4j/issues/427
- https://github.com/orgs/eclipse-orbit/discussions/24

Also removes unused Import-Package: org.slf4j from three test bundle
manifests where no source file references org.slf4j:
- com.avaloq.tools.ddk.check.ui.test
- com.avaloq.tools.ddk.xtext.test
- com.avaloq.tools.ddk.xtext.test.core

Retains org.slf4j in com.avaloq.tools.ddk.test.ui — its SwtBot wrapper
classes directly use org.slf4j.Logger.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
---
 com.avaloq.tools.ddk.check.ui.test/META-INF/MANIFEST.MF   | 3 +--
 com.avaloq.tools.ddk.xtext.test.core/META-INF/MANIFEST.MF | 2 +-
 com.avaloq.tools.ddk.xtext.test/META-INF/MANIFEST.MF      | 3 +--
 ddk-target/ddk.target                                     | 3 ++-
 4 files changed, 5 insertions(+), 6 deletions(-)

diff --git a/com.avaloq.tools.ddk.check.ui.test/META-INF/MANIFEST.MF b/com.avaloq.tools.ddk.check.ui.test/META-INF/MANIFEST.MF
index c475c69de2..bf21036e77 100644
--- a/com.avaloq.tools.ddk.check.ui.test/META-INF/MANIFEST.MF
+++ b/com.avaloq.tools.ddk.check.ui.test/META-INF/MANIFEST.MF
@@ -29,8 +29,7 @@ Require-Bundle: com.avaloq.tools.ddk.check.ui,
 Export-Package: com.avaloq.tools.ddk.check.ui.test,
  com.avaloq.tools.ddk.check.ui.test.util,
  com.avaloq.tools.ddk.check
-Import-Package: org.slf4j,
- org.hamcrest.core,
+Import-Package: org.hamcrest.core,
  org.junit.runner;version="4.5.0",
  org.junit.runner.manipulation;version="4.5.0",
  org.junit.runner.notification;version="4.5.0",
diff --git a/com.avaloq.tools.ddk.xtext.test.core/META-INF/MANIFEST.MF b/com.avaloq.tools.ddk.xtext.test.core/META-INF/MANIFEST.MF
index 47dd4106a2..820ccc4b2e 100644
--- a/com.avaloq.tools.ddk.xtext.test.core/META-INF/MANIFEST.MF
+++ b/com.avaloq.tools.ddk.xtext.test.core/META-INF/MANIFEST.MF
@@ -26,7 +26,7 @@ Require-Bundle: com.avaloq.tools.ddk.xtext,
  org.eclipse.emf.common,
  com.avaloq.tools.ddk,
  junit-jupiter-api
-Import-Package: org.slf4j, org.apache.logging.log4j
+Import-Package: org.apache.logging.log4j
 Export-Package: com.avaloq.tools.ddk.xtext.test,
  com.avaloq.tools.ddk.xtext.test.contentassist,
  com.avaloq.tools.ddk.xtext.test.conversion,
diff --git a/com.avaloq.tools.ddk.xtext.test/META-INF/MANIFEST.MF b/com.avaloq.tools.ddk.xtext.test/META-INF/MANIFEST.MF
index cd82ca7785..7da0eca885 100644
--- a/com.avaloq.tools.ddk.xtext.test/META-INF/MANIFEST.MF
+++ b/com.avaloq.tools.ddk.xtext.test/META-INF/MANIFEST.MF
@@ -46,8 +46,7 @@ Import-Package: com.avaloq.tools.ddk.check.runtime.test.core,
  com.avaloq.tools.ddk.xtext.test.export,
  com.avaloq.tools.ddk.xtext.test.format,
  com.avaloq.tools.ddk.xtext.ui.test,
- org.apache.logging.log4j,
- org.slf4j
+ org.apache.logging.log4j
 Export-Package: com.avaloq.tools.ddk.xtext,
  com.avaloq.tools.ddk.xtext.formatter,
  com.avaloq.tools.ddk.xtext.formatter.services,
diff --git a/ddk-target/ddk.target b/ddk-target/ddk.target
index 75884c0d25..746fd8a8fa 100644
--- a/ddk-target/ddk.target
+++ b/ddk-target/ddk.target
@@ -57,7 +57,8 @@
     <unit id="org.apache.commons.lang3" version="3.20.0"/>
     <unit id="org.apache.logging.log4j.core" version="2.25.3"/>
     <unit id="org.apache.logging.log4j.api" version="2.25.3"/>
-    <unit id="org.apache.log4j" version="1.2.26"/> <!--use reload4j 1.2.26 for org.eclipse.xtext dependencies to log4j--> 
+    <unit id="org.apache.log4j" version="1.2.26"/> <!--use reload4j 1.2.26 for org.eclipse.xtext dependencies to log4j-->
+    <unit id="slf4j.nop" version="2.0.17"/>
   </location>
   <location includeAllPlatforms="false" includeConfigurePhase="false" includeMode="planner" includeSource="true" type="InstallableUnit">
     <repository location="https://download.eclipse.org/tools/orbit/simrel/maven-osgi/release/4.39.0"/>