Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 20 Dec 2013 04:55:56 GMT
From:      Phil Phillips <pphillips@experts-exchange.com>
To:        freebsd-gnats-submit@FreeBSD.org
Subject:   ports/185015: [PATCH] java/openjdk7: unlimited strength policy option not working
Message-ID:  <201312200455.rBK4tuNX078478@oldred.freebsd.org>
Resent-Message-ID: <201312200500.rBK50018011142@freefall.freebsd.org>

next in thread | raw e-mail | index | archive | help

>Number:         185015
>Category:       ports
>Synopsis:       [PATCH] java/openjdk7: unlimited strength policy option not working
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-ports-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Fri Dec 20 05:00:00 UTC 2013
>Closed-Date:
>Last-Modified:
>Originator:     Phil Phillips
>Release:        
>Organization:
Experts Exchange
>Environment:
>Description:
Building openjdk7 with the POLICY option enabled doesn't actually install the unlimited strength policy files.

The javax.crypto Makefile changed a little, so the post-patch command in the port is no longer modifying the Makefile to call the "install-unlimited" target.  Instead, the UNLIMITED_CRYPTO environment variable can be set.
>How-To-Repeat:
1) Install openjdk7 with POLICY option enabled
2) Run a java program that requires unlimited strength policy files
>Fix:
Patch attached.  I included a PORTREVISION bump with this change.

Patch attached with submission follows:

Index: Makefile
===================================================================
--- Makefile	(revision 336970)
+++ Makefile	(working copy)
@@ -3,6 +3,7 @@
 
 PORTNAME=	openjdk
 PORTVERSION=	${JDK_MAJOR_VERSION}.${PORT_MINOR_VERSION}.${PORT_BUILD_NUMBER}
+PORTREVISION=	1
 CATEGORIES=	java devel
 MASTER_SITES=	http://download.java.net/openjdk/jdk${JDK_MAJOR_VERSION}u${JDK_MINOR_VERSION}/promoted/b${JDK_BUILD_NUMBER}/ \
 		http://download.java.net/jaxp/1.4.5/:jaxp \
@@ -110,6 +111,10 @@
 MAKE_ENV+=	ALT_OBJCOPY="/nonexistent"
 .endif
 
+.if ${PORT_OPTIONS:MPOLICY}
+MAKE_ENV+=	UNLIMITED_CRYPTO=1
+.endif
+
 .if ${PORT_OPTIONS:MTZUPDATE}
 RUN_DEPENDS+=  ${LOCALBASE}/share/java/zi:${PORTSDIR}/java/java-zoneinfo
 .endif
@@ -135,10 +140,6 @@
 post-patch:
 	@${REINPLACE_CMD} "s|%%LOCALBASE%%|${LOCALBASE}|" \
 		${WRKSRC}/hotspot/src/os/bsd/vm/os_bsd.cpp
-.if ${PORT_OPTIONS:MPOLICY}
-	@${REINPLACE_CMD} 's|build-policy install-limited|build-policy install-unlimited|' \
-		${WRKSRC}/jdk/make/javax/crypto/Makefile
-.endif
 .if empty(ICONV_LIB)
 	@${REINPLACE_CMD} 's| -liconv||' \
 		${WRKSRC}/Makefile \


>Release-Note:
>Audit-Trail:
>Unformatted:



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201312200455.rBK4tuNX078478>