Compiling openoffice-2.0.2-r1 fails when using the java migration overlay. The ebuild should inherit java-pkg and call java-pkg_pkg_setup in the function pkg-setup.
Created attachment 87332 [details] fixed ebuild using java-pkg eclass Fixed ebuild using java-pkg eclass. It's still compiling so I can't be sure, but it looks good.
We don't support overlays... Also, please attach a unified diff instead of a huge ebuild.
All that needs to happen is for someone to add java-pkg to the inherited eclasses at the top of the ebuild. The in pkg_setup add somewhere use java && java-pkg_pkg_setup A patch can be made, but very minor changes. One should not use JAVA_HOME without inheriting java-pkg, and calling java-pkg_pkg_setup.
Created attachment 87334 [details, diff] diff against the original ebuild
My experience is that openoffice emerge fails with migration overlay if jdk 1.5 is set as generation-2 system vm, because the ebuild doesn't determine build vm and probably somewhere executes directly "javac" without explicit path to 1.4 jdk (although the ebuild says at some place that it's using 1.4 jdk path) and thus calls 1.5 javac because of 1.5 system vm. It doesn't fail with 1.4 system vm.
Well if it was a proper generation 1 ebuild then that would not be the case. Thus it needs to inherit java-pkg, and call java-pkg_pkg_setup. Now once the java migraiton overlay has been merged with portage. In the upcomming weeks. Then all java related ebuilds will need to be updated to a generation 2 ebuild. Which basically is a difference of java-pkg-2 vs just java-pkg. Since it is just calling ${JAVA_HOME} on a system using migration overlay, it will use what ever that is set to. Usually a generation 2 jvm/jdk, >= 1.5.
Jan, This diff is not unified. As such it can't be used to patch the ebuild. I do agree though that the java eclasses will have to be used.
Created attachment 87364 [details, diff] diff against the original ebuild openoffice-2.0.2-r1 sorry, I made a unified one. This is diffed against -r1, there is a -r2 in ~x86, if necessary I also want to create a diff for that.
I've fixed this now in -r2, thanks for reporting, closing
