This is an automated email from the ASF dual-hosted git repository.

pkarwasz pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/logging-parent.git


The following commit(s) were added to refs/heads/main by this push:
     new 9e3dda8  Fix deployment of test artifacts (#360)
9e3dda8 is described below

commit 9e3dda83cba64360283ed751ba05544add3fad1e
Author: Piotr P. Karwasz <[email protected]>
AuthorDate: Mon Mar 24 11:09:29 2025 +0100

    Fix deployment of test artifacts (#360)
    
    * Fix deployment of test artifacts
    
    Due to a difference between plugins used, if the `deploy` profile is 
activated, artifacts with `maven.deploy.skip` are being **deployed**.
    This change fixes it.
    
    * Add changelog entry
    
    * Improve changelog
    
    ---------
    
    Co-authored-by: Volkan Yazıcı <[email protected]>
---
 pom.xml                                  | 4 ++++
 src/changelog/.12.x.x/fix_deployment.xml | 9 +++++++++
 2 files changed, 13 insertions(+)

diff --git a/pom.xml b/pom.xml
index a1d0d83..a9635bb 100644
--- a/pom.xml
+++ b/pom.xml
@@ -168,6 +168,9 @@
     <!-- `bnd-maven-plugin` 7.x requires a Maven version `3.8.1` or higher -->
     <minimalMavenBuildVersion>3.8.1</minimalMavenBuildVersion>
 
+    <!-- The same property will be used by `maven-deploy-plugin` and 
`nexus-staging-maven-plugin` -->
+    <maven.deploy.skip>false</maven.deploy.skip>
+
     <!-- JPMS and OSGi options -->
     <!-- Overrides some options in multi-release JARs -->
     <bnd-multi-release>false</bnd-multi-release>
@@ -1363,6 +1366,7 @@
                 </goals>
                 <configuration>
                   <nexusUrl>https://repository.apache.org</nexusUrl>
+                  
<skipNexusStagingDeployMojo>${maven.deploy.skip}</skipNexusStagingDeployMojo>
                   <serverId>${distMgmtReleasesId}</serverId>
                 </configuration>
               </execution>
diff --git a/src/changelog/.12.x.x/fix_deployment.xml 
b/src/changelog/.12.x.x/fix_deployment.xml
new file mode 100644
index 0000000..f679543
--- /dev/null
+++ b/src/changelog/.12.x.x/fix_deployment.xml
@@ -0,0 +1,9 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<entry xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance";
+       xmlns="https://logging.apache.org/xml/ns";
+       xsi:schemaLocation="https://logging.apache.org/xml/ns 
https://logging.apache.org/xml/ns/log4j-changelog-0.xsd";
+       type="fixed">
+  <issue id="360" link="https://github.com/apache/logging-parent/pull/360"/>
+  <description format="asciidoc">Use the `maven.deploy.skip` Maven property in 
`nexus-staging-maven-plugin`.
+This effectively fixes the skipping of test artifacts' 
deployments.</description>
+</entry>

Reply via email to