Skip to content

Commit

Permalink
Improve BND config (apache/logging-log4j2#1895)
Browse files Browse the repository at this point in the history
  • Loading branch information
vy committed Oct 24, 2023
1 parent cfdda22 commit d4c2081
Show file tree
Hide file tree
Showing 4 changed files with 36 additions and 1 deletion.
10 changes: 9 additions & 1 deletion pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -173,6 +173,8 @@
~ See: https://bnd.bndtools.org/chapters/330-jpms.html#advanced-options
-->
<bnd-extra-module-options />
<!-- Allows to extend the `bnd-maven-plugin` configuration with extras: -->
<bnd-extra-config />

<!-- dependency versions -->
<org.eclipse.jgit.version>6.7.0.202309050840-r</org.eclipse.jgit.version>
Expand Down Expand Up @@ -719,7 +721,10 @@
-jpms-module-info: $[bnd-module-name];access=0
# Prevents an execution error in multi-release jars:
-fixupmessages: "Classes found in the wrong directory";restrict:=error;is:=warning
-fixupmessages.classes_in_wrong_dir: "Classes found in the wrong directory";restrict:=error;is:=warning
# Convert API leakage warnings to errors
-fixupmessages.priv_refs: "private references";restrict:=warning;is:=error
# 1. OSGI modules do not make sense in JPMS
# 2. BND has a problem detecting the name of multi-release JPMS modules
Expand All @@ -740,6 +745,9 @@
# Add manifests and modules for each multi-release version:
-jpms-multi-release: $[bnd-multi-release]
# Extra configuration provided by the consumer:
${bnd-extra-config}
]]></bnd>
</configuration>
<executions>
Expand Down
9 changes: 9 additions & 0 deletions src/changelog/.10.x.x/add-bnd-extra-config.xml
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
<?xml version="1.0" encoding="UTF-8"?>
<entry xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xmlns="http://logging.apache.org/log4j/changelog"
xsi:schemaLocation="http://logging.apache.org/log4j/changelog https://logging.apache.org/log4j/changelog-0.1.2.xsd"
type="added">
<issue id="apache/logging-log4j2#1895" link="https://github.com/apache/logging-log4j2/issues/1895"/>
<author id="github:vy"/>
<description format="asciidoc">Added support to extend the `bnd-maven-plugin` configuration with `bnd-extra-config` property</description>
</entry>
9 changes: 9 additions & 0 deletions src/changelog/.10.x.x/fail-bnd-on-api-leak.xml
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
<?xml version="1.0" encoding="UTF-8"?>
<entry xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xmlns="http://logging.apache.org/log4j/changelog"
xsi:schemaLocation="http://logging.apache.org/log4j/changelog https://logging.apache.org/log4j/changelog-0.1.2.xsd"
type="changed">
<issue id="apache/logging-log4j2#1895" link="https://github.com/apache/logging-log4j2/issues/1895"/>
<author id="github:vy"/>
<description format="asciidoc">Converted `bnd-maven-plugin` API leakage warnings to errors</description>
</entry>
9 changes: 9 additions & 0 deletions src/site/_release-notes/_10.x.x.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,16 @@
This minor release contains several small improvements.
==== Added
* Added support to extend the `bnd-maven-plugin` configuration with `bnd-extra-config` property (https://github.com/apache/logging-log4j2/issues/1895[apache/logging-log4j2#1895])
==== Changed
* Converted `bnd-maven-plugin` API leakage warnings to errors (https://github.com/apache/logging-log4j2/issues/1895[apache/logging-log4j2#1895])
* Update `com.google.errorprone:error_prone_core` to version `2.23.0` (https://github.com/apache/logging-parent/pull/49[49])
* Update `github/codeql-action` to version `2.22.4` (https://github.com/apache/logging-parent/pull/51[51])
==== Fixed
* Attach `flatten:clean` to `clean` phase

0 comments on commit d4c2081

Please sign in to comment.