Skip to content

Commit

Permalink
Merge pull request #3886 from ctrueden/logging-dep
Browse files Browse the repository at this point in the history
Make logback dependencies optional
  • Loading branch information
dgault authored Jan 30, 2023
2 parents 3493350 + 36824f9 commit ef03cc9
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 0 deletions.
2 changes: 2 additions & 0 deletions components/bio-formats-plugins/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -72,11 +72,13 @@
<groupId>ch.qos.logback</groupId>
<artifactId>logback-core</artifactId>
<version>${logback.version}</version>
<optional>true</optional>
</dependency>
<dependency>
<groupId>ch.qos.logback</groupId>
<artifactId>logback-classic</artifactId>
<version>${logback.version}</version>
<optional>true</optional>
</dependency>

<!-- NB: dependency:analyze has false warning about xml-apis:xml-apis. -->
Expand Down
2 changes: 2 additions & 0 deletions components/bio-formats-tools/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -67,13 +67,15 @@
<artifactId>logback-core</artifactId>
<version>${logback.version}</version>
<scope>runtime</scope>
<optional>true</optional>
</dependency>
<dependency>
<!-- NB: We want this, despite warning from dependency:analyze. -->
<groupId>ch.qos.logback</groupId>
<artifactId>logback-classic</artifactId>
<version>${logback.version}</version>
<scope>runtime</scope>
<optional>true</optional>
</dependency>
<dependency>
<!-- NB: We want this, despite warning from dependency:analyze. -->
Expand Down

0 comments on commit ef03cc9

Please sign in to comment.