search

Jasig / maven-notice-plugin

Apache License 2.0
7 stars 16 forks source link

Does not work with Java 11 #18

Open jakub-lore opened 4 years ago

jakub-lore commented 4 years ago

Seems that the plugin requires javax.xml.bind package that is not a part of default distribution anymore

ChristianMurphy commented 4 years ago

Thanks for reaching out @Leithain! A pull request adding the dependencies needed for Java 11 would be welcome!

jakobbraun commented 4 years ago

I prepared a PR for that. I can however not upload it until #19 is fixed, since I do not want trouble with the Apache Maven Trademark ;-)

ChristianMurphy commented 3 years ago

@jakobbraun the license is unrelated to this issue, and can be addressed separately, so it shouldn't really be a blocker.

jcodagnone commented 2 years ago

A workarround that works at least on Java 17 is adding the missing dependencies to the runtime:

      <plugin>
        <groupId>org.jasig.maven</groupId>
        <artifactId>maven-notice-plugin</artifactId>
        <version>1.0.6.1</version>
        <dependencies>
          <dependency>
              <groupId>jakarta.xml.bind</groupId>
              <artifactId>jakarta.xml.bind-api</artifactId>
              <version>2.3.3</version>
          </dependency>
          <dependency>
              <groupId>org.glassfish.jaxb</groupId>
              <artifactId>jaxb-runtime</artifactId>
              <version>2.3.3</version>
          </dependency>
        </dependencies>
       …