dev-java/javacup: Use installed javacup over bundled where possible
authorJames Le Cuirot <chewi@gentoo.org>
Wed, 18 Nov 2015 22:54:08 +0000 (22:54 +0000)
committerJames Le Cuirot <chewi@gentoo.org>
Wed, 18 Nov 2015 22:59:02 +0000 (22:59 +0000)
Fixes bug #565338.

Annoyingly javacup bundles an older version of itself that will break
jflex if that has already been built against this newer version
beforehand. Even more annoyingly, the binary download is built with
Java 8, which isn't much use to us. We therefore use any installed
javacup that is newer than the bundled version where possible. If this
approach turns out to be unworkable then we may just have to use the
bundled jflex for bootstrapping.

Package-Manager: portage-2.2.20.1

dev-java/javacup/javacup-0.11b_p20151001.ebuild

index ad8742f2b4c45fadfefb23a86dfd4e7088e83e97..b44a0da26d9ea05af8dd7c87ab62c541f4100741 100644 (file)
@@ -48,8 +48,23 @@ src_configure() {
 }
 
 src_compile() {
-       einfo "Bootstrapping with bundled javacup ..."
-       EANT_GENTOO_CLASSPATH_EXTRA+=":${S}/bin/${MY_P:0:-1}.jar" eant
+       # Annoyingly javacup bundles an older version of itself that will
+       # break jflex if that has already been built against this newer
+       # version beforehand. Even more annoyingly, the binary download is
+       # built with Java 8, which isn't much use to us. We therefore use
+       # any installed javacup that is newer than the bundled version where
+       # possible. If this approach turns out to be unworkable then we may
+       # just have to use the bundled jflex for bootstrapping.
+       if has_version \>=${CATEGORY}/${PN}-0.11b:${SLOT}; then
+               # Use PORTAGE_QUIET to suppress a QA warning that is spurious
+               # thanks to has_version above. This is Portage-specific but
+               # showing the warning elsewhere isn't the end of the world.
+               einfo "Bootstrapping with installed javacup ..."
+               EANT_GENTOO_CLASSPATH_EXTRA+=":$(PORTAGE_QUIET=1 java-pkg_getjars --build-only javacup)" eant
+       else
+               einfo "Bootstrapping with bundled javacup ..."
+               EANT_GENTOO_CLASSPATH_EXTRA+=":${S}/bin/${MY_P:0:-1}.jar" eant
+       fi
 
        # Clean everything except the new jar.
        rm -rv java/ classes/ || die