From 022f0216295a14924eed5f89e6cd651e4295da02 Mon Sep 17 00:00:00 2001
From: Zhe Sun <zhe@vaadin.com>
Date: Thu, 29 Jun 2023 13:06:01 +0300
Subject: [PATCH] chore: group platform modules in a profile, remove gradle
 module activation rule (CP: 23.3)

---
 hilla-react/pom.xml                           |  3 ++
 hilla-spring-boot-starter/pom.xml             |  4 ++
 hilla/pom.xml                                 |  4 ++
 pom.xml                                       | 41 ++++++++++++-------
 .../templates/template-hilla-bom.xml          |  4 ++
 .../template-hilla-maven-plugin-pom.xml       |  1 +
 .../template-servlet-containers-tests-pom.xml |  2 +-
 vaadin-platform-hybrid-test/pom.xml           |  2 +-
 vaadin-platform-test/pom.xml                  |  2 +-
 9 files changed, 46 insertions(+), 17 deletions(-)

diff --git a/hilla-react/pom.xml b/hilla-react/pom.xml
index 99a56d1ad..fd67e8408 100644
--- a/hilla-react/pom.xml
+++ b/hilla-react/pom.xml
@@ -14,6 +14,9 @@
     <description>Hilla Platform for React-based Applications</description>
     <version>${hilla.version}</version>
     <url>https://hilla.dev</url>
+    <properties>
+        <skipNexusStagingDeployMojo>false</skipNexusStagingDeployMojo>
+    </properties>
     <distributionManagement>
         <repository>
             <id>vaadin-prereleases</id>
diff --git a/hilla-spring-boot-starter/pom.xml b/hilla-spring-boot-starter/pom.xml
index 74236cccb..a95592b93 100644
--- a/hilla-spring-boot-starter/pom.xml
+++ b/hilla-spring-boot-starter/pom.xml
@@ -14,6 +14,10 @@
     <description>Spring Boot Starter for Hilla applications.</description>
     <version>${hilla.version}</version>
 
+    <properties>
+        <skipNexusStagingDeployMojo>false</skipNexusStagingDeployMojo>
+    </properties>
+
     <repositories>
         <!-- The order of definitions matters. Explicitly defining central here to make sure it has the highest priority. -->
 
diff --git a/hilla/pom.xml b/hilla/pom.xml
index 7beea8f39..c1c8b717f 100644
--- a/hilla/pom.xml
+++ b/hilla/pom.xml
@@ -14,6 +14,10 @@
     <description>Hilla Platform</description>
     <version>${hilla.version}</version>
     <url>https://hilla.dev</url>
+
+    <properties>
+        <skipNexusStagingDeployMojo>false</skipNexusStagingDeployMojo>
+    </properties>
     <distributionManagement>
         <repository>
             <id>vaadin-prereleases</id>
diff --git a/pom.xml b/pom.xml
index 650ea3bdd..de2b7df94 100644
--- a/pom.xml
+++ b/pom.xml
@@ -29,17 +29,6 @@
         </repository>
     </distributionManagement>
 
-    <modules>
-        <module>vaadin</module>
-        <module>vaadin-spring-bom</module>
-        <module>vaadin-bom</module>
-        <module>vaadin-core</module>
-        <module>vaadin-maven-plugin</module>
-        <module>vaadin-testbench</module>
-        <module>vaadin-spring-boot-starter</module>
-        <module>vaadin-platform-javadoc</module>
-    </modules>
-
     <profiles>
         <profile>
             <id>sbom</id>
@@ -49,9 +38,6 @@
         </profile>
         <profile>
             <id>gradle</id>
-            <activation>
-                <jdk>(,17]</jdk>
-            </activation>
             <modules>
                 <module>vaadin-gradle-plugin</module>
                 <module>vaadin-gradle-plugin/pom-portal.xml</module>
@@ -75,6 +61,33 @@
                 <module>hilla-spring-boot-starter</module>
                 <module>hilla-maven-plugin</module>
             </modules>
+            <!-- This prevents deploying all com.vaadin modules, but we need to set it to true in
+            all hilla modules  -->
+            <properties>
+                <skipNexusStagingDeployMojo>true</skipNexusStagingDeployMojo>
+            </properties>
+        </profile>
+        <!-- separate platform modules for active by default it must be after hilla modules -->
+        <profile>
+            <id>platform</id>
+            <activation>
+                <property>
+                    <name>!skipPlatform</name>
+                </property>
+            </activation>
+            <modules>
+                <module>vaadin</module>
+                <module>vaadin-spring-bom</module>
+                <module>vaadin-bom</module>
+                <module>vaadin-core</module>
+                <module>vaadin-maven-plugin</module>
+                <module>vaadin-testbench</module>
+                <module>vaadin-spring-boot-starter</module>
+                <module>vaadin-platform-javadoc</module>
+            </modules>
+            <properties>
+                <skipNexusStagingDeployMojo>false</skipNexusStagingDeployMojo>
+            </properties>
         </profile>
         <!-- Needed so as we can skip this module on releasing and still nexus stating works -->
         <profile>
diff --git a/scripts/generator/templates/template-hilla-bom.xml b/scripts/generator/templates/template-hilla-bom.xml
index a1cb23f5b..c0526ed71 100644
--- a/scripts/generator/templates/template-hilla-bom.xml
+++ b/scripts/generator/templates/template-hilla-bom.xml
@@ -14,6 +14,10 @@
     <version>${hilla.version}</version>
     <url>https://hilla.dev</url>
 
+    <properties>
+        <skipNexusStagingDeployMojo>false</skipNexusStagingDeployMojo>
+    </properties>
+
     <distributionManagement>
         <repository>
             <id>vaadin-prereleases</id>
diff --git a/scripts/generator/templates/template-hilla-maven-plugin-pom.xml b/scripts/generator/templates/template-hilla-maven-plugin-pom.xml
index 0e71373d0..a74c743c1 100644
--- a/scripts/generator/templates/template-hilla-maven-plugin-pom.xml
+++ b/scripts/generator/templates/template-hilla-maven-plugin-pom.xml
@@ -24,6 +24,7 @@
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
         <project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding>
         <maven.version>3.8.4</maven.version>
+        <skipNexusStagingDeployMojo>false</skipNexusStagingDeployMojo>
     </properties>
 
     <dependencies>
diff --git a/scripts/generator/templates/template-servlet-containers-tests-pom.xml b/scripts/generator/templates/template-servlet-containers-tests-pom.xml
index 853e3e9fb..ba0c2ad5d 100644
--- a/scripts/generator/templates/template-servlet-containers-tests-pom.xml
+++ b/scripts/generator/templates/template-servlet-containers-tests-pom.xml
@@ -18,7 +18,7 @@
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
         <project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding>
         <failOnMissingWebXml>false</failOnMissingWebXml>
-        <maven.deploy.skip>true</maven.deploy.skip>
+        <skipNexusStagingDeployMojo>false</skipNexusStagingDeployMojo>
         <!-- this version is needed for app.bndrun file -->
         {{javadeps}}
     </properties>
diff --git a/vaadin-platform-hybrid-test/pom.xml b/vaadin-platform-hybrid-test/pom.xml
index 52a3039b7..0767bd9e8 100644
--- a/vaadin-platform-hybrid-test/pom.xml
+++ b/vaadin-platform-hybrid-test/pom.xml
@@ -18,7 +18,7 @@
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
         <project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding>
         <failOnMissingWebXml>false</failOnMissingWebXml>
-        <maven.deploy.skip>true</maven.deploy.skip>
+        <skipNexusStagingDeployMojo>false</skipNexusStagingDeployMojo>
     </properties>
     <dependencyManagement>
         <dependencies>
diff --git a/vaadin-platform-test/pom.xml b/vaadin-platform-test/pom.xml
index 9c086ac87..47a7180cb 100644
--- a/vaadin-platform-test/pom.xml
+++ b/vaadin-platform-test/pom.xml
@@ -18,7 +18,7 @@
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
         <project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding>
         <failOnMissingWebXml>false</failOnMissingWebXml>
-        <maven.deploy.skip>true</maven.deploy.skip>
+        <skipNexusStagingDeployMojo>false</skipNexusStagingDeployMojo>
         <osgi.bundle.version>${parsedVersion.majorVersion}.${parsedVersion.minorVersion}.${parsedVersion.incrementalVersion}</osgi.bundle.version>
     </properties>
     <dependencyManagement>