java-utils-2.eclass patch. Support for BUILD_DEPEND being recorded within package.env.

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

java-utils-2.eclass patch. Support for BUILD_DEPEND being recorded within package.env.

Alistair Bush-2
Firstly, fellow developer please review this eclass patch and read on if
you are interested in what it actually does.

Java developers:

The following patch adds 3 new values to our package.env

PVR and CATEGORY being the easy ones.  These are being added because I
think they should be there and they will help with implementing a
paludis re-emerge-everything-java script.

the 3rd BUILD_DEPEND records the packages/jars that have been passed as
parameters to our java functions ( jar-from, getjars, getjar ) and also
have --build-only specified.  The format is exactly like package.env's
DEPEND variable. The main reason for this patch is to allow Serkan to
add --build-only dependencies to the classpath for java-dep-check.
java-config support is not currently planned but this may change if need be.

Here are some example package.env files.

# more
/var/tmp/portage/app-office/hourglass-0.7.2/image/usr/share/hourglass/package.env
DESCRIPTION="A PSP (personal software process) time tracking utility
written in Java"
GENERATION="2"
SLOT="0"
CATEGORY="app-office"
PVR="0.7.2"
CLASSPATH="/usr/share/hourglass/lib/hourglass.jar"
DEPEND="log4j.jar@log4j:jcommon.jar@jcommon-1.0:jdom.jar@jdom-1.0"
VM=">=virtual/jre-1.5"
BUILD_DEPEND="ant.jar@ant-core"
TARGET="1.5"
SOURCE="1.5"
JAVADOC_PATH="/usr/share/doc/hourglass-0.7.2/html/api"
JAVA_SOURCES="/usr/share/hourglass/sources//hourglass-src.zip"
MERGE_VM="sun-jdk-1.7"
MERGE_COMPILER="javac"

 # more /var/tmp/portage/dev-java/sbaz-1.25/image/usr/share/sbaz/package.env
DESCRIPTION="A system used by Scala enthusiasts to share computer files
with each other."
GENERATION="2"
SLOT="0"
CATEGORY="dev-java"
PVR="1.25"
CLASSPATH="/usr/share/sbaz/lib/sbaz.jar"
DEPEND="scala-library.jar@scala"
VM=">=virtual/jre-1.5"
BUILD_DEPEND="servlet-api.jar@servlet-api-2.4:ant-contrib"
TARGET="1.5"
SOURCE="1.5"
MERGE_VM="sun-jdk-1.7"
MERGE_COMPILER="javac

--- java-utils-2.eclass.old 2009-06-04 19:46:37.711668962 +1200
+++ java-utils-2.eclass 2009-06-04 22:13:22.694684085 +1200
@@ -874,6 +874,7 @@
  local destdir="."
  local deep=""
  local virtual=""
+ local record_jar=""
 
  [[ "${EBUILD_PHASE}" == "test" ]] && build_only="build"
 
@@ -918,7 +919,7 @@
  [[ -z "${build_only}" ]] && java-pkg_record-jar_ "${pkg}"
  done
  # setting this disables further record-jar_ calls later
- build_only="build"
+ record_jar="true"
  else
  java-pkg_ensure-dep "${build_only}" "${target_pkg}"
  fi
@@ -928,7 +929,7 @@
  if [[ -z "${build_only}" && -n "${virtual}" ]]; then
  java-pkg_record-jar_ "${target_pkg}"
  # setting this disables further record-jars_ calls later
- build_only="build"
+ record_jar="true"
  fi
 
  pushd ${destdir} > /dev/null \
@@ -946,13 +947,25 @@
  [[ -f "${target_jar}" ]]  && rm "${target_jar}"
  ln -snf "${jar}" \
  || die "Failed to make symlink from ${jar} to ${jar_name}"
- [[ -z "${build_only}" ]] && java-pkg_record-jar_ "${target_pkg}" "${jar}"
- # otherwise, if the current jar is the target jar, link it
+ if [[ -z "${record_jar}" ]]; then
+ if [[ -z "${build_only}" ]]; then
+ java-pkg_record-jar_ "${target_pkg}" "${jar}"
+ else
+ java-pkg_record-jar_ --build-only "${target_pkg}" "${jar}"
+ fi
+ fi
+ # otherwise, if the current jar is the target jar, link it
  elif [[ "${jar_name}" == "${target_jar}" ]] ; then
  [[ -f "${destjar}" ]]  && rm "${destjar}"
  ln -snf "${jar}" "${destjar}" \
  || die "Failed to make symlink from ${jar} to ${destjar}"
- [[ -z "${build_only}" ]] && java-pkg_record-jar_ "${target_pkg}" "${jar}"
+ if [[ -z "${record_jar}" ]]; then
+ if [[ -z "${build_only}" ]]; then
+ java-pkg_record-jar_ "${target_pkg}" "${jar}"
+ else
+ java-pkg_record-jar_ --build-only "${target_jar}" "${jar}"
+ fi
+ fi
  popd > /dev/null
  return 0
  fi
