Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FIX - Java 11 javax.xml.bind dependencies #254

Merged
merged 2 commits into from
Apr 9, 2019

Conversation

carlwilson
Copy link
Contributor

@carlwilson carlwilson commented Apr 8, 2019

  • added required dependencies to pom.xml dependencyManagement;
  • added managed dependencies to greenfield-apps/pom.xml and `pdfbox-apps/pom.xml';
  • removed --add-modules=java.xml.bind from all startup scripts as this fails from Java 11 onwards and the dependencies are now packaged; and
  • now sourcing download from OPF site, updates happen less often.

Closes veraPDF/veraPDF-library#986

- added required dependencies to `pom.xml` dependencyManagement;
- added managed dependencies to `greenfield-apps/pom.xml` and `pdfbox-apps/pom.xml';
- removed `--add-modules=java.xml.bind` from all startup scripts as this fails from Java 11 onwards and the dependencies are now packaged; and
- now sourcing download from OPF site, updates happen less often.
@carlwilson carlwilson added the bug label Apr 8, 2019
@carlwilson carlwilson self-assigned this Apr 8, 2019
@carlwilson carlwilson merged commit a902d9e into integration Apr 9, 2019
@carlwilson carlwilson deleted the fix/javax-dependencies-986 branch April 9, 2019 14:41
ShockwaveNN added a commit to ONLYOFFICE-QA/verapdf-verifier that referenced this pull request Aug 1, 2019
Image for ruby:2.6 update and include
java 11
And verapdf 1.12 cause error while working with this java
See:
veraPDF/veraPDF-apps#254
ShockwaveNN added a commit to ONLYOFFICE-QA/verapdf-verifier that referenced this pull request Aug 1, 2019
Image for ruby:2.6 update and include
java 11
And verapdf 1.12 cause error while working with this java
See:
veraPDF/veraPDF-apps#254
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants