commit a4efb7030866dda067b659efc549047cfb71fecc3a5053014c995ab469c22b08 Author: Fridrich Strba Date: Thu Oct 25 19:28:37 2018 +0000 OBS-URL: https://build.opensuse.org/package/show/Java:packages/xbean?expand=0&rev=1 diff --git a/.gitattributes b/.gitattributes new file mode 100644 index 0000000..9b03811 --- /dev/null +++ b/.gitattributes @@ -0,0 +1,23 @@ +## Default LFS +*.7z filter=lfs diff=lfs merge=lfs -text +*.bsp filter=lfs diff=lfs merge=lfs -text +*.bz2 filter=lfs diff=lfs merge=lfs -text +*.gem filter=lfs diff=lfs merge=lfs -text +*.gz filter=lfs diff=lfs merge=lfs -text +*.jar filter=lfs diff=lfs merge=lfs -text +*.lz filter=lfs diff=lfs merge=lfs -text +*.lzma filter=lfs diff=lfs merge=lfs -text +*.obscpio filter=lfs diff=lfs merge=lfs -text +*.oxt filter=lfs diff=lfs merge=lfs -text +*.pdf filter=lfs diff=lfs merge=lfs -text +*.png filter=lfs diff=lfs merge=lfs -text +*.rpm filter=lfs diff=lfs merge=lfs -text +*.tbz filter=lfs diff=lfs merge=lfs -text +*.tbz2 filter=lfs diff=lfs merge=lfs -text +*.tgz filter=lfs diff=lfs merge=lfs -text +*.ttf filter=lfs diff=lfs merge=lfs -text +*.txz filter=lfs diff=lfs merge=lfs -text +*.whl filter=lfs diff=lfs merge=lfs -text +*.xz filter=lfs diff=lfs merge=lfs -text +*.zip filter=lfs diff=lfs merge=lfs -text +*.zst filter=lfs diff=lfs merge=lfs -text diff --git a/.gitignore b/.gitignore new file mode 100644 index 0000000..57affb6 --- /dev/null +++ b/.gitignore @@ -0,0 +1 @@ +.osc diff --git a/0001-Unshade-ASM.patch b/0001-Unshade-ASM.patch new file mode 100644 index 0000000..bf83472 --- /dev/null +++ b/0001-Unshade-ASM.patch @@ -0,0 +1,35 @@ +From 5b1a0fe4400bffddd1ab31af91beaef7bb560f6c Mon Sep 17 00:00:00 2001 +From: Michael Simacek +Date: Thu, 12 May 2016 11:40:13 +0200 +Subject: [PATCH 1/3] Unshade ASM + +--- + .../org/apache/xbean/recipe/XbeanAsmParameterNameLoader.java | 12 ++++++------ + 1 file changed, 6 insertions(+), 6 deletions(-) + +diff --git a/xbean-reflect/src/main/java/org/apache/xbean/recipe/XbeanAsmParameterNameLoader.java b/xbean-reflect/src/main/java/org/apache/xbean/recipe/XbeanAsmParameterNameLoader.java +index dea2f2a..303dfc3 100644 +--- a/xbean-reflect/src/main/java/org/apache/xbean/recipe/XbeanAsmParameterNameLoader.java ++++ b/xbean-reflect/src/main/java/org/apache/xbean/recipe/XbeanAsmParameterNameLoader.java +@@ -17,12 +17,12 @@ + */ + package org.apache.xbean.recipe; + +-import org.apache.xbean.asm5.ClassReader; +-import org.apache.xbean.asm5.ClassVisitor; +-import org.apache.xbean.asm5.Label; +-import org.apache.xbean.asm5.MethodVisitor; +-import org.apache.xbean.asm5.Opcodes; +-import org.apache.xbean.asm5.Type; ++import org.objectweb.asm.ClassReader; ++import org.objectweb.asm.ClassVisitor; ++import org.objectweb.asm.Label; ++import org.objectweb.asm.MethodVisitor; ++import org.objectweb.asm.Opcodes; ++import org.objectweb.asm.Type; + + import java.io.IOException; + import java.io.InputStream; +-- +2.9.3 + diff --git a/0002-Port-to-Eclipse-Luna-OSGi.patch b/0002-Port-to-Eclipse-Luna-OSGi.patch new file mode 100644 index 0000000..51a15dd --- /dev/null +++ b/0002-Port-to-Eclipse-Luna-OSGi.patch @@ -0,0 +1,62 @@ +From 9e1bb9bfee86490163fb1e226556620aa8cdf7bc Mon Sep 17 00:00:00 2001 +From: Mikolaj Izdebski +Date: Fri, 21 Nov 2014 10:05:05 +0100 +Subject: [PATCH 2/3] Port to Eclipse Luna OSGi + +--- + xbean-bundleutils/pom.xml | 8 +------- + .../apache/xbean/osgi/bundle/util/DelegatingBundleContext.java | 10 ++++++++++ + 2 files changed, 11 insertions(+), 7 deletions(-) + +diff --git a/xbean-bundleutils/pom.xml b/xbean-bundleutils/pom.xml +index 72f4865..2b3459d 100644 +--- a/xbean-bundleutils/pom.xml ++++ b/xbean-bundleutils/pom.xml +@@ -35,15 +35,9 @@ + slf4j-api + + +- org.osgi +- org.osgi.core +- 4.3.1 +- provided +- +- + org.eclipse + osgi +- 3.6.0.v20100517 ++ 3.10.0-v20140606-1445 + provided + + +diff --git a/xbean-bundleutils/src/main/java/org/apache/xbean/osgi/bundle/util/DelegatingBundleContext.java b/xbean-bundleutils/src/main/java/org/apache/xbean/osgi/bundle/util/DelegatingBundleContext.java +index 58a392b..cc8d081 100644 +--- a/xbean-bundleutils/src/main/java/org/apache/xbean/osgi/bundle/util/DelegatingBundleContext.java ++++ b/xbean-bundleutils/src/main/java/org/apache/xbean/osgi/bundle/util/DelegatingBundleContext.java +@@ -31,7 +31,9 @@ import org.osgi.framework.BundleListener; + import org.osgi.framework.Filter; + import org.osgi.framework.FrameworkListener; + import org.osgi.framework.InvalidSyntaxException; ++import org.osgi.framework.ServiceFactory; + import org.osgi.framework.ServiceListener; ++import org.osgi.framework.ServiceObjects; + import org.osgi.framework.ServiceReference; + import org.osgi.framework.ServiceRegistration; + +@@ -153,5 +155,13 @@ public class DelegatingBundleContext implements BundleContext { + public Bundle getBundle(String location) { + return bundleContext.getBundle(location); + } ++ ++ public ServiceObjects getServiceObjects(ServiceReference reference) { ++ return bundleContext.getServiceObjects(reference); ++ } ++ ++ public ServiceRegistration registerService(Class clazz, ServiceFactory factory, Dictionary properties) { ++ return bundleContext.registerService(clazz, factory, properties); ++ } + + } +-- +2.9.3 + diff --git a/0003-Port-to-QDox-2.0.patch b/0003-Port-to-QDox-2.0.patch new file mode 100644 index 0000000..0626391 --- /dev/null +++ b/0003-Port-to-QDox-2.0.patch @@ -0,0 +1,689 @@ +From 928bd5a98dc500a31197a56c8f6c5e19a3a273f8 Mon Sep 17 00:00:00 2001 +From: Mikolaj Izdebski +Date: Fri, 21 Nov 2014 10:51:38 +0100 +Subject: [PATCH 3/3] Port to QDox 2.0 + +--- + pom.xml | 2 +- + .../blueprint/generator/QdoxMappingLoader.java | 144 +++++++++++--------- + .../xbean/spring/generator/QdoxMappingLoader.java | 148 ++++++++++++--------- + .../org/apache/xbean/spring/generator/Type.java | 7 +- + 4 files changed, 166 insertions(+), 135 deletions(-) + +diff --git a/pom.xml b/pom.xml +index 6e53649..9132fe4 100644 +--- a/pom.xml ++++ b/pom.xml +@@ -297,7 +297,7 @@ + + com.thoughtworks.qdox + qdox +- 1.6.3 ++ 2.0-M5 + + + +diff --git a/xbean-blueprint/src/main/java/org/apache/xbean/blueprint/generator/QdoxMappingLoader.java b/xbean-blueprint/src/main/java/org/apache/xbean/blueprint/generator/QdoxMappingLoader.java +index 6635937..e17fd08 100644 +--- a/xbean-blueprint/src/main/java/org/apache/xbean/blueprint/generator/QdoxMappingLoader.java ++++ b/xbean-blueprint/src/main/java/org/apache/xbean/blueprint/generator/QdoxMappingLoader.java +@@ -20,6 +20,7 @@ import java.io.File; + import java.io.IOException; + import java.net.URL; + import java.util.ArrayList; ++import java.util.Collection; + import java.util.Collections; + import java.util.Enumeration; + import java.util.HashMap; +@@ -31,14 +32,17 @@ import java.util.TreeSet; + import java.util.jar.JarEntry; + import java.util.jar.JarFile; + +-import com.thoughtworks.qdox.JavaDocBuilder; ++import com.thoughtworks.qdox.JavaProjectBuilder; + import com.thoughtworks.qdox.model.BeanProperty; + import com.thoughtworks.qdox.model.DocletTag; + import com.thoughtworks.qdox.model.JavaClass; ++import com.thoughtworks.qdox.model.JavaConstructor; + import com.thoughtworks.qdox.model.JavaMethod; ++import com.thoughtworks.qdox.model.JavaModel; + import com.thoughtworks.qdox.model.JavaParameter; + import com.thoughtworks.qdox.model.JavaSource; +-import com.thoughtworks.qdox.model.Type; ++import com.thoughtworks.qdox.model.JavaType; ++ + import org.apache.commons.logging.Log; + import org.apache.commons.logging.LogFactory; + +@@ -62,7 +66,7 @@ public class QdoxMappingLoader implements MappingLoader { + private final String defaultNamespace; + private final File[] srcDirs; + private final String[] excludedClasses; +- private Type collectionType; ++ private JavaClass collectionType; + + public QdoxMappingLoader(String defaultNamespace, File[] srcDirs, String[] excludedClasses) { + this.defaultNamespace = defaultNamespace; +@@ -79,7 +83,7 @@ public class QdoxMappingLoader implements MappingLoader { + } + + public Set loadNamespaces() throws IOException { +- JavaDocBuilder builder = new JavaDocBuilder(); ++ JavaProjectBuilder builder = new JavaProjectBuilder(); + + log.debug("Source directories: "); + +@@ -92,11 +96,11 @@ public class QdoxMappingLoader implements MappingLoader { + getSourceFiles(sourceDirectory, excludedClasses, builder); + } + +- collectionType = builder.getClassByName("java.util.Collection").asType(); ++ collectionType = builder.getClassByName("java.util.Collection"); + return loadNamespaces(builder); + } + +- private Set loadNamespaces(JavaDocBuilder builder) { ++ private Set loadNamespaces(JavaProjectBuilder builder) { + // load all of the elements + List elements = loadElements(builder); + +@@ -131,14 +135,14 @@ public class QdoxMappingLoader implements MappingLoader { + return Collections.unmodifiableSet(namespaces); + } + +- private List loadElements(JavaDocBuilder builder) { +- JavaSource[] javaSources = builder.getSources(); ++ private List loadElements(JavaProjectBuilder builder) { ++ Collection javaSources = builder.getSources(); + List elements = new ArrayList(); + for (JavaSource javaSource : javaSources) { +- if (javaSource.getClasses().length == 0) { ++ if (javaSource.getClasses().isEmpty()) { + log.info("No Java Classes defined in: " + javaSource.getURL()); + } else { +- JavaClass[] classes = javaSource.getClasses(); ++ Collection classes = javaSource.getClasses(); + for (JavaClass javaClass : classes) { + ElementMapping element = loadElement(builder, javaClass); + if (element != null && !javaClass.isAbstract()) { +@@ -152,7 +156,7 @@ public class QdoxMappingLoader implements MappingLoader { + return elements; + } + +- private ElementMapping loadElement(JavaDocBuilder builder, JavaClass javaClass) { ++ private ElementMapping loadElement(JavaProjectBuilder builder, JavaClass javaClass) { + DocletTag xbeanTag = javaClass.getTagByName(XBEAN_ANNOTATION); + if (xbeanTag == null) { + return null; +@@ -176,7 +180,7 @@ public class QdoxMappingLoader implements MappingLoader { + Map attributesByPropertyName = new HashMap(); + + for (JavaClass jClass = javaClass; jClass != null; jClass = jClass.getSuperJavaClass()) { +- BeanProperty[] beanProperties = jClass.getBeanProperties(); ++ Collection beanProperties = jClass.getBeanProperties(); + for (BeanProperty beanProperty : beanProperties) { + // we only care about properties with a setter + if (beanProperty.getMutator() != null) { +@@ -219,9 +223,9 @@ public class QdoxMappingLoader implements MappingLoader { + String destroyMethod = null; + String factoryMethod = null; + for (JavaClass jClass = javaClass; jClass != null; jClass = jClass.getSuperJavaClass()) { +- JavaMethod[] methods = javaClass.getMethods(); ++ Collection methods = javaClass.getMethods(); + for (JavaMethod method : methods) { +- if (method.isPublic() && !method.isConstructor()) { ++ if (method.isPublic()) { + if (initMethod == null && method.getTagByName(INIT_METHOD_ANNOTATION) != null) { + initMethod = method.getName(); + } +@@ -237,22 +241,43 @@ public class QdoxMappingLoader implements MappingLoader { + } + + List> constructorArgs = new ArrayList>(); +- JavaMethod[] methods = javaClass.getMethods(); +- for (JavaMethod method : methods) { +- JavaParameter[] parameters = method.getParameters(); +- if (isValidConstructor(factoryMethod, method, parameters)) { +- List args = new ArrayList(parameters.length); +- for (JavaParameter parameter : parameters) { +- AttributeMapping attributeMapping = attributesByPropertyName.get(parameter.getName()); +- if (attributeMapping == null) { +- attributeMapping = loadParameter(parameter); +- +- attributes.add(attributeMapping); +- attributesByPropertyName.put(attributeMapping.getPropertyName(), attributeMapping); ++ if (factoryMethod == null) { ++ Collection constructors = javaClass.getConstructors(); ++ for (JavaConstructor constructor : constructors) { ++ Collection parameters = constructor.getParameters(); ++ if (constructor.isPublic() && parameters.size() > 0) { ++ List args = new ArrayList(parameters.size()); ++ for (JavaParameter parameter : parameters) { ++ AttributeMapping attributeMapping = attributesByPropertyName.get(parameter.getName()); ++ if (attributeMapping == null) { ++ attributeMapping = loadParameter(parameter, constructor); ++ ++ attributes.add(attributeMapping); ++ attributesByPropertyName.put(attributeMapping.getPropertyName(), attributeMapping); ++ } ++ args.add(new ParameterMapping(attributeMapping.getPropertyName(), toMappingType(parameter.getType(), null))); + } +- args.add(new ParameterMapping(attributeMapping.getPropertyName(), toMappingType(parameter.getType(), null))); ++ constructorArgs.add(Collections.unmodifiableList(args)); ++ } ++ } ++ } else { ++ Collection methods = javaClass.getMethods(); ++ for (JavaMethod method : methods) { ++ Collection parameters = method.getParameters(); ++ if (method.isPublic() && parameters.size() > 0 && method.getName().equals(factoryMethod)) { ++ List args = new ArrayList(parameters.size()); ++ for (JavaParameter parameter : parameters) { ++ AttributeMapping attributeMapping = attributesByPropertyName.get(parameter.getName()); ++ if (attributeMapping == null) { ++ attributeMapping = loadParameter(parameter, method); ++ ++ attributes.add(attributeMapping); ++ attributesByPropertyName.put(attributeMapping.getPropertyName(), attributeMapping); ++ } ++ args.add(new ParameterMapping(attributeMapping.getPropertyName(), toMappingType(parameter.getType(), null))); ++ } ++ constructorArgs.add(Collections.unmodifiableList(args)); + } +- constructorArgs.add(Collections.unmodifiableList(args)); + } + } + +@@ -303,7 +328,7 @@ public class QdoxMappingLoader implements MappingLoader { + interfaces); + } + +- private List getFullyQualifiedNames(JavaClass[] implementedInterfaces) { ++ private List getFullyQualifiedNames(Collection implementedInterfaces) { + ArrayList l = new ArrayList(); + for (JavaClass implementedInterface : implementedInterfaces) { + l.add(implementedInterface.getFullyQualifiedName()); +@@ -395,19 +420,19 @@ public class QdoxMappingLoader implements MappingLoader { + return defaultDescription; + } + +- private AttributeMapping loadParameter(JavaParameter parameter) { ++ private AttributeMapping loadParameter(JavaParameter parameter, JavaModel methodOrConstructor) { + String parameterName = parameter.getName(); + String parameterDescription = getParameterDescription(parameter); + + // first attempt to load the attribute from the java beans accessor methods +- JavaClass javaClass = parameter.getParentMethod().getParentClass(); ++ JavaClass javaClass = parameter.getParentClass(); + BeanProperty beanProperty = javaClass.getBeanProperty(parameterName); + if (beanProperty != null) { + AttributeMapping attributeMapping = loadAttribute(beanProperty, parameterDescription); + // if the attribute mapping is null, the property was tagged as hidden and this is an error + if (attributeMapping == null) { + throw new InvalidModelException("Hidden property usage: " + +- "The construction method " + toMethodLocator(parameter.getParentMethod()) + ++ "The construction method " + toMethodLocator(parameter.getParentClass(), methodOrConstructor) + + " can not use a hidded property " + parameterName); + } + return attributeMapping; +@@ -426,9 +451,9 @@ public class QdoxMappingLoader implements MappingLoader { + + private String getParameterDescription(JavaParameter parameter) { + String parameterName = parameter.getName(); +- DocletTag[] tags = parameter.getParentMethod().getTagsByName("param"); ++ Collection tags = parameter.getTagsByName("param"); + for (DocletTag tag : tags) { +- if (tag.getParameters()[0].equals(parameterName)) { ++ if (tag.getParameters().get(0).equals(parameterName)) { + String parameterDescription = tag.getValue().trim(); + if (parameterDescription.startsWith(parameterName)) { + parameterDescription = parameterDescription.substring(parameterName.length()).trim(); +@@ -439,18 +464,6 @@ public class QdoxMappingLoader implements MappingLoader { + return null; + } + +- private boolean isValidConstructor(String factoryMethod, JavaMethod method, JavaParameter[] parameters) { +- if (!method.isPublic() || parameters.length == 0) { +- return false; +- } +- +- if (factoryMethod == null) { +- return method.isConstructor(); +- } else { +- return method.getName().equals(factoryMethod); +- } +- } +- + private static String getProperty(DocletTag propertyTag, String propertyName) { + return getProperty(propertyTag, propertyName, null); + } +@@ -477,14 +490,17 @@ public class QdoxMappingLoader implements MappingLoader { + return false; + } + +- private org.apache.xbean.blueprint.generator.Type toMappingType(Type type, String nestedType) { ++ private org.apache.xbean.blueprint.generator.Type toMappingType(JavaType type, String nestedType) { + try { +- if (type.isArray()) { +- return org.apache.xbean.blueprint.generator.Type.newArrayType(type.getValue(), type.getDimensions()); +- } else if (type.isA(collectionType)) { +- if (nestedType == null) nestedType = "java.lang.Object"; +- return org.apache.xbean.blueprint.generator.Type.newCollectionType(type.getValue(), +- org.apache.xbean.blueprint.generator.Type.newSimpleType(nestedType)); ++ if (type instanceof JavaClass) { ++ JavaClass clazz = (JavaClass)type; ++ if (clazz.isArray()) { ++ return org.apache.xbean.blueprint.generator.Type.newArrayType(type.getValue(), clazz.getDimensions()); ++ } else if (clazz.isA(collectionType)) { ++ if (nestedType == null) nestedType = "java.lang.Object"; ++ return org.apache.xbean.blueprint.generator.Type.newCollectionType(type.getValue(), ++ org.apache.xbean.blueprint.generator.Type.newSimpleType(nestedType)); ++ } + } + } catch (Throwable t) { + log.debug("Could not load type mapping", t); +@@ -492,26 +508,28 @@ public class QdoxMappingLoader implements MappingLoader { + return org.apache.xbean.blueprint.generator.Type.newSimpleType(type.getValue()); + } + +- private static String toMethodLocator(JavaMethod method) { ++ private static String toMethodLocator(JavaClass parentClass, JavaModel methodOrConstructor) { ++ JavaMethod method = methodOrConstructor instanceof JavaMethod ? (JavaMethod) methodOrConstructor : null; ++ JavaConstructor constructor = methodOrConstructor instanceof JavaConstructor ? (JavaConstructor) methodOrConstructor : null; + StringBuffer buf = new StringBuffer(); +- buf.append(method.getParentClass().getFullyQualifiedName()); +- if (!method.isConstructor()) { ++ buf.append(parentClass.getFullyQualifiedName()); ++ if (method != null) { + buf.append(".").append(method.getName()); + } + buf.append("("); +- JavaParameter[] parameters = method.getParameters(); +- for (int i = 0; i < parameters.length; i++) { +- JavaParameter parameter = parameters[i]; ++ List parameters = method != null ? method.getParameters() : constructor.getParameters(); ++ for (int i = 0; i < parameters.size(); i++) { ++ JavaParameter parameter = parameters.get(i); + if (i > 0) { + buf.append(", "); + } + buf.append(parameter.getName()); + } +- buf.append(") : ").append(method.getLineNumber()); ++ buf.append(") : ").append(method != null ? method.getLineNumber() : constructor.getLineNumber()); + return buf.toString(); + } + +- private static void getSourceFiles(File base, String[] excludedClasses, JavaDocBuilder builder) throws IOException { ++ private static void getSourceFiles(File base, String[] excludedClasses, JavaProjectBuilder builder) throws IOException { + if (base.isDirectory()) { + listAllFileNames(base, "", excludedClasses, builder); + } else { +@@ -519,7 +537,7 @@ public class QdoxMappingLoader implements MappingLoader { + } + } + +- private static void listAllFileNames(File base, String prefix, String[] excludedClasses, JavaDocBuilder builder) throws IOException { ++ private static void listAllFileNames(File base, String prefix, String[] excludedClasses, JavaProjectBuilder builder) throws IOException { + if (!base.canRead() || !base.isDirectory()) { + throw new IllegalArgumentException(base.getAbsolutePath()); + } +@@ -536,7 +554,7 @@ public class QdoxMappingLoader implements MappingLoader { + } + } + +- private static void listAllJarEntries(File base, String[] excludedClasses, JavaDocBuilder builder) throws IOException { ++ private static void listAllJarEntries(File base, String[] excludedClasses, JavaProjectBuilder builder) throws IOException { + JarFile jarFile = new JarFile(base); + for (Enumeration entries = jarFile.entries(); entries.hasMoreElements(); ) { + JarEntry entry = (JarEntry) entries.nextElement(); +diff --git a/xbean-spring/src/main/java/org/apache/xbean/spring/generator/QdoxMappingLoader.java b/xbean-spring/src/main/java/org/apache/xbean/spring/generator/QdoxMappingLoader.java +index 94bd7a1..228117e 100644 +--- a/xbean-spring/src/main/java/org/apache/xbean/spring/generator/QdoxMappingLoader.java ++++ b/xbean-spring/src/main/java/org/apache/xbean/spring/generator/QdoxMappingLoader.java +@@ -20,6 +20,7 @@ import java.io.File; + import java.io.IOException; + import java.net.URL; + import java.util.ArrayList; ++import java.util.Collection; + import java.util.Collections; + import java.util.Enumeration; + import java.util.HashMap; +@@ -31,14 +32,17 @@ import java.util.TreeSet; + import java.util.jar.JarEntry; + import java.util.jar.JarFile; + +-import com.thoughtworks.qdox.JavaDocBuilder; ++import com.thoughtworks.qdox.JavaProjectBuilder; + import com.thoughtworks.qdox.model.BeanProperty; + import com.thoughtworks.qdox.model.DocletTag; + import com.thoughtworks.qdox.model.JavaClass; ++import com.thoughtworks.qdox.model.JavaConstructor; + import com.thoughtworks.qdox.model.JavaMethod; ++import com.thoughtworks.qdox.model.JavaModel; + import com.thoughtworks.qdox.model.JavaParameter; + import com.thoughtworks.qdox.model.JavaSource; +-import com.thoughtworks.qdox.model.Type; ++import com.thoughtworks.qdox.model.JavaType; ++ + import org.apache.commons.logging.Log; + import org.apache.commons.logging.LogFactory; + +@@ -62,7 +66,7 @@ public class QdoxMappingLoader implements MappingLoader { + private final String defaultNamespace; + private final File[] srcDirs; + private final String[] excludedClasses; +- private Type collectionType; ++ private JavaClass collectionType; + + public QdoxMappingLoader(String defaultNamespace, File[] srcDirs, String[] excludedClasses) { + this.defaultNamespace = defaultNamespace; +@@ -79,7 +83,7 @@ public class QdoxMappingLoader implements MappingLoader { + } + + public Set loadNamespaces() throws IOException { +- JavaDocBuilder builder = new JavaDocBuilder(); ++ JavaProjectBuilder builder = new JavaProjectBuilder(); + + log.debug("Source directories: "); + +@@ -92,11 +96,11 @@ public class QdoxMappingLoader implements MappingLoader { + getSourceFiles(sourceDirectory, excludedClasses, builder); + } + +- collectionType = builder.getClassByName("java.util.Collection").asType(); ++ collectionType = builder.getClassByName("java.util.Collection"); + return loadNamespaces(builder); + } + +- private Set loadNamespaces(JavaDocBuilder builder) { ++ private Set loadNamespaces(JavaProjectBuilder builder) { + // load all of the elements + List elements = loadElements(builder); + +@@ -131,14 +135,14 @@ public class QdoxMappingLoader implements MappingLoader { + return Collections.unmodifiableSet(namespaces); + } + +- private List loadElements(JavaDocBuilder builder) { +- JavaSource[] javaSources = builder.getSources(); ++ private List loadElements(JavaProjectBuilder builder) { ++ Collection javaSources = builder.getSources(); + List elements = new ArrayList(); + for (JavaSource javaSource : javaSources) { +- if (javaSource.getClasses().length == 0) { ++ if (javaSource.getClasses().isEmpty()) { + log.info("No Java Classes defined in: " + javaSource.getURL()); + } else { +- JavaClass[] classes = javaSource.getClasses(); ++ Collection classes = javaSource.getClasses(); + for (JavaClass javaClass : classes) { + ElementMapping element = loadElement(builder, javaClass); + if (element != null && !javaClass.isAbstract()) { +@@ -152,7 +156,7 @@ public class QdoxMappingLoader implements MappingLoader { + return elements; + } + +- private ElementMapping loadElement(JavaDocBuilder builder, JavaClass javaClass) { ++ private ElementMapping loadElement(JavaProjectBuilder builder, JavaClass javaClass) { + DocletTag xbeanTag = javaClass.getTagByName(XBEAN_ANNOTATION); + if (xbeanTag == null) { + return null; +@@ -176,7 +180,7 @@ public class QdoxMappingLoader implements MappingLoader { + Map attributesByPropertyName = new HashMap(); + + for (JavaClass jClass = javaClass; jClass != null; jClass = jClass.getSuperJavaClass()) { +- BeanProperty[] beanProperties = jClass.getBeanProperties(); ++ Collection beanProperties = jClass.getBeanProperties(); + for (BeanProperty beanProperty : beanProperties) { + // we only care about properties with a setter + if (beanProperty.getMutator() != null) { +@@ -219,9 +223,9 @@ public class QdoxMappingLoader implements MappingLoader { + String destroyMethod = null; + String factoryMethod = null; + for (JavaClass jClass = javaClass; jClass != null; jClass = jClass.getSuperJavaClass()) { +- JavaMethod[] methods = javaClass.getMethods(); ++ Collection methods = javaClass.getMethods(); + for (JavaMethod method : methods) { +- if (method.isPublic() && !method.isConstructor()) { ++ if (method.isPublic()) { + if (initMethod == null && method.getTagByName(INIT_METHOD_ANNOTATION) != null) { + initMethod = method.getName(); + } +@@ -237,27 +241,48 @@ public class QdoxMappingLoader implements MappingLoader { + } + + List> constructorArgs = new ArrayList>(); +- JavaMethod[] methods = javaClass.getMethods(); +- for (JavaMethod method : methods) { +- JavaParameter[] parameters = method.getParameters(); +- if (isValidConstructor(factoryMethod, method, parameters)) { +- List args = new ArrayList(parameters.length); +- for (JavaParameter parameter : parameters) { +- AttributeMapping attributeMapping = attributesByPropertyName.get(parameter.getName()); +- if (attributeMapping == null) { +- attributeMapping = loadParameter(parameter); +- +- attributes.add(attributeMapping); +- attributesByPropertyName.put(attributeMapping.getPropertyName(), attributeMapping); ++ if (factoryMethod == null) { ++ Collection constructors = javaClass.getConstructors(); ++ for (JavaConstructor constructor : constructors) { ++ Collection parameters = constructor.getParameters(); ++ if (constructor.isPublic() && parameters.size() > 0) { ++ List args = new ArrayList(parameters.size()); ++ for (JavaParameter parameter : parameters) { ++ AttributeMapping attributeMapping = attributesByPropertyName.get(parameter.getName()); ++ if (attributeMapping == null) { ++ attributeMapping = loadParameter(parameter, constructor); ++ ++ attributes.add(attributeMapping); ++ attributesByPropertyName.put(attributeMapping.getPropertyName(), attributeMapping); ++ } ++ args.add(new ParameterMapping(attributeMapping.getPropertyName(), toMappingType(parameter.getType(), null))); + } +- args.add(new ParameterMapping(attributeMapping.getPropertyName(), toMappingType(parameter.getType(), null))); ++ constructorArgs.add(Collections.unmodifiableList(args)); ++ } ++ } ++ } else { ++ Collection methods = javaClass.getMethods(); ++ for (JavaMethod method : methods) { ++ Collection parameters = method.getParameters(); ++ if (method.isPublic() && parameters.size() > 0 && method.getName().equals(factoryMethod)) { ++ List args = new ArrayList(parameters.size()); ++ for (JavaParameter parameter : parameters) { ++ AttributeMapping attributeMapping = attributesByPropertyName.get(parameter.getName()); ++ if (attributeMapping == null) { ++ attributeMapping = loadParameter(parameter, method); ++ ++ attributes.add(attributeMapping); ++ attributesByPropertyName.put(attributeMapping.getPropertyName(), attributeMapping); ++ } ++ args.add(new ParameterMapping(attributeMapping.getPropertyName(), toMappingType(parameter.getType(), null))); ++ } ++ constructorArgs.add(Collections.unmodifiableList(args)); + } +- constructorArgs.add(Collections.unmodifiableList(args)); + } + } + + HashSet interfaces = new HashSet(); +- interfaces.addAll(getFullyQualifiedNames(javaClass.getImplementedInterfaces())); ++ interfaces.addAll(getFullyQualifiedNames(javaClass.getInterfaces())); + + JavaClass actualClass = javaClass; + if (factoryClass != null) { +@@ -282,7 +307,7 @@ public class QdoxMappingLoader implements MappingLoader { + } + p = s; + superClasses.add(p.getFullyQualifiedName()); +- interfaces.addAll(getFullyQualifiedNames(p.getImplementedInterfaces())); ++ interfaces.addAll(getFullyQualifiedNames(p.getInterfaces())); + } + + return new ElementMapping(namespace, +@@ -303,7 +328,7 @@ public class QdoxMappingLoader implements MappingLoader { + interfaces); + } + +- private List getFullyQualifiedNames(JavaClass[] implementedInterfaces) { ++ private List getFullyQualifiedNames(Collection implementedInterfaces) { + ArrayList l = new ArrayList(); + for (JavaClass implementedInterface : implementedInterfaces) { + l.add(implementedInterface.getFullyQualifiedName()); +@@ -395,19 +420,19 @@ public class QdoxMappingLoader implements MappingLoader { + return defaultDescription; + } + +- private AttributeMapping loadParameter(JavaParameter parameter) { ++ private AttributeMapping loadParameter(JavaParameter parameter, JavaModel methodOrConstructor) { + String parameterName = parameter.getName(); + String parameterDescription = getParameterDescription(parameter); + + // first attempt to load the attribute from the java beans accessor methods +- JavaClass javaClass = parameter.getParentMethod().getParentClass(); ++ JavaClass javaClass = parameter.getDeclaringClass(); + BeanProperty beanProperty = javaClass.getBeanProperty(parameterName); + if (beanProperty != null) { + AttributeMapping attributeMapping = loadAttribute(beanProperty, parameterDescription); + // if the attribute mapping is null, the property was tagged as hidden and this is an error + if (attributeMapping == null) { + throw new InvalidModelException("Hidden property usage: " + +- "The construction method " + toMethodLocator(parameter.getParentMethod()) + ++ "The construction method " + toMethodLocator(parameter.getDeclaringClass(), methodOrConstructor) + + " can not use a hidded property " + parameterName); + } + return attributeMapping; +@@ -426,9 +451,9 @@ public class QdoxMappingLoader implements MappingLoader { + + private String getParameterDescription(JavaParameter parameter) { + String parameterName = parameter.getName(); +- DocletTag[] tags = parameter.getParentMethod().getTagsByName("param"); ++ Collection tags = parameter.getTagsByName("param"); + for (DocletTag tag : tags) { +- if (tag.getParameters()[0].equals(parameterName)) { ++ if (tag.getParameters().get(0).equals(parameterName)) { + String parameterDescription = tag.getValue().trim(); + if (parameterDescription.startsWith(parameterName)) { + parameterDescription = parameterDescription.substring(parameterName.length()).trim(); +@@ -439,18 +464,6 @@ public class QdoxMappingLoader implements MappingLoader { + return null; + } + +- private boolean isValidConstructor(String factoryMethod, JavaMethod method, JavaParameter[] parameters) { +- if (!method.isPublic() || parameters.length == 0) { +- return false; +- } +- +- if (factoryMethod == null) { +- return method.isConstructor(); +- } else { +- return method.getName().equals(factoryMethod); +- } +- } +- + private static String getProperty(DocletTag propertyTag, String propertyName) { + return getProperty(propertyTag, propertyName, null); + } +@@ -477,14 +490,17 @@ public class QdoxMappingLoader implements MappingLoader { + return false; + } + +- private org.apache.xbean.spring.generator.Type toMappingType(Type type, String nestedType) { ++ private org.apache.xbean.spring.generator.Type toMappingType(JavaType type, String nestedType) { + try { +- if (type.isArray()) { +- return org.apache.xbean.spring.generator.Type.newArrayType(type.getValue(), type.getDimensions()); +- } else if (type.isA(collectionType)) { +- if (nestedType == null) nestedType = "java.lang.Object"; +- return org.apache.xbean.spring.generator.Type.newCollectionType(type.getValue(), +- org.apache.xbean.spring.generator.Type.newSimpleType(nestedType)); ++ if (type instanceof JavaClass) { ++ JavaClass clazz = (JavaClass)type; ++ if (clazz.isArray()) { ++ return org.apache.xbean.spring.generator.Type.newArrayType(type.getValue(), clazz.getDimensions()); ++ } else if (clazz.isA(collectionType)) { ++ if (nestedType == null) nestedType = "java.lang.Object"; ++ return org.apache.xbean.spring.generator.Type.newCollectionType(type.getValue(), ++ org.apache.xbean.spring.generator.Type.newSimpleType(nestedType)); ++ } + } + } catch (Throwable t) { + log.debug("Could not load type mapping", t); +@@ -492,26 +508,28 @@ public class QdoxMappingLoader implements MappingLoader { + return org.apache.xbean.spring.generator.Type.newSimpleType(type.getValue()); + } + +- private static String toMethodLocator(JavaMethod method) { ++ private static String toMethodLocator(JavaClass parentClass, JavaModel methodOrConstructor) { ++ JavaMethod method = methodOrConstructor instanceof JavaMethod ? (JavaMethod) methodOrConstructor : null; ++ JavaConstructor constructor = methodOrConstructor instanceof JavaConstructor ? (JavaConstructor) methodOrConstructor : null; + StringBuffer buf = new StringBuffer(); +- buf.append(method.getParentClass().getFullyQualifiedName()); +- if (!method.isConstructor()) { ++ buf.append(parentClass.getFullyQualifiedName()); ++ if (method != null) { + buf.append(".").append(method.getName()); + } + buf.append("("); +- JavaParameter[] parameters = method.getParameters(); +- for (int i = 0; i < parameters.length; i++) { +- JavaParameter parameter = parameters[i]; ++ List parameters = method != null ? method.getParameters() : constructor.getParameters(); ++ for (int i = 0; i < parameters.size(); i++) { ++ JavaParameter parameter = parameters.get(i); + if (i > 0) { + buf.append(", "); + } + buf.append(parameter.getName()); + } +- buf.append(") : ").append(method.getLineNumber()); ++ buf.append(") : ").append(method != null ? method.getLineNumber() : constructor.getLineNumber()); + return buf.toString(); + } + +- private static void getSourceFiles(File base, String[] excludedClasses, JavaDocBuilder builder) throws IOException { ++ private static void getSourceFiles(File base, String[] excludedClasses, JavaProjectBuilder builder) throws IOException { + if (base.isDirectory()) { + listAllFileNames(base, "", excludedClasses, builder); + } else { +@@ -519,7 +537,7 @@ public class QdoxMappingLoader implements MappingLoader { + } + } + +- private static void listAllFileNames(File base, String prefix, String[] excludedClasses, JavaDocBuilder builder) throws IOException { ++ private static void listAllFileNames(File base, String prefix, String[] excludedClasses, JavaProjectBuilder builder) throws IOException { + if (!base.canRead() || !base.isDirectory()) { + throw new IllegalArgumentException(base.getAbsolutePath()); + } +@@ -536,7 +554,7 @@ public class QdoxMappingLoader implements MappingLoader { + } + } + +- private static void listAllJarEntries(File base, String[] excludedClasses, JavaDocBuilder builder) throws IOException { ++ private static void listAllJarEntries(File base, String[] excludedClasses, JavaProjectBuilder builder) throws IOException { + JarFile jarFile = new JarFile(base); + for (Enumeration entries = jarFile.entries(); entries.hasMoreElements(); ) { + JarEntry entry = (JarEntry) entries.nextElement(); +diff --git a/xbean-spring/src/main/java/org/apache/xbean/spring/generator/Type.java b/xbean-spring/src/main/java/org/apache/xbean/spring/generator/Type.java +index 5eac64c..0d9fa63 100644 +--- a/xbean-spring/src/main/java/org/apache/xbean/spring/generator/Type.java ++++ b/xbean-spring/src/main/java/org/apache/xbean/spring/generator/Type.java +@@ -41,12 +41,7 @@ public class Type { + public static Type newArrayType(String type, int dimensions) { + if (type == null) throw new NullPointerException("type"); + if (dimensions < 1) throw new IllegalArgumentException("dimensions must be atleast one"); +- StringBuffer buf = new StringBuffer(type.length() + (dimensions * 2)); +- buf.append(type); +- for (int i = 0; i < dimensions; i ++) { +- buf.append("[]"); +- } +- return new Type(buf.toString(), newSimpleType(type)); ++ return new Type(type, newSimpleType(type.replaceAll("\\[\\]", ""))); + } + + public static Type newCollectionType(String collectionType, Type elementType) { +-- +2.9.3 + diff --git a/xbean-4.5-source-release.zip b/xbean-4.5-source-release.zip new file mode 100644 index 0000000..858c32f --- /dev/null +++ b/xbean-4.5-source-release.zip @@ -0,0 +1,3 @@ +version https://git-lfs.github.com/spec/v1 +oid sha256:501a3cdacd5c42d0e5f691a0c3fd903a5f335f0d446f96c0be52ab0c58533b17 +size 1017685 diff --git a/xbean.changes b/xbean.changes new file mode 100644 index 0000000..80ccff3 --- /dev/null +++ b/xbean.changes @@ -0,0 +1,5 @@ +------------------------------------------------------------------- +Thu Oct 25 19:27:30 UTC 2018 - Fridrich Strba + +- Initial packaging of last xbean version that works with asm5 +- Spec file inspired by Fedora rpm package diff --git a/xbean.spec b/xbean.spec new file mode 100644 index 0000000..c663bcc --- /dev/null +++ b/xbean.spec @@ -0,0 +1,251 @@ +# +# spec file for package xbean +# +# Copyright (c) 2018 SUSE LINUX GmbH, Nuernberg, Germany. +# +# All modifications and additions to the file contributed by third parties +# remain the property of their copyright owners, unless otherwise agreed +# upon. The license for this file, and modifications and additions to the +# file, is the same license as for the pristine package itself (unless the +# license for the pristine package is not an Open Source License, in which +# case the license is the MIT License). An "Open Source License" is a +# license that conforms to the Open Source Definition (Version 1.9) +# published by the Open Source Initiative. + +# Please submit bugfixes or comments via https://bugs.opensuse.org/ +# + + +%bcond_with equinox +%bcond_with groovy +%bcond_with spring +Name: xbean +Version: 4.5 +Release: 0 +Summary: Java plugin based web server +License: Apache-2.0 +Group: Development/Libraries/Java +URL: http://geronimo.apache.org/xbean/ +Source0: http://repo2.maven.org/maven2/org/apache/%{name}/%{name}/%{version}/%{name}-%{version}-source-release.zip +# Fix dependency on xbean-asm4-shaded to original objectweb-asm +Patch0: 0001-Unshade-ASM.patch +# Compatibility with Eclipse Luna (rhbz#1087461) +Patch1: 0002-Port-to-Eclipse-Luna-OSGi.patch +Patch2: 0003-Port-to-QDox-2.0.patch +BuildRequires: apache-commons-logging +BuildRequires: asm5 +BuildRequires: fdupes +BuildRequires: java-devel +BuildRequires: javapackages-local +BuildRequires: javapackages-tools +BuildRequires: log4j +BuildRequires: slf4j +BuildRequires: unzip +BuildRequires: mvn(org.osgi:org.osgi.core) +# The code uses sun.misc.URLClassloader +BuildConflicts: java-devel >= 9 +Requires: apache-commons-loging +Requires: asm5 +Requires: log4j +Requires: slf4j +BuildArch: noarch +%if %{with equinox} +BuildRequires: mvn(org.eclipse:osgi) +%endif +%if %{with groovy} +BuildRequires: mvn(org.codehaus.groovy:groovy-all) +%endif +%if %{with spring} +BuildRequires: mvn(ant:ant) +BuildRequires: mvn(com.thoughtworks.qdox:qdox) +BuildRequires: mvn(commons-logging:commons-logging) +BuildRequires: mvn(org.apache.maven.plugins:maven-antrun-plugin) +BuildRequires: mvn(org.apache.maven.plugins:maven-plugin-plugin) +BuildRequires: mvn(org.apache.maven:maven-archiver) +BuildRequires: mvn(org.apache.maven:maven-artifact) +BuildRequires: mvn(org.apache.maven:maven-plugin-api) +BuildRequires: mvn(org.apache.maven:maven-project) +BuildRequires: mvn(org.codehaus.plexus:plexus-archiver) +BuildRequires: mvn(org.codehaus.plexus:plexus-utils) +BuildRequires: mvn(org.springframework:spring-beans) +BuildRequires: mvn(org.springframework:spring-context) +BuildRequires: mvn(org.springframework:spring-web) +%endif + +%description +The goal of XBean project is to create a plugin based server +analogous to Eclipse being a plugin based IDE. XBean will be able to +discover, download and install server plugins from an Internet based +repository. In addition, we include support for multiple IoC systems, +support for running with no IoC system, JMX without JMX code, +lifecycle and class loader management, and a rock solid Spring +integration. + +%if %{with spring} +# For now blueprint module fails to compile. Disable it. +%if 0 +%package blueprint +Summary: Schema-driven namespace handler for Apache Aries Blueprint +Group: Development/Libraries/Java + +%description blueprint +This package provides %{summary}. +%endif + +%package classloader +Summary: A flexibie multi-parent classloader +Group: Development/Libraries/Java + +%description classloader +This package provides %{summary}. + +%package spring +Summary: Schema-driven namespace handler for spring contexts +Group: Development/Libraries/Java +Requires: %{name} = %{version}-%{release} + +%description spring +This package provides %{summary}. + +%package -n maven-%{name}-plugin +Summary: XBean plugin for Apache Maven +Group: Development/Libraries/Java + +%description -n maven-%{name}-plugin +This package provides %{summary}. +%endif + +%package javadoc +Summary: API documentation for %{name} +Group: Development/Libraries/Java + +%description javadoc +This package provides %{summary}. + +%prep +%setup -q +# build failing on this due to doxia-sitetools problems +rm src/site/site.xml + +%patch0 -p1 +%if %{with equinox} +%patch1 -p1 +%endif +%patch2 -p1 + +%pom_remove_parent +%pom_remove_dep mx4j:mx4j + +%pom_remove_dep -r :xbean-asm5-shaded +%pom_remove_dep -r :xbean-finder-shaded +%pom_disable_module xbean-asm5-shaded +%pom_disable_module xbean-finder-shaded + +%pom_xpath_remove pom:scope xbean-asm-util +%pom_xpath_remove pom:optional xbean-asm-util + +# Prevent modules depending on springframework from building. +%if %{without spring} + %pom_remove_dep org.springframework: + #%%pom_disable_module xbean-blueprint + %pom_disable_module xbean-classloader + %pom_disable_module xbean-spring + %pom_disable_module maven-xbean-plugin + rm -rf maven-xbean-plugin +%else +# %mvn_package :xbean-classloader classloader +# %mvn_package :xbean-spring spring +# %mvn_package :maven-xbean-plugin maven-xbean-plugin +%endif +# blueprint FTBFS, disable for now +%pom_disable_module xbean-blueprint + +%if %{without equinox} + %pom_remove_dep :xbean-bundleutils xbean-finder + rm -r xbean-finder/src/main/java/org/apache/xbean/finder{,/archive}/Bundle* + %pom_disable_module xbean-bundleutils +%endif + +%if %{without groovy} +%pom_disable_module xbean-telnet +%endif + +# maven-xbean-plugin invocation makes no sense as there are no namespaces +%pom_remove_plugin :maven-xbean-plugin xbean-classloader + +# As auditing tool RAT is useful for upstream only. +%pom_remove_plugin :apache-rat-plugin + +# disable copy of internal aries-blueprint +sed -i "s|||" xbean-blueprint/pom.xml + +%build +for i in xbean-asm-util xbean-classpath xbean-finder xbean-naming xbean-reflect; do + pushd $i + mkdir -p build/classes + javac -d build/classes -encoding utf-8 -source 6 -target 6 \ + -cp $(build-classpath log4j commons-logging-api slf4j/api asm5/asm asm5/asm-commons):../xbean-asm-util/xbean-asm-util.jar \ + $(find src/main/java -name *.java) + jar cf $i.jar -C build/classes . + popd +done +mkdir -p build/apidoc +javadoc -d build/apidoc -source 6 -encoding utf-8 \ + -classpath $(build-classpath log4j commons-logging-api slf4j/api asm5/asm asm5/asm-commons) \ + $(for i in xbean-asm-util xbean-classpath xbean-finder xbean-naming xbean-reflect; do find $i/src/main/java -name *.java; done | xargs) + +%install +# jars && poms +install -dm 755 %{buildroot}%{_javadir}/%{name} +install -dm 755 %{buildroot}%{_mavenpomdir}/%{name} +for i in xbean-asm-util xbean-classpath xbean-finder xbean-naming xbean-reflect; do + install -m 0644 $i/$i.jar %{buildroot}%{_javadir}/%{name} + install -m 0644 $i/pom.xml %{buildroot}%{_mavenpomdir}/%{name}/$i.pom + %add_maven_depmap %{name}/$i.pom %{name}/$i.jar +done +install -m 0644 pom.xml %{buildroot}%{_mavenpomdir}/%{name}/%{name}.pom +%add_maven_depmap %{name}/%{name}.pom + +# javadoc +install -dm 755 %{buildroot}/%{_javadocdir}/%{name} +cp -aL build/apidoc/* %{buildroot}/%{_javadocdir}/%{name} +%fdupes -s %{buildroot}/%{_javadocdir}/%{name} + +%files +%license LICENSE +%doc NOTICE +%{_javadir}/%{name} +%{_mavenpomdir}/%{name} +%if %{defined _maven_repository} +%{_mavendepmapfragdir}/%{name} +%else +%{_datadir}/maven-metadata/%{name}.xml* +%endif + +%if %{with spring} +%if 0 +%files blueprint -f .mfiles-blueprint +%license LICENSE +%doc NOTICE %{name}-blueprint/target/restaurant.xsd* +%endif + +%files classloader -f .mfiles-classloader +%license LICENSE +%doc NOTICE + +%files spring -f .mfiles-spring +%license LICENSE +%doc NOTICE + +%files -n maven-%{name}-plugin -f .mfiles-maven-%{name}-plugin +%license LICENSE +%doc NOTICE +%endif + +%files javadoc +%license LICENSE +%doc NOTICE +%{_javadocdir}/%{name} + +%changelog