Uploaded image for project: 'Apache Jena'
  1. Apache Jena
  2. JENA-2253

Add module-info.java

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Trivial
    • Resolution: Fixed
    • Jena 4.3.2
    • Jena 4.4.0
    • Jena
    • None

    Description

      I am quite new to the Java 9 modularity,  but my developers told me that it can be "enabled" by adding some basic module metadata in a module-info.java at the root of the java sources.

      Therefore, I suggest adding a module-info to your Jena project, so that users who are using a java module based project setup can benefit from this.

      We, the ODF Toolkit, are using Jena and we started such module-info.java

      https://github.com/tdf/odftoolkit/pull/149/files

      But somehow as soon we got this metadata the Java classes of the jena-core-tests JAR can no longer be found.
      The tests are not in our package list as we use them just during build/test time.

      Perhaps I can make one of you curious enough to test to create such simple module-info.java files to the root of your Java sources similar as done in the PR from us: https://github.com/tdf/odftoolkit/pull/149

      (The other reason for this issue, is also that we face an anomaly of lacking to find your jena-core-tests classes after using this new module-info.java. I hope that this phenomenon goes away when you add your module-info.java as well. Currently, accessing the jena-core-tests via pom.xml in the old way,https://github.com/tdf/odftoolkit/blob/master/odfdom/pom.xml#L72 but the [new way using apache-jena-libs in pom.xml|https://jena.apache.org/download/maven.html does not work either.).

      Thanks in advance,

      Svante

      Attachments

        Issue Links

          Activity

            People

              andy Andy Seaborne
              svanteschubert Svante Schubert
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: