Skip to content

Commit 1652abe

Browse files
committed
Embed the google cache dependency in the bundle
Guava is known as a bit "problematic" dependency, as lemminx only uses guava-cache, this can be embedded in the bundle and be made embedded package. This changes the used felix-bundle to bnd-maven-plugin with the conditionalpackage to include the caching api in the generated bundle Signed-off-by: Christoph Läubrich <laeubi@laeubi-soft.de>
1 parent 0aadd16 commit 1652abe

File tree

2 files changed

+19
-20
lines changed

2 files changed

+19
-20
lines changed

org.eclipse.lemminx/pom.xml

Lines changed: 16 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -72,13 +72,26 @@
7272
</configuration>
7373
</plugin>
7474
<plugin>
75-
<groupId>org.apache.felix</groupId>
76-
<artifactId>maven-bundle-plugin</artifactId>
75+
<groupId>biz.aQute.bnd</groupId>
76+
<artifactId>bnd-maven-plugin</artifactId>
77+
<executions>
78+
<execution>
79+
<id>bnd-process</id>
80+
<goals>
81+
<goal>bnd-process</goal>
82+
</goals>
83+
<configuration>
84+
<bnd><![CDATA[
85+
Bundle-SymbolicName: ${project.artifactId}
86+
-conditionalpackage: com.google.common.cache
87+
]]></bnd>
88+
</configuration>
89+
</execution>
90+
</executions>
7791
</plugin>
7892
<plugin>
7993
<groupId>org.apache.maven.plugins</groupId>
8094
<artifactId>maven-jar-plugin</artifactId>
81-
<version>3.3.0</version>
8295
<executions>
8396
<execution>
8497
<goals>

pom.xml

Lines changed: 3 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -159,23 +159,9 @@
159159
</configuration>
160160
</plugin>
161161
<plugin>
162-
<groupId>org.apache.felix</groupId>
163-
<artifactId>maven-bundle-plugin</artifactId>
164-
<version>5.1.9</version>
165-
<executions>
166-
<execution>
167-
<id>bundle-manifest</id>
168-
<phase>process-classes</phase>
169-
<goals>
170-
<goal>manifest</goal>
171-
</goals>
172-
<configuration>
173-
<instructions>
174-
<Bundle-SymbolicName>${project.artifactId}</Bundle-SymbolicName>
175-
</instructions>
176-
</configuration>
177-
</execution>
178-
</executions>
162+
<groupId>biz.aQute.bnd</groupId>
163+
<artifactId>bnd-maven-plugin</artifactId>
164+
<version>6.4.0</version>
179165
</plugin>
180166
<plugin>
181167
<groupId>org.apache.maven.plugins</groupId>

0 commit comments

Comments
 (0)