Merge pull request #41902 from aloubyansky/gradle-quarkusConditionalC…

…onfig Use quarkusConditional prefix for Quarkus conditional dependency configurations
quarkusio · Jul 15, 2024 · d7b84f6 · d7b84f6
2 parents 19c0e27 + 647a3d5
commit d7b84f6
Showing 1 changed file with 26 additions and 2 deletions.
diff --git a/...adle-model/src/main/java/io/quarkus/gradle/dependency/ConditionalDependenciesEnabler.java b/...adle-model/src/main/java/io/quarkus/gradle/dependency/ConditionalDependenciesEnabler.java
@@ -170,8 +170,7 @@ private void queueConditionalDependency(ExtensionDependency<?> extension, Depend
     private Configuration createConditionalDependenciesConfiguration(Project project, Dependency conditionalDep) {
         // previously we used a detached configuration here but apparently extendsFrom(enforcedPlatforms)
         // wouldn't actually enforce platforms on a detached configuration
-        final String name = conditionalDep.getGroup() + ":" + conditionalDep.getName() + ":" + conditionalDep.getVersion()
-                + "Configuration";
+        var name = getConditionalConfigurationName(conditionalDep);
         var config = project.getConfigurations().findByName(name);
         if (config == null) {
             project.getConfigurations().register(name, configuration -> {
@@ -184,6 +183,31 @@ private Configuration createConditionalDependenciesConfiguration(Project project
         return config;
     }
 
+    private static String getConditionalConfigurationName(Dependency conditionalDep) {
+        var name = new StringBuilder().append("quarkusConditional");
+        appendCapitalized(name, conditionalDep.getGroup());
+        appendCapitalized(name, conditionalDep.getName());
+        appendCapitalized(name, conditionalDep.getVersion());
+        return name.append("Configuration").toString();
+    }
+
+    private static void appendCapitalized(StringBuilder sb, String part) {
+        if (part != null && !part.isEmpty()) {
+            boolean toUpperCase = true;
+            for (int i = 0; i < part.length(); ++i) {
+                var c = part.charAt(i);
+                if (toUpperCase) {
+                    sb.append(Character.toUpperCase(c));
+                    toUpperCase = false;
+                } else if (c == '.' || c == '-') {
+                    toUpperCase = true;
+                } else {
+                    sb.append(c);
+                }
+            }
+        }
+    }
+
     private void enableConditionalDependency(ModuleVersionIdentifier dependency) {
         final Set<ExtensionDependency<?>> extensions = featureVariants.remove(getFeatureKey(dependency));
         if (extensions == null) {