[patch] proposed patch

java-utils-2.eclass-subshells.patch (text/plain), 3.97 KB, created by Vlastimil Babka (Caster) (RETIRED) on 2006-09-08 04:50:53 UTC

--- java-utils-2.eclass.orig	2006-09-08 10:24:15.000000000 +0200
+++ java-utils-2.eclass	2006-09-08 13:42:25.000000000 +0200
@@ -1156,7 +1156,7 @@
 java-pkg_get-javac() {
 	debug-print-function ${FUNCNAME} $*
 
-	java-pkg_init-compiler_
+
 	local compiler="${GENTOO_COMPILER}"
 
 	local compiler_executable
@@ -1174,16 +1174,19 @@
 			compiler_executable="$(source ${compiler_env} 1>/dev/null 2>&1; echo ${JAVAC})"
 			export JAVAC=${old_javac}
 
-			[[ -z ${compiler_executable} ]] && die "JAVAC is empty or undefined in ${compiler_env}"
+			if [[ -z ${compiler_executable} ]]; then
+				echo "JAVAC is empty or undefined in ${compiler_env}"
+				return 1
+			fi
 	
 			# check that it's executable
 			if [[ ! -x ${compiler_executable} ]]; then
-				eerror "Could not find ${compiler_executable}!"
-				die "${compiler_executable} doesn't exist, or isn't executable"
+				echo "${compiler_executable} doesn't exist, or isn't executable"
+				return 1
 			fi
 		else
-			eerror "Could not find environment file for ${compiler}"
-			die "Could not find ${compiler_env}"
+			echo "Could not find environment file for ${compiler}"
+			return 1
 		fi
 	fi
 	echo ${compiler_executable}
@@ -1210,8 +1213,9 @@
 	debug-print "want target: ${want_target}"
 
 	if [[ -z "${want_source}" || -z "${want_target}" ]]; then
-		debug-print "could not find valid -source/-target values"
-		die "Could not find valid -source/-target values"
+		debug-print "could not find valid -source/-target values for javac"
+		echo "Could not find valid -source/-target values for javac"
+		return 1
 	else
 		if java-pkg_is-vm-version-ge "1.4"; then
 			echo "${source_str} ${target_str}"
@@ -1326,12 +1330,21 @@
 ejavac() {
 	debug-print-function ${FUNCNAME} $*
 
-	# FIXME using get-javac ends up printing stuff with einfo
-#	local compiler_executable=$(java-pkg_get-javac)
-	local compiler_executable="javac"
+	java-pkg_init-compiler_
 
-	[[ -n ${JAVA_PKG_DEBUG} ]] && echo ${compiler_executable} $(java-pkg_javac-args) "${@}"
-	${compiler_executable} $(java-pkg_javac-args) "${@}" || die "ejavac failed"
+	local compiler_executable
+	compiler_executable=$(java-pkg_get-javac)
+	if [[ ! ${?} ]]; then
+		eerror "${compiler_executable}"
+		die "get-javac failed"
+	fi
+
+	local javac_args
+	javac_args="$(java-pkg_javac-args)"
+	[[ ${?} ]] || die "java-pkg_javac-args failed"
+
+	[[ -n ${JAVA_PKG_DEBUG} ]] && echo ${compiler_executable} ${javac_args} "${@}"
+	${compiler_executable} ${javac_args} "${@}" || die "ejavac failed"
 }
 
 # ------------------------------------------------------------------------------
@@ -1780,9 +1793,14 @@
 		elif [[ "${JAVA_PKG_ALLOW_VM_CHANGE}" == "yes" ]]; then
 			debug-print "depend-java-query:  NV_DEPEND:	${JAVA_PKG_NV_DEPEND:-${DEPEND}} VNEED: ${JAVA_PKG_VNEED}"
 			if [[ -n ${JAVA_PKG_VNEED} ]]; then
-				export GENTOO_VM="$(depend-java-query --need-virtual "${JAVA_PKG_VNEED}" --get-vm "${JAVA_PKG_NV_DEPEND:-${DEPEND}}")"
+				GENTOO_VM="$(depend-java-query --need-virtual "${JAVA_PKG_VNEED}" --get-vm "${JAVA_PKG_NV_DEPEND:-${DEPEND}}")"
+			else
+				GENTOO_VM="$(depend-java-query --get-vm "${JAVA_PKG_NV_DEPEND:-${DEPEND}}")"
+			fi
+			if [[ -z "${GENTOO_VM}" || "${GENTOO_VM}" == "None" ]]; then
+				die "Unable to determine VM for building from dependencies. NV_DEPEND: ${JAVA_PKG_NV_DEPEND:-${DEPEND}} VNEED: ${JAVA_PKG_VNEED}"
 			else
-				export GENTOO_VM="$(depend-java-query --get-vm "${JAVA_PKG_NV_DEPEND:-${DEPEND}}")"
+				export GENTOO_VM
 			fi
 		# otherwise just make sure the current VM is sufficient
 		else
@@ -1794,8 +1812,10 @@
 
 		export JAVA=$(java-config --java)
 		export JAVAC=$(java-config --javac)
-		export JAVACFLAGS="$(java-pkg_javac-args)" 
-		[[ -n ${JAVACFLAGS_EXTRA} ]] && export JAVACFLAGS="${JAVACFLAGS_EXTRA} ${JAVACFLAGS}"
+		JAVACFLAGS="$(java-pkg_javac-args)"
+		[[ ${?} ]] || die "java-pkg_javac-args failed"
+		[[ -n ${JAVACFLAGS_EXTRA} ]] && JAVACFLAGS="${JAVACFLAGS_EXTRA} ${JAVACFLAGS}"
+		export JAVACFLAGS
 
 		export JAVA_HOME="$(java-config -g JAVA_HOME)"
 		export JDK_HOME=${JAVA_HOME}