--- 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}