ports/167903: [PATCH] java/openjdk6: Fix build failure in 8.3
jail running on 9+ system
Jung-uk Kim
jkim at FreeBSD.org
Tue May 15 17:48:53 UTC 2012
On 2012-05-15 01:40:05 -0400, Bryan Drewery wrote:
> The following reply was made to PR ports/167903; it has been noted by GNATS.
>
> From: Bryan Drewery <bryan at shatow.net>
> To: bug-followup at FreeBSD.org
> Cc:
> Subject: Re: ports/167903: [PATCH] java/openjdk6: Fix build failure in 8.3
> jail running on 9+ system
> Date: Tue, 15 May 2012 00:39:06 -0500
>
> This is a multi-part message in MIME format.
> --------------060109060400070605040004
> Content-Type: text/plain; charset=ISO-8859-1
> Content-Transfer-Encoding: 7bit
>
> This should be LIB_DEPENDS+=. This patch is correct.
>
> Bryan
>
> --------------060109060400070605040004
> Content-Type: text/plain; charset=windows-1252;
> name="patch-openjdk16-compat7.txt"
> Content-Transfer-Encoding: 7bit
> Content-Disposition: attachment;
> filename="patch-openjdk16-compat7.txt"
>
> --- Makefile.orig 2012-05-15 00:16:27.000000000 -0500
> +++ Makefile 2012-05-15 00:34:09.000000000 -0500
> @@ -78,6 +78,10 @@
>
> .include <bsd.port.pre.mk>
>
> +.if ${OSVERSION} >= 800105
> +LIB_DEPENDS+= z.4:${PORTSDIR}/misc/compat7x
> +.endif
> +
> .if defined(WITH_IPV6)
> CATEGORIES+= ipv6
> .endif
I don't think this patch is acceptable because it introduces unnecessary
dependency for "normal" case.
In fact, the patch in 138638 was incomplete and it had to add this:
--- diablo-jdk16/Makefile.orig 2012-05-15 13:19:41.000000000 -0400
+++ diablo-jdk16/Makefile 2012-05-15 13:42:23.000000000 -0400
@@ -44,7 +44,7 @@
SUB_LIST+= ARCH="${ARCH}" \
JDK_VERSION="${JDK_VERSION}"
-PKG_IGNORE_DEPENDS= '^[^x|^j]'
+PKG_IGNORE_DEPENDS= '^[^c|^j|^x]'
INSTALL_DIR= ${PREFIX}/${PKGNAMEPREFIX}jdk${JDK_VERSION}
After this change, we have:
# pkg_info -r diablo-jdk-1.6.0.07.02_19
Information for diablo-jdk-1.6.0.07.02_19:
Depends on:
Dependency: xproto-7.0.22
Dependency: xextproto-7.2.0
Dependency: compat7x-amd64-7.3.703000.201008_1
Dependency: javavmwrapper-2.3.5
Dependency: java-zoneinfo-2012.c
Jung-uk Kim
More information about the freebsd-java
mailing list