@@ -1035,12 +1048,13 @@
  java-pkg_ensure-dep "${build_only}" "${pkg}"
  done
 
- # Only record jars that aren't build-only
- if [[ -z "${build_only}" ]]; then
- for pkg in ${pkgs//,/ }; do
+ for pkg in ${pkgs//,/ }; do
+ if [[ -z "${build_only}" ]]; then
  java-pkg_record-jar_ "${pkg}"
- done
- fi
+ else
+ java-pkg_record-jar_ --build-only "${pkg}"
+ fi
+ done
 
  echo "${jars}"
 }
@@ -1071,6 +1085,7 @@
 
  local build_only=""
  local virtual=""
+ local record_jar=""
 
  [[ "${EBUILD_PHASE}" == "test" ]] && build_only="build"
 
@@ -1106,8 +1121,12 @@
  # Record the package(Virtual) as a dependency and then set build_only
  # So that individual jars are not recorded.
  if [[ -n "${virtual}" ]]; then
- java-pkg_record-jar_ "${pkg}"
- build_only="true"
+ if [[ -z "${build_only}" ]]; then
+ java-pkg_record-jar_ "${pkg}"
+ else
+ java-pkg_record-jar_ --build-only "${pkg}"
+ fi
+ record_jar="true"
  fi
 
  for jar in ${classpath//:/ }; do
@@ -1117,7 +1136,13 @@
 
  if [[ "$(basename ${jar})" == "${target_jar}" ]] ; then
  # Only record jars that aren't build-only
- [[ -z "${build_only}" ]] && java-pkg_record-jar_ "${pkg}" "${jar}"
+ if [[ -z "${record_jar}" ]]; then
+ if [[ -z "${build_only}" ]]; then
+ java-pkg_record-jar_ "${pkg}" "${jar}"
+ else
+ java-pkg_record-jar_ --build-only "${pkg}" "${jar}"
+ fi
+ fi
  echo "${jar}"
  return 0
  fi
@@ -2243,6 +2268,8 @@
  echo "DESCRIPTION=\"${DESCRIPTION}\""
  echo "GENERATION=\"2\""
  echo "SLOT=\"${SLOT}\""
+ echo "CATEGORY=\"${CATEGORY}\""
+ echo "PVR=\"${PVR}\""
 
  [[ -n "${JAVA_PKG_CLASSPATH}" ]] && echo "CLASSPATH=\"${JAVA_PKG_CLASSPATH}\""
  [[ -n "${JAVA_PKG_LIBRARY}" ]] && echo "LIBRARY_PATH=\"${JAVA_PKG_LIBRARY}\""
@@ -2252,6 +2279,8 @@
  [[ -f "${JAVA_PKG_OPTIONAL_DEPEND_FILE}" ]] \
  && echo "OPTIONAL_DEPEND=\"$(cat "${JAVA_PKG_OPTIONAL_DEPEND_FILE}" | uniq | tr '\n' ':')\""
  echo "VM=\"$(echo ${RDEPEND} ${DEPEND} | sed -e 's/ /\n/g' | sed -n -e '/virtual\/\(jre\|jdk\)/ { p;q }')\"" # TODO cleanup !
+ [[ -f "${JAVA_PKG_BUILD_DEPEND_FILE}" ]] \
+ && echo "BUILD_DEPEND=\"$(cat "${JAVA_PKG_BUILD_DEPEND_FILE}" | uniq | tr '\n' ':')\""
  ) > "${JAVA_PKG_ENV}"
 
  # register target/source
@@ -2294,20 +2323,26 @@
 #
 # Record an (optional) dependency to the package.env
 # @param --optional - record dependency as optional
+# @param --build - record dependency as build_only
 # @param $1 - package to record
 # @param $2 - (optional) jar of package to record
 # ------------------------------------------------------------------------------
 JAVA_PKG_DEPEND_FILE="${T}/java-pkg-depend"
 JAVA_PKG_OPTIONAL_DEPEND_FILE="${T}/java-pkg-optional-depend"
+JAVA_PKG_BUILD_DEPEND_FILE="${T}/java-pkg-build-depend"
 
 java-pkg_record-jar_() {
  debug-print-function ${FUNCNAME} $*
 
  local depend_file="${JAVA_PKG_DEPEND_FILE}"
- if [[ "${1}" == "--optional" ]]; then
- depend_file="${JAVA_PKG_OPTIONAL_DEPEND_FILE}"
- shift
- fi
+ case "${1}" in
+ "--optional") depend_file="${JAVA_PKG_OPTIONAL_DEPEND_FILE}"; shift;;
+ "--build-only") depend_file="${JAVA_PKG_BUILD_DEPEND_FILE}"; shift;;
+ esac
+ #if [[ "${1}" == "--optional" ]]; then
+ # depend_file="${JAVA_PKG_OPTIONAL_DEPEND_FILE}"
+ # shift
+ #fi
 
  local pkg=${1} jar=${2} append
  if [[ -z "${jar}" ]]; then