Apache OpenOffice (AOO) Bugzilla – Issue 127876
unbreak ODK build with Java 8
Last modified: 2022-10-28 12:54:16 UTC
When building OpenOffice with the --enable-odk option, the build fails if Java 8 in installed. Since earlier versions of Java are difficult to obtain, this could be problematical for anyone who attempts to build OpenOffice 4.1.6 if they don't already have an older version of Java installed. To fix, these commits should be merged from trunk: https://svn.apache.org/viewvc?view=revision&revision=1591062 (partial) https://svn.apache.org/viewvc?view=revision&revision=1697228 https://svn.apache.org/viewvc?view=revision&revision=1697237 https://svn.apache.org/viewvc?view=revision&revision=1697247 https://svn.apache.org/viewvc?view=revision&revision=1697306 https://svn.apache.org/viewvc?view=revision&revision=1697312 The merge of 1591062 is only needed to prevent a merge conflict in one of the later merges. It is only a partial merge because a full merge of 1591062 fails with a merge conflict. In the top level directory of AOO416: svn merge -c 1591062 '^/openoffice/trunk/main/xmerge' main/xmerge svn merge -c 1697228 '^/openoffice/trunk' . svn merge -c 1697237 '^/openoffice/trunk' . svn merge -c 1697247 '^/openoffice/trunk' . svn merge -c 1697306 '^/openoffice/trunk' . svn merge -c 1697312 '^/openoffice/trunk' . Fix tested in 4.1.6 on 32-bit Windows 7 with java 8 and CentOS 6 x86_64 with java 1.7.
I did some experiments with these commits on Windows: With trunk (built with JAVA 8) I can use JAVA 7 or 8 (both installed). With AOO 4.1.x (built with Java 8) I can only use JAVA 8. JAVA 7 will not be recognized. Since both versions are found in trunk, there may be a minor issue in detecting JAVA 7 left in 4.1.x.
Accepted for 4.1.6
"petko" committed SVN revision 1843109 into branches/AOO416: Issue 127876 - unbreak ODK build with Java 8
Revoked with r1843571. Since Java 7 isn't detected on Windows anymore after applying these patches I think we should hold back this issue until this is solved. There are a lot of people who still use Java 7 (or older) on Windows.
Okay moving this out of the 4.1.6 release and into 4.2.0
changed to much. Sry
(In reply to Peter from comment #5) > Okay moving this out of the 4.1.6 release and into 4.2.0 4.2.0 already has all these fixes. We just need to get older Java versions detected in 4.1.x. like they are in trunk.
I don't know why this is affecting Java 8 detection. Almost all of the changes are to comments in .java files that are only parsed to produce documentation for ODK. Unfortunately I can't test Java 7 on Windows since I don't have it installed in my Win7 VM and it is no longer downloadable.
(In reply to Don Lewis from comment #8) > I don't know why this is affecting Java 8 detection. Almost all of the > changes are to comments in .java files that are only parsed to produce > documentation for ODK. Something might be missing, as detection works in trunk. > Unfortunately I can't test Java 7 on Windows since I don't have it installed > in my Win7 VM and it is no longer downloadable. https://www.oracle.com/technetwork/java/javase/downloads/java-archive-downloads-javase7-521261.html
I just built and installed unmodified AOOO416 and it also doesn't detect JAVA7.
(In reply to Don Lewis from comment #10) > I just built and installed unmodified AOOO416 and it also doesn't detect > JAVA7. Did you build with Java 8 or Java 7?
Java 8. I'm starting a Java 7 build now.
BTW, the java detection code appears to be in jvmfwk, and that really hasn't changed in a long time.
If I build AOO416 with this patch and Java 7, then OpenOffice detects both Java 7 and Java 8. That sounds about like what we observed previously.
Yes, basically we seem to have another bug that building 4.1.x with Java 8 disables detection of Java 7 (and maybe older versions). The ODK patch itself is OK and can be applied after this is fixed. (Detection of Java 7 works in trunk, when AOO is built with Java 8)
The jvmfwk code appears to be identical between trunk and AOO416. The only thing that shows up in the output of diff are some typo fixes in comments.
Committed to AOO418 with: https://github.com/apache/openoffice/pull/95