On 2017-12-06 11:39, Robert Munteanu wrote:
Hi Konrad,

On Tue, 2017-12-05 at 16:47 +0100, Konrad Windszus wrote:
Hi Robert,
please see my inline comments

On 5. Dec 2017, at 16:23, Robert Munteanu <[email protected]>
wrote:

Hi,

...

- Tika is yet to have a release which removes JAXB usage

This tika release should happen very soon (tm):
http://lucene.472066.n3.nabble.com/Tika-1-17-td4361894.html

Well, they've been discussing this for a month :-)


- We did not have the time to replace deprecated modules with third
party jars, partly because we would need to do it for all supported
Java versions, which is complicated.

Maybe I don't see all the details, but if I understand correctly we
are talking mostly about https://issues.apache.org/jira/browse/SLING-
7135 <https://issues.apache.org/jira/browse/SLING-7135> (i.e. the
class javax/annotation/PostConstruct). This should be contained in ht
tps://mvnrepository.com/artifact/javax.annotation/jsr250-api <https:/
/mvnrepository.com/artifact/javax.annotation/jsr250-api>. Isn't it
possible to add that to the Sling Launchpad with a bundle header
which prevents it from being started with a JRE < 9?
The only other one is https://issues.apache.org/jira/browse/SLING-
7106 <https://issues.apache.org/jira/browse/SLING-7106> but I am not
sure, whether the javax/xml/bind/JAXBException is only used in the
test class or also in a code path that is executed at run time.

That's the root of the problem. We would need to add external jars for
all packages that are no in java.see.ee, and make sure we don't break
anything for older JVMs. Which AFAICT is problematic.
...

<https://issues.apache.org/jira/browse/SUREFIRE-1424> has some related discussion.

FWIW, adding "--add-modules java.se.ee" doesn't work for some packages, such as javax.transaction.UserTransaction (see https://issues.apache.org/jira/browse/JCR-4200).

Best regards, Julian

Reply via email to