This is an automated email from the git hooks/post-receive script.

ebourg-guest pushed a commit to branch master
in repository java-wrappers.

commit cc665e7a5080a2de68720120b9a834247aa3d092
Author: Vincent Fourmond <fourm...@debian.org>
Date:   Mon Apr 4 19:10:52 2011 +0000

    [java-wrappers] Severely trim down the list of valid JVMs + ensure the 
likes of find_java_runtime default-jre are spotted in due time
---
 debian/changelog |  8 ++++++--
 java-wrappers.sh | 14 +++++++++-----
 jvm-list.sh      | 24 ++++++------------------
 3 files changed, 21 insertions(+), 25 deletions(-)

diff --git a/debian/changelog b/debian/changelog
index 446caf9..cd284a5 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -2,9 +2,13 @@ java-wrappers (0.1.21) experimental; urgency=low
 
   * Final form of the find_jar_classpath function that now only performs
     path expansion for the argument, but not for the elements found in the
-    manifests that should be full paths anyway.
+    manifests that should be full paths anyway (closes: #563197)
+  * Trim down significantly the number of JVMs available, as fortunately
+    the dozens-of-jvms dark age is long gone (closes: #612903)
+  * Make it sure that mistakes such as find_java_runtime default-jre won't
+    go unnoticed.
 
- -- Vincent Fourmond <fourm...@debian.org>  Mon, 04 Apr 2011 20:31:36 +0200
+ -- Vincent Fourmond <fourm...@debian.org>  Mon, 04 Apr 2011 21:08:31 +0200
 
 java-wrappers (0.1.20) experimental; urgency=low
 
diff --git a/java-wrappers.sh b/java-wrappers.sh
index 033dec4..0ae6957 100644
--- a/java-wrappers.sh
+++ b/java-wrappers.sh
@@ -105,13 +105,17 @@ find_java_runtime() {
        
        while test "$1"
        do 
-           found_jvm=$(eval "echo -n \$__jvm_$1")
-           if [ -z "$found_jvm" ]; then
-               java_warning "JVM flavor '${1:-none}' not understood"
+           if echo $1 | egrep -- '[^0-9A-Za-z_]'; then
+               java_warning "JVM flavor '${1}' contains illegal characters"
            else
-               DIRS="$DIRS $found_jvm"
-               shift
+               found_jvm=$(eval "echo -n \$__jvm_$1")
+               if [ -z "$found_jvm" ] ; then
+                   java_warning "JVM flavor '${1:-none}' not understood"
+               else
+                   DIRS="$DIRS $found_jvm"
+               fi
            fi
+           shift
        done
        # Nothing means everything
        if [ -z "$DIRS" ]; then
diff --git a/jvm-list.sh b/jvm-list.sh
index 383f9b2..d96b257 100644
--- a/jvm-list.sh
+++ b/jvm-list.sh
@@ -26,39 +26,27 @@
 __jvm_default="/usr/lib/jvm/default-java"
 
 # Sun runtimes
-__jvm_sun4="/usr/lib/j2*1.4-sun"
-__jvm_sun5="/usr/lib/jvm/java-1.5.0-sun /usr/lib/j2*1.5-sun"
 __jvm_sun6="/usr/lib/jvm/java-6-sun /usr/lib/j2*1.6-sun"
 
 # all sun runtimes
-__jvm_sun="$__jvm_sun4 $__jvm_sun5 $__jvm_sun6"
+__jvm_sun="$__jvm_sun6"
 # Sun version at least 5:
-__jvm_sunmin5="$__jvm_sun5 $__jvm_sun6"
-
-# IBM runtimes. This comes from argouml.sh
-# I'm not quite sure it is still useful, but, we never know !
-__jvm_ibm="/usr/lib/j2*1.[456]-ibm"
-
+__jvm_sunmin5="$__jvm_sun6"
 
 # Now, free runtimes:
 __jvm_gcj2="/usr/lib/jvm/java-*-gcj-4.* "
 __jvm_gcj5="/usr/lib/jvm/java-1.5*-gcj-4.* "
-__jvm_sablevm="/usr/lib/sablevm" # Probably should be dropped, only in etch
-__jvm_kaffe="/usr/lib/kaffe /usr/lib/kaffe/pthreads /usr/lib/kaffe/jthreads"
-__jvm_icedtea="/usr/lib/jvm/java-7-icedtea" # Probably should be dropped... 
-__jvm_cacao="/usr/lib/jvm/cacao"
 __jvm_openjdk6="/usr/lib/jvm/java-6-openjdk"
-__jvm_cacaooj6="/usr/lib/jvm/java-6-cacao/jre"
 
 # And a few aliases
 __jvm_gcj="$__jvm_gcj2"
-__jvm_openjdk="$__jvm_openjdk6 $__jvm_cacaooj6"
+__jvm_openjdk="$__jvm_openjdk6"
 
 # The java* runtimes:
 # -> corresponds to Provides: java6-runtime 
-__jvm_java6="$__jvm_openjdk6 $__jvm_cacaooj6 $__jvm_sun6"
+__jvm_java6="$__jvm_openjdk6 $__jvm_sun6"
 # -> corresponds to Provides: java5-runtime 
-__jvm_java5="$__jvm_gcj5 $__jvm_java6 $__jvm_sun5"
+__jvm_java5="$__jvm_gcj5 $__jvm_java6"
 # -> corresponds to Provides: java2-runtime 
 __jvm_java2="$__jvm_java5 $__jvm_gcj2"
 
@@ -66,6 +54,6 @@ __jvm_java2="$__jvm_java5 $__jvm_gcj2"
 __jvm_alt=$(readlink /etc/alternatives/java|sed -n 
's/\(\/usr\/lib\/jvm\/[^\/]*\)\/.*/\1/p')
 
 # All JVMs
-__jvm_all="$__jvm_default /usr/lib/jvm/* $__jvm_ibm $__jvm_sun4 $__jvm_sablevm 
$__jvm_kaffe"
+__jvm_all="$__jvm_default /usr/lib/jvm/*"
 
 # Probably here should come a few meaningful global aliases.

-- 
Alioth's /usr/local/bin/git-commit-notice on 
/srv/git.debian.org/git/pkg-java/java-wrappers.git

_______________________________________________
pkg-java-commits mailing list
pkg-java-comm...@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-java-commits

Reply via email to