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

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

commit 75a4fab30615827b32139b0c97640e2ed2ab4475
Author: Emmanuel Bourg <ebo...@apache.org>
Date:   Tue Nov 28 13:22:27 2017 +0100

    Depend on libasm-java (>= 5.0) instead of libasm3-java (Closes: #800850)
---
 debian/ant.properties                     |   2 +-
 debian/changelog                          |   1 +
 debian/control                            |   4 +-
 debian/patches/03_asm_compatibility.patch | 335 ++++++++++++++++++++++++++++++
 debian/patches/series                     |   1 +
 5 files changed, 340 insertions(+), 3 deletions(-)

diff --git a/debian/ant.properties b/debian/ant.properties
index a51ca88..8535ef2 100644
--- a/debian/ant.properties
+++ b/debian/ant.properties
@@ -1,5 +1,5 @@
 jcommjar = /usr/share/java/RXTXcomm.jar
 servletjar = /usr/share/java/servlet-api-3.1.jar
-bytecodelib = /usr/share/java/asm3.jar:/usr/share/java/asm3-commons.jar
+bytecodelib = /usr/share/java/asm.jar:/usr/share/java/asm-commons.jar
 junitlib = /usr/share/java/junit4.jar
 BUILD_DIR = ./jar-build
diff --git a/debian/changelog b/debian/changelog
index 3aa8044..da7015f 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,6 +1,7 @@
 jtharness (5.0-2) UNRELEASED; urgency=medium
 
   * Team upload.
+  * Depend on libasm-java (>= 5.0) instead of libasm3-java (Closes: #800850)
   * Replaced 02_dependencies.patch with debian/ant.properties
 
  -- Emmanuel Bourg <ebo...@apache.org>  Tue, 28 Nov 2017 13:09:07 +0100
diff --git a/debian/control b/debian/control
index d8b212d..7e0242e 100644
--- a/debian/control
+++ b/debian/control
@@ -9,7 +9,7 @@ Build-Depends-Indep: default-jdk,
                      ant,
                      junit4,
                      librxtx-java,
-                     libasm3-java,
+                     libasm-java (>= 5.0),
                      libservlet3.1-java
 Standards-Version: 4.1.0
 Vcs-Git: https://anonscm.debian.org/git/pkg-java/jtharness.git
@@ -20,7 +20,7 @@ Package: libjtharness-java
 Architecture: all
 Depends: junit4,
          librxtx-java,
-         libasm3-java,
+         libasm-java (>= 5.0),
          libservlet3.1-java,
          ${java:Depends},
          ${misc:Depends}
diff --git a/debian/patches/03_asm_compatibility.patch 
b/debian/patches/03_asm_compatibility.patch
new file mode 100644
index 0000000..f04c3ec
--- /dev/null
+++ b/debian/patches/03_asm_compatibility.patch
@@ -0,0 +1,335 @@
+
+# HG changeset patch
+# User afedorch
+# Date 1508808304 25200
+# Node ID df5f3f3fe4b3be828db3570506d35a7c3f8a6f6f
+# Parent  a5dc7a40b3aa86189524b50e742fd0e9004994df
+CODETOOLS-7901981 Update asm dependency, contributed by: erik.he...@oracle.com
+
+diff -r a5dc7a40b3aa -r df5f3f3fe4b3 
src/com/sun/javatest/junit/JUnitAnnotationTestFinder.java
+--- a/src/com/sun/javatest/junit/JUnitAnnotationTestFinder.java        Mon Oct 
23 17:51:16 2017 -0700
++++ b/src/com/sun/javatest/junit/JUnitAnnotationTestFinder.java        Mon Oct 
23 18:25:04 2017 -0700
+@@ -33,11 +33,8 @@
+ import java.util.HashMap;
+ 
+ import org.objectweb.asm.AnnotationVisitor;
+-import org.objectweb.asm.Attribute;
+ import org.objectweb.asm.ClassReader;
+ import org.objectweb.asm.ClassVisitor;
+-import org.objectweb.asm.FieldVisitor;
+-import org.objectweb.asm.Label;
+ import org.objectweb.asm.MethodVisitor;
+ import org.objectweb.asm.Opcodes;
+ 
+@@ -59,8 +56,7 @@
+  * @see com.sun.javatest.TestFinder
+  * @see com.sun.javatest.TestDescription
+  */
+-public class JUnitAnnotationTestFinder extends JUnitTestFinder
+-        implements ClassVisitor, MethodVisitor {
++public class JUnitAnnotationTestFinder extends JUnitTestFinder {
+     /**
+      * Constructs the list of file names to exclude for pruning in the search
+      * for files to examine for test descriptions.
+@@ -195,7 +191,7 @@
+             try {
+                 FileInputStream fis = new FileInputStream(classFile);
+                 ClassReader cr = new ClassReader(fis);
+-                cr.accept(this, 0);
++                cr.accept(new JUnitAnnotationClassVisitor(this), 0);
+                 // action happens in visit(...) below
+ 
+                 // methods are necessary for this to be a test
+@@ -227,114 +223,46 @@
+         return;
+     }
+ 
+-    //----------ASM interface -----------
+-    public void visit(int version, int access, String name, String signature,
+-            String superName, String[] interfaces) {
+-        if (verbose)
+-            System.out.println("found class " + name + " with super " 
+superName);
++    class JUnitAnnotationMethodVisistor extends MethodVisitor {
++        private JUnitAnnotationTestFinder outer;
++
++        public JUnitAnnotationMethodVisistor(JUnitAnnotationTestFinder outer) 
{
++            super(Opcodes.ASM4);
++            this.outer = outer;
++        }
++
++        public AnnotationVisitor visitAnnotation(String string, boolean b) {
++            if (outer.methodAnnotation.equals(string))
++                outer.foundTestMethod(outer.currMethod);
++
++            return null;
++        }
+     }
+ 
+-    // class visitor methods we are interested in
+-    public MethodVisitor visitMethod(int access, String name, String desc,
+-            String signature, String[] exceptions) {
+-        if (access == Opcodes.ACC_PUBLIC) {
+-            currMethod = name;
+-            return this;
+-        } else
+-            return null;
+-    }
++    class JUnitAnnotationClassVisitor extends ClassVisitor {
++        private JUnitAnnotationTestFinder outer;
++        private JUnitAnnotationMethodVisistor methodVisitor;
+ 
+-    public void visitSource(String string, String string0) {
+-    }
++        public JUnitAnnotationClassVisitor(JUnitAnnotationTestFinder outer) {
++            super(Opcodes.ASM4);
++            this.outer = outer;
++            methodVisitor = new JUnitAnnotationMethodVisistor(outer);
++        }
+ 
+-    public void visitOuterClass(String string, String string0, String 
string1) {
+-    }
++        public void visit(int version, int access, String name, String 
signature,
++                          String superName, String[] interfaces) {
++            if (verbose)
++                System.out.println("found class " + name + " with super " + 
superName);
++        }
+ 
+-    public AnnotationVisitor visitAnnotation(String string, boolean b) {
+-        if (methodAnnotation.equals(string))
+-            foundTestMethod(currMethod);
+-
+-        return null;
+-    }
+-
+-    public void visitAttribute(Attribute attribute) {
+-    }
+-
+-    public void visitInnerClass(String string, String string0, String 
string1, int i) {
+-    }
+-
+-    public FieldVisitor visitField(int i, String string, String string0, 
String string1, Object object) {
+-        return null;
+-    }
+-
+-    public void visitEnd() {
+-    }
+-
+-
+-    // -- Method visitor --
+-    public void visitJumpInsn(int i, Label label) {
+-    }
+-
+-    public void visitLineNumber(int i, Label label) {
+-    }
+-
+-    public void visitLabel(Label label) {
+-    }
+-
+-    public void visitTableSwitchInsn(int i, int i0, Label label, Label[] 
label0) {
+-    }
+-
+-    public void visitLdcInsn(Object object) {
+-    }
+-
+-    public void visitFrame(int i, int i0, Object[] object, int i1, Object[] 
object0) {
+-    }
+-
+-    public void visitInsn(int i) {
+-    }
+-
+-    public void visitLookupSwitchInsn(Label label, int[] i, Label[] label0) {
+-    }
+-
+-    public void visitTryCatchBlock(Label label, Label label0, Label label1, 
String string) {
+-    }
+-
+-    public void visitTypeInsn(int i, String string) {
+-    }
+-
+-    public void visitMultiANewArrayInsn(String string, int i) {
+-    }
+-
+-    public void visitMethodInsn(int i, String string, String string0, String 
string1) {
+-    }
+-
+-    public void visitFieldInsn(int i, String string, String string0, String 
string1) {
+-    }
+-
+-    public void visitLocalVariable(String string, String string0, String 
string1, Label label, Label label0, int i) {
+-    }
+-
+-    public AnnotationVisitor visitParameterAnnotation(int i, String string, 
boolean b) {
+-        return null;
+-    }
+-
+-    public void visitVarInsn(int i, int i0) {
+-    }
+-
+-    public void visitMaxs(int i, int i0) {
+-    }
+-
+-    public AnnotationVisitor visitAnnotationDefault() {
+-        return null;
+-    }
+-
+-    public void visitCode() {
+-    }
+-
+-    public void visitIincInsn(int i, int i0) {
+-    }
+-
+-    public void visitIntInsn(int i, int i0) {
++        public MethodVisitor visitMethod(int access, String name, String desc,
++                                         String signature, String[] 
exceptions) {
++            if (access == Opcodes.ACC_PUBLIC) {
++                outer.currMethod = name;
++                return methodVisitor;
++            } else
++                return null;
++        }
+     }
+ 
+     //----------member 
variables------------------------------------------------
+diff -r a5dc7a40b3aa -r df5f3f3fe4b3 
src/com/sun/javatest/junit/JUnitSuperTestFinder.java
+--- a/src/com/sun/javatest/junit/JUnitSuperTestFinder.java     Mon Oct 23 
17:51:16 2017 -0700
++++ b/src/com/sun/javatest/junit/JUnitSuperTestFinder.java     Mon Oct 23 
18:25:04 2017 -0700
+@@ -33,14 +33,10 @@
+ import java.util.ArrayList;
+ import java.util.HashMap;
+ 
+-import org.objectweb.asm.AnnotationVisitor;
+-import org.objectweb.asm.Attribute;
+ import org.objectweb.asm.ClassReader;
+ import org.objectweb.asm.ClassVisitor;
+-import org.objectweb.asm.FieldVisitor;
+ import org.objectweb.asm.MethodVisitor;
+ import org.objectweb.asm.Opcodes;
+-import org.objectweb.asm.commons.EmptyVisitor;
+ 
+ import com.sun.javatest.util.I18NResourceBundle;
+ 
+@@ -56,7 +52,7 @@
+  * @see com.sun.javatest.TestFinder
+  * @see com.sun.javatest.TestDescription
+  */
+-public class JUnitSuperTestFinder extends JUnitTestFinder implements 
ClassVisitor {
++public class JUnitSuperTestFinder extends JUnitTestFinder {
+     /**
+      * Constructs the list of file names to exclude for pruning in the search
+      * for files to examine for test descriptions.
+@@ -201,7 +197,7 @@
+             try {
+                 FileInputStream fis = new FileInputStream(classFile);
+                 ClassReader cr = new ClassReader(fis);
+-                cr.accept(this, 0);
++                cr.accept(new JUnitClassVisitor(this), 0);
+                 // action happens in visit(...) below
+ 
+                 if (tdValues.get("executeClass") == null)
+@@ -275,63 +271,47 @@
+         else
+             return false;
+     }
+-//----------ASM inteface-----------
+-    public void visit(int version, int access, String name, String signature,
+-            String superName, String[] interfaces) {
+-        if (verbose)
+-            System.out.println("found class " + name + " with super " 
+superName);
+ 
+-        if (isMatchSuper(superName.replaceAll("/", "."))) {
++    class JUnitClassVisitor extends ClassVisitor {
++        private JUnitSuperTestFinder outer;
+ 
+-            //tdValues.put("sources", sources);
+-            tdValues.put("executeClass", name.replaceAll("/", "."));
+-        } else {
++        JUnitClassVisitor(JUnitSuperTestFinder outer) {
++            super(Opcodes.ASM4);
++            this.outer = outer;
++        }
+ 
++        public void visit(int version, int access, String name, String 
signature,
++                          String superName, String[] interfaces) {
++            if (verbose)
++                System.out.println("found class " + name + " with super " + 
superName);
++
++            if (outer.isMatchSuper(superName.replaceAll("/", "."))) {
++                outer.tdValues.put("executeClass", name.replaceAll("/", "."));
++            }
++        }
++
++        /**
++         * Looks for methods which are test methods by calling 
<tt>isTestMethodSignature</tt>.
++         * You can override that method or this one.  If overriding this one,
++         * use foundTestMethod(String) to register any test methods which you
++         * find.
++         */
++        public MethodVisitor visitMethod(int access, String name, String desc,
++                                         String signature, String[] 
exceptions) {
++            if (access == Opcodes.ACC_PUBLIC) {
++                if (outer.isTestMethodSignature(name))
++                    outer.foundTestMethod(name);
++            }
++
++            return null;
+         }
+     }
+ 
+-// class visitor methods we are interested in
+-    /**
+-     * Looks for methods which are test methods by calling 
<tt>isTestMethodSignature</tt>.
+-     * You can override that method or this one.  If overriding this one,
+-     * use foundTestMethod(String) to register any test methods which you
+-     * find.
+-     */
+-    public MethodVisitor visitMethod(int access, String name, String desc,
+-            String signature, String[] exceptions) {
+-        if (access == Opcodes.ACC_PUBLIC) {
+-            //System.out.println("found method " + name);
+-            if (isTestMethodSignature(name))
+-                foundTestMethod(name);
++    private static class MethodFinderVisitor extends ClassVisitor {
++
++        public MethodFinderVisitor() {
++            super(Opcodes.ASM4);
+         }
+-
+-        return null;
+-    }
+-
+-    public void visitSource(String string, String string0) {
+-    }
+-
+-    public void visitOuterClass(String string, String string0, String 
string1) {
+-    }
+-
+-    public AnnotationVisitor visitAnnotation(String string, boolean b) {
+-        return null;
+-    }
+-
+-    public void visitAttribute(Attribute attribute) {
+-    }
+-
+-    public void visitInnerClass(String string, String string0, String 
string1, int i) {
+-    }
+-
+-    public FieldVisitor visitField(int i, String string, String string0, 
String string1, Object object) {
+-        return null;
+-    }
+-
+-    public void visitEnd() {
+-    }
+-
+-    private static class MethodFinderVisitor extends EmptyVisitor {
+         /**
+          * Return the given class' superclass name in dotted notation.
+          */
+@@ -349,6 +329,7 @@
+     }
+ 
+ 
++
+ //----------member variables------------------------------------------------
+ 
+     protected ArrayList<String> requiredSuperclass = new ArrayList();
+
diff --git a/debian/patches/series b/debian/patches/series
index 6e1ea93..79f8934 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -1 +1,2 @@
 01_imports.patch
+03_asm_compatibility.patch

-- 
Alioth's /usr/local/bin/git-commit-notice on 
/srv/git.debian.org/git/pkg-java/jtharness.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