HSEARCH-4688 Upgrade to Jandex 3.0

hibernate · Jan 2, 2023 · fa0c31c · fa0c31c
1 parent c5a1265
commit fa0c31c
Show file tree

Hide file tree

Showing 8 changed files with 19 additions and 14 deletions.
diff --git a/integrationtest/showcase/library/pom.xml b/integrationtest/showcase/library/pom.xml
@@ -123,7 +123,7 @@
     <build>
         <plugins>
             <plugin>
-                <groupId>org.jboss.jandex</groupId>
+                <groupId>io.smallrye</groupId>
                 <artifactId>jandex-maven-plugin</artifactId>
                 <executions>
                     <execution>

diff --git a/orm6/build/parents/integrationtest/pom.xml b/orm6/build/parents/integrationtest/pom.xml
@@ -67,7 +67,7 @@
                          (which, unfortunately, might be different from the version used in ORM 5.x
                          on which our utils are aligned) -->
                     <exclusion>
-                        <groupId>org.jboss</groupId>
+                        <groupId>io.smallrye</groupId>
                         <artifactId>jandex</artifactId>
                     </exclusion>
                     <exclusion>

diff --git a/orm6/build/parents/internal/pom.xml b/orm6/build/parents/internal/pom.xml
@@ -67,7 +67,7 @@
                          (which, unfortunately, might be different from the version used in ORM 5.x
                          on which our utils are aligned) -->
                     <exclusion>
-                        <groupId>org.jboss</groupId>
+                        <groupId>io.smallrye</groupId>
                         <artifactId>jandex</artifactId>
                     </exclusion>
                     <exclusion>

diff --git a/orm6/build/parents/public/pom.xml b/orm6/build/parents/public/pom.xml
@@ -67,7 +67,7 @@
                          (which, unfortunately, might be different from the version used in ORM 5.x
                          on which our utils are aligned) -->
                     <exclusion>
-                        <groupId>org.jboss</groupId>
+                        <groupId>io.smallrye</groupId>
                         <artifactId>jandex</artifactId>
                     </exclusion>
                     <exclusion>

diff --git a/pom.xml b/pom.xml
@@ -257,10 +257,9 @@
              when the versions start diverging, thanks to the maven-enforcer-plugin
              and to our own explicit dependencies in the relevant artifacts.
           -->
-        <version.org.jboss.jandex>2.4.2.Final</version.org.jboss.jandex>
         <version.org.hibernate.commons.annotations>${version.org.hibernate.commons.annotations.orm6}</version.org.hibernate.commons.annotations>
         <version.net.bytebuddy>${version.net.bytebuddy.orm6}</version.net.bytebuddy>
-        <version.io.smallrye.jandex>3.0.0</version.io.smallrye.jandex>
+        <version.io.smallrye.jandex>3.0.3</version.io.smallrye.jandex>
 
         <!-- >>> ORM 6 / Jakarta Persistence -->
         <version.org.hibernate.orm>6.2.0-SNAPSHOT</version.org.hibernate.orm>
@@ -355,7 +354,7 @@
         <version.help.plugin>3.3.0</version.help.plugin>
         <version.install.plugin>3.1.0</version.install.plugin>
         <version.io.takari.maven>0.7.7</version.io.takari.maven>
-        <version.jandex.plugin>1.2.3</version.jandex.plugin>
+        <version.jandex.plugin>3.0.3</version.jandex.plugin>
         <version.japicmp.plugin>0.17.1</version.japicmp.plugin>
         <version.jar.plugin>3.3.0</version.jar.plugin>
         <version.javadoc.plugin>3.4.1</version.javadoc.plugin>
@@ -1572,6 +1571,12 @@
                                     <version>${maven.min.version}</version>
                                 </requireMavenVersion>
                                 <banDuplicatePomDependencyVersions />
+                                <bannedDependencies>
+                                    <excludes>
+                                        <!-- We use Jandex 3 instead: io.smallrye:jandex, io.smallrye:jandex-maven-plugin -->
+                                        <exclude>org.jboss:jandex*</exclude>
+                                    </excludes>
+                                </bannedDependencies>
                             </rules>
                         </configuration>
                     </execution>
@@ -2023,7 +2028,7 @@
                     <version>1.0.2</version>
                 </plugin>
                 <plugin>
-                    <groupId>org.jboss.jandex</groupId>
+                    <groupId>io.smallrye</groupId>
                     <artifactId>jandex-maven-plugin</artifactId>
                     <version>${version.jandex.plugin}</version>
                     <executions>

diff --git a/util/common/pom.xml b/util/common/pom.xml
@@ -26,10 +26,10 @@
             <artifactId>jboss-logging-annotations</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.jboss</groupId>
+            <groupId>io.smallrye</groupId>
             <artifactId>jandex</artifactId>
             <!-- DO NOT REMOVE and DO NOT MANAGE the version of this dependency. See the version property declaration. -->
-            <version>${version.org.jboss.jandex}</version>
+            <version>${version.io.smallrye.jandex}</version>
         </dependency>
 
         <dependency>

diff --git a/util/common/src/main/java/org/hibernate/search/util/common/jar/impl/JandexUtils.java b/util/common/src/main/java/org/hibernate/search/util/common/jar/impl/JandexUtils.java
@@ -70,7 +70,7 @@ public static Set<DotName> findAnnotatedAnnotationsAndContaining(IndexView index
 		for ( AnnotationInstance retentionAnnotation : index.getAnnotations( metaAnnotation ) ) {
 			ClassInfo annotation = retentionAnnotation.target().asClass();
 			annotations.add( annotation.name() );
-			AnnotationInstance repeatable = annotation.classAnnotation( REPEATABLE );
+			AnnotationInstance repeatable = annotation.declaredAnnotation( REPEATABLE );
 			if ( repeatable != null ) {
 				Type containing = repeatable.value().asClass();
 				annotations.add( containing.name() );

diff --git a/util/internal/test/common/pom.xml b/util/internal/test/common/pom.xml
@@ -76,10 +76,10 @@
             <artifactId>guava</artifactId>
         </dependency>
         <dependency>
-            <groupId>org.jboss</groupId>
+            <groupId>io.smallrye</groupId>
             <artifactId>jandex</artifactId>
             <!-- DO NOT MANAGE this dependency. See the version property declaration. -->
-            <version>${version.org.jboss.jandex}</version>
+            <version>${version.io.smallrye.jandex}</version>
         </dependency>
         <dependency>
             <groupId>net.bytebuddy</groupId>
@@ -97,7 +97,7 @@
             </plugin>
             <!-- This is only necessary to implement a few tests in other Maven modules; see JandexUtilsTest -->
             <plugin>
-                <groupId>org.jboss.jandex</groupId>
+                <groupId>io.smallrye</groupId>
                 <artifactId>jandex-maven-plugin</artifactId>
                 <executions>
                     <execution>