Current status:

* Pull request merged
* jena-osgi-test commented out of the build
   * Fatal problem building
   * LGPL issue (not an issue ATM as not in the build)


1/ Build failure

jena-osgi-test fails the build in step "verify"

tried: java-8-openjdk and java-7-openjdk

------------------------
Exception in thread "main" java.lang.Error:

There is an incompatible JNA native library installed on this system.
To resolve this issue you may do one of the following:
 - remove or uninstall the offending library
 - set the system property jna.nosys=true
 - set jna.boot.library.path to include the path to the version of the
   jnidispatch library included with the JNA jar file you are using

        at com.sun.jna.Native.<clinit>(Native.java:142)
...
------------------------

2/ There is a LGPL dependency (scope test) which needs investigation.
(I would have appreciated that having been pointed out first)

It means an end-user could not build jena from source without that dependency (the build would fail). It is not "provided".


and some questions thrown up by merging the contribution:

Q1/

Why are the dependencies of jena-core etc in jena-osgi "provided"?

Q2/

There is a note about a change for when we use jsonld-java 0.5.1 ... and we now are. Should that change be made? Has it been tested?

Q3/

Why was there a separate <modules> for jena-osgi-test?
jena-osgi isn't in all profiles.
(moved to next to jena-osgi)

Q4/

We have a style of nesting directories (see Elephas, jena-jdbc, Fuseki2).

Can we put jena-osgi-test inside jena-osgi?

Q5/ It says:

<timeout>150000</timeout><!-- 15000ms = 15s -->

no!

150000 is 150s -- which is right, code or comment?


Q6/

3/ Build warnings : do they matter?

If not, can they be suppressed?

[WARNING] Bundle org.apache.jena:jena-osgi:bundle:2.12.2-SNAPSHOT : Unused Private-Package instructions, no such package(s) on the class path: [!*] [WARNING] Bundle org.apache.jena:jena-osgi:bundle:2.12.2-SNAPSHOT : Export com.hp.hpl.jena.datatypes.xsd, has 1, private references [org.apache.xerces.impl.dv],

and 8 others about private references

        Andy






Reply via email to