From bd03bd8cad71355c652a2b5a9c6b9514777f2e69 Mon Sep 17 00:00:00 2001
From: Brian Long <brian@inteligr8.com>
Date: Fri, 30 Sep 2022 22:58:00 -0400
Subject: [PATCH] minor POM fixes

---
 pom.xml | 21 ++++++++++++++-------
 1 file changed, 14 insertions(+), 7 deletions(-)

diff --git a/pom.xml b/pom.xml
index 6de2c78..6c3cf7d 100644
--- a/pom.xml
+++ b/pom.xml
@@ -1,3 +1,4 @@
+<?xml version="1.0" encoding="UTF-8"?>
 <project xmlns="http://maven.apache.org/POM/4.0.0"
 		xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
 		xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 https://maven.apache.org/xsd/maven-4.0.0.xsd">
@@ -43,7 +44,6 @@
 		<maven.compiler.target>11</maven.compiler.target>
 		<maven.compiler.release>11</maven.compiler.release>
 		
-		<aps.port>8081</aps.port>
 		<unimportant.version>[1.0.0,)</unimportant.version>
 	</properties>
 
@@ -60,27 +60,31 @@
 	</dependencyManagement>
 
 	<dependencies>
+		<!--
+			All 'provided' dependencies are already provided by ACS OOTB
+			They are declared here for one of two reasons:
+			(1) Prevent dependencies of other non-provided depenencies from being included
+			(2) Prevent projects depending on this project from including these dupilcate libraries
+		-->
+
 		<dependency>
 			<groupId>org.alfresco</groupId>
 			<artifactId>alfresco-repository</artifactId>
 			<scope>provided</scope>
 		</dependency>
+
 		<dependency>
 			<groupId>com.fasterxml.jackson.jaxrs</groupId>
 			<artifactId>jackson-jaxrs-json-provider</artifactId>
 			<version>${jackson.version}</version>
 		</dependency>
+		
+		<!-- Version managed by acs-community-packaging, yet not included by ACS -->
 		<dependency>
 			<groupId>org.apache.cxf</groupId>
 			<artifactId>cxf-rt-frontend-jaxrs</artifactId>
 		</dependency>
 		
-		<!-- Already provided by ACS -->
-		<!--
-			We need to specify versions here, unlike when we list these in the 
-			dependencyManagement block.  However, that block does not get
-			inherited by dependent projects.
-		-->
 		<dependency>
 			<groupId>org.apache.cxf</groupId>
 			<artifactId>cxf-core</artifactId>
@@ -271,6 +275,9 @@
 		</profile>
 		<profile>
 			<id>acs72</id>
+			<activation>
+				<activeByDefault>true</activeByDefault>
+			</activation>
 			<properties>
 				<alfresco.sdk.version>5.2.0</alfresco.sdk.version>
 				<alfresco.platform.version>7.2.0</alfresco.platform.version>