fix: introduces package-info together with About class

arquillian · Feb 19, 2020 · 1a8677b · 1a8677b
1 parent d58509a
commit 1a8677b
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/...ss/arquillian/extension/jacoco/About.java → ...rquillian/extension/jacoco/asm/About.java b/...ss/arquillian/extension/jacoco/About.java → ...rquillian/extension/jacoco/asm/About.java
@@ -1,4 +1,4 @@
-package org.jboss.arquillian.extension.jacoco;
+package org.jboss.arquillian.extension.jacoco.asm;
 
 /**
  * This file exists purely to satisfy Maven release validation process, as jar modules are expected to
@@ -9,5 +9,5 @@
  *
  * As a workaround, this modules provides an alternate `with-asm` flavour that includes "private" ASM and JaCoCo packages, shaded via `maven-shade-plugin`.
  */
-public interface About {
+public class About {
 }
diff --git a/...jacoco-with-asm/src/main/java/org/jboss/arquillian/extension/jacoco/asm/package-info.java b/...jacoco-with-asm/src/main/java/org/jboss/arquillian/extension/jacoco/asm/package-info.java
@@ -0,0 +1,10 @@
+/**
+ * This file exists purely to satisfy Maven release validation process, as jar modules are expected to
+ * contain -sources and -javadoc JARs.
+ *
+ * JaCoCo requires ASM to work properly. Since ASM is also used by many other libraries like Apache CXF, you might run into version conflicts.
+ * E.g. JBoss EAP 6.4 ships CXF 2.7 which requires ASM 3 but JaCoCo requires ASM 7+.
+ *
+ * As a workaround, this modules provides an alternate `with-asm` flavour that includes "private" ASM and JaCoCo packages, shaded via `maven-shade-plugin`.
+ */
+package org.jboss.arquillian.extension.jacoco.asm;