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: Open JDK 11 build #716

Merged
merged 3 commits into from
Apr 6, 2022
Merged

FIX: Open JDK 11 build #716

merged 3 commits into from
Apr 6, 2022

Conversation

carlwilson
Copy link
Member

  • moved the Jhove and JhoveView classes from default packageas Java 11 tooling complained;
  • updated jhove-apps POM to add new Jhove class package to manifest line;
  • updated windows batch and linux shell start scripts to add new Jhove class package to manifest line;
  • added JAXB API and implementation classes to pom.xml;
  • updated coverage call Jacoco version number in ./jhove-bbt/scripts/bbt-jhove.sh b/jhove-bbt/scripts/bbt-jhove.sh and
  • updated Maven dependencies for izpack -> 5.1.3 and jacoco -> 0.8.7for Java 11 compatibility.

- moved the `Jhove` and `JhoveView` classes from default packageas Java 11 tooling complained;
- updated `jhove-apps` POM to add new `Jhove` class package to manifest line;
- updated windows batch and linux shell start scripts to add new `Jhove` class package to manifest line; and
- updated Maven dependencies for `izpack -> 5.1.3` and `jacoco -> 0.8.7`for Java 11 compatibility.
- added JAXB API and implementation classes to `pom.xml`; and
- updated coverage call Jacoco version number in `./jhove-bbt/scripts/bbt-jhove.sh b/jhove-bbt/scripts/bbt-jhove.sh`.
@carlwilson carlwilson added bug A product defect that needs fixing P1 High priority issues to be scheduled in the upcoming release RC1.26 labels Apr 6, 2022
@carlwilson carlwilson added this to the JHOVE 1.26 milestone Apr 6, 2022
@carlwilson carlwilson self-assigned this Apr 6, 2022
@codecov
Copy link

codecov bot commented Apr 6, 2022

Codecov Report

Merging #716 (6d7753e) into integration (6aac988) will increase coverage by 0.00%.
The diff coverage is n/a.

@@              Coverage Diff               @@
##             integration     #716   +/-   ##
==============================================
  Coverage          45.63%   45.63%           
  Complexity          1046     1046           
==============================================
  Files                 57       57           
  Lines               9149     9144    -5     
  Branches            1687     1682    -5     
==============================================
- Hits                4175     4173    -2     
+ Misses              4424     4422    -2     
+ Partials             550      549    -1     
Impacted Files Coverage Δ
...main/java/edu/harvard/hul/ois/jhove/JhoveBase.java 42.82% <0.00%> (-0.25%) ⬇️
.../main/java/edu/harvard/hul/ois/jhove/MacStuff.java 0.00% <0.00%> (ø)
...ava/org/openpreservation/jhove/ReleaseDetails.java 82.75% <0.00%> (ø)
...va/edu/harvard/hul/ois/jhove/ObjectIdentifier.java 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ae53c2c...6d7753e. Read the comment docs.

@carlwilson carlwilson merged commit e1a78f4 into integration Apr 6, 2022
@carlwilson carlwilson deleted the fix/java11 branch April 6, 2022 18:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A product defect that needs fixing P1 High priority issues to be scheduled in the upcoming release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant