From 919cc6823f8f70c5b2e9cc5e35650ab1dae42a2f Mon Sep 17 00:00:00 2001
From: Konrad Windszus <konrad.windszus@netcentric.biz>
Date: Tue, 9 Apr 2024 20:28:27 +0200
Subject: [PATCH] Use compile scope to transitively inherit
 acs-aem-commons-bundle

This eases introducing Cloud-only API as the consumer should only depend
on "acs-aem-commons-bundle-cloud" instead of "acs-aem-commons-bundle"
(when running in AEMaaCS)
---
 bundle-cloud/pom.xml | 23 ++++++++++++++++++++++-
 1 file changed, 22 insertions(+), 1 deletion(-)

diff --git a/bundle-cloud/pom.xml b/bundle-cloud/pom.xml
index c20a1df62b..39d181b003 100644
--- a/bundle-cloud/pom.xml
+++ b/bundle-cloud/pom.xml
@@ -66,6 +66,26 @@ Fragment-Host: com.adobe.acs.acs-aem-commons-bundle
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-jar-plugin</artifactId>
             </plugin>
+            <plugin>
+                <groupId>org.apache.maven.plugins</groupId>
+                <artifactId>maven-enforcer-plugin</artifactId>
+                <executions>
+                    <execution>
+                        <id>enforce-banned-dependencies</id>
+                        <configuration>
+                            <rules>
+                                <bannedDependencies>
+                                    <!-- here we need to explicitly list the set of dependencies which 
+                                    are allowed with scope=compile -->
+                                    <includes>
+                                        <include>com.adobe.acs:acs-aem-commons-bundle</include>
+                                    </includes>
+                                </bannedDependencies>
+                            </rules>
+                        </configuration>
+                    </execution>
+                </executions>
+            </plugin>
         </plugins>
     </build>
 
@@ -74,7 +94,8 @@ Fragment-Host: com.adobe.acs.acs-aem-commons-bundle
             <groupId>com.adobe.acs</groupId>
             <artifactId>acs-aem-commons-bundle</artifactId>
             <version>${project.version}</version>
-            <scope>provided</scope>
+            <!-- this dependency should be transitively inherited -->
+            <scope>compile</scope>
         </dependency>
         <!-- Start Test dependencies -->
         <dependency>