[groovy] Fix FTBFS

Michal Srb msrb at fedoraproject.org
Mon Jun 16 07:25:14 UTC 2014


commit d2cb476a3c4fc8cb595204d7272df289f86927dd
Author: Michal Srb <msrb at redhat.com>
Date:   Mon Jun 16 09:24:45 2014 +0200

    Fix FTBFS

 groovy-build-with-java8.patch |   22 ++++++++++++++++++++++
 groovy.spec                   |   19 ++++++++-----------
 2 files changed, 30 insertions(+), 11 deletions(-)
---
diff --git a/groovy-build-with-java8.patch b/groovy-build-with-java8.patch
new file mode 100644
index 0000000..5c77860
--- /dev/null
+++ b/groovy-build-with-java8.patch
@@ -0,0 +1,22 @@
+diff --git a/src/main/org/codehaus/groovy/vmplugin/v5/Java5.java b/src/main/org/codehaus/groovy/vmplugin/v5/Java5.java
+index 21d213e..7945767 100644
+--- a/src/main/org/codehaus/groovy/vmplugin/v5/Java5.java
++++ b/src/main/org/codehaus/groovy/vmplugin/v5/Java5.java
+@@ -23,7 +23,16 @@ import org.codehaus.groovy.ast.stmt.ReturnStatement;
+ import org.codehaus.groovy.vmplugin.VMPlugin;
+ 
+ import java.lang.annotation.*;
+-import java.lang.reflect.*;
++import java.lang.reflect.Array;
++import java.lang.reflect.Constructor;
++import java.lang.reflect.Field;
++import java.lang.reflect.GenericArrayType;
++import java.lang.reflect.InvocationTargetException;
++import java.lang.reflect.Method;
++import java.lang.reflect.ParameterizedType;
++import java.lang.reflect.Type;
++import java.lang.reflect.TypeVariable;
++import java.lang.reflect.WildcardType;
+ import java.util.List;
+ 
+ /**
diff --git a/groovy.spec b/groovy.spec
index 55c1025..a2691d0 100644
--- a/groovy.spec
+++ b/groovy.spec
@@ -4,7 +4,7 @@
 
 Name:           groovy
 Version:        1.8.9
-Release:        10%{?dist}
+Release:        11%{?dist}
 Summary:        Dynamic language for the Java Platform
 
 Group:          Development/Languages
@@ -23,6 +23,7 @@ Source5:        epl-v10.txt
 Source6:        http://www.apache.org/licenses/LICENSE-2.0.txt
 # http://jira.codehaus.org/browse/GROOVY-6085
 Patch0:         groovy-inner-interface-annotations.patch
+Patch1:         groovy-build-with-java8.patch
 BuildArch:      noarch
 
 BuildRequires:  ant
@@ -37,7 +38,7 @@ BuildRequires:  tomcat-jsp-2.2-api
 BuildRequires:  junit
 BuildRequires:  tomcat-servlet-3.0-api
 BuildRequires:  xstream
-BuildRequires:  java-devel >= 1.6
+BuildRequires:  java-devel >= 1.8
 BuildRequires:  desktop-file-utils
 BuildRequires:  jpackage-utils
 BuildRequires:  apache-commons-cli
@@ -61,7 +62,7 @@ Requires:       tomcat-jsp-2.2-api
 Requires:       junit
 Requires:       tomcat-servlet-3.0-api
 Requires:       xstream
-Requires:       java-devel >= 1.6
+Requires:       java-devel >= 1.8
 
 
 %description
@@ -88,6 +89,7 @@ cp %{SOURCE4} %{SOURCE5} %{SOURCE6} .
 find \( -name *.jar -o -name *.class \) -delete
 
 %patch0 -p1
+%patch1 -p1
 
 %build
 mkdir -p target/lib/{compile,tools}
@@ -100,19 +102,11 @@ build-jar-repository target/lib/compile servlet jsp \
         bsf jline1 xstream ant junit ivy commons-cli \
         jansi
 
-# Use ECJ instead of OpenJDK to compile MethodHandle.  This is a
-# workaround for a bug in OpenJDK that causes compilation of
-# MethodHandle to take many minutes (15min or even more), while ECJ
-# can compile it under five seconds.  See rhbz#971483 and
-# http://mail.openjdk.java.net/pipermail/compiler-dev/2013-May/006339.html
-ecj -d target/classes `find -name MethodHandle.java -o -name ArrayUtil.java`
-
 # Build
 # TODO: Build at least tests, maybe examples
 ant -DskipTests=on -DskipExamples=on -DskipFetch=on -DskipEmbeddable=on \
         createJars javadoc
 
-
 %install
 
 # Code
@@ -167,6 +161,9 @@ install -p -m644 pom.xml $RPM_BUILD_ROOT/%{_mavenpomdir}/JPP-%{name}.pom
 %doc LICENSE.txt LICENSE-2.0.txt NOTICE.txt cpl-v10.txt epl-v10.txt
 
 %changelog
+* Mon Jun 16 2014 Michal Srb <msrb at redhat.com> - 1.8.9-11
+- Fix FTBFS
+
 * Sat Jun 07 2014 Fedora Release Engineering <rel-eng at lists.fedoraproject.org> - 1.8.9-10
 - Rebuilt for https://fedoraproject.org/wiki/Fedora_21_Mass_Rebuild
 


More information about the scm-commits mailing list