Script 'mail_helper' called by obssrc
Hello community,

here is the log from the commit of package javapackages-tools for 
openSUSE:Factory checked in at 2022-05-12 22:57:29
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/javapackages-tools (Old)
 and      /work/SRC/openSUSE:Factory/.javapackages-tools.new.1538 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "javapackages-tools"

Thu May 12 22:57:29 2022 rev:30 rq:975900 version:5.3.1

Changes:
--------
--- /work/SRC/openSUSE:Factory/javapackages-tools/javapackages-tools.changes    
2022-01-03 10:49:08.971574246 +0100
+++ 
/work/SRC/openSUSE:Factory/.javapackages-tools.new.1538/javapackages-tools.changes
  2022-05-12 22:57:33.268570765 +0200
@@ -1,0 +2,10 @@
+Fri Apr 29 10:21:41 UTC 2022 - Fridrich Strba <fst...@suse.com>
+
+- Added patches:
+  * 0001-Let-maven_depmap.py-generate-metadata-with-dependenc.patch
+  * 0002-Do-not-try-to-construct-POM-from-maven-coordinate-st.patch
+  * 0003-Fix-tests-after-the-recent-maven_depmap.py-changes.patch
+    + Let maven_depmap.py generate metadata with dependencies under
+      certain circumstances
+
+-------------------------------------------------------------------

New:
----
  0001-Let-maven_depmap.py-generate-metadata-with-dependenc.patch
  0002-Do-not-try-to-construct-POM-from-maven-coordinate-st.patch
  0003-Fix-tests-after-the-recent-maven_depmap.py-changes.patch

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ javapackages-tools.spec ++++++
--- /var/tmp/diff_new_pack.dw0vXA/_old  2022-05-12 22:57:33.768571436 +0200
+++ /var/tmp/diff_new_pack.dw0vXA/_new  2022-05-12 22:57:33.776571447 +0200
@@ -1,7 +1,7 @@
 #
 # spec file
 #
-# Copyright (c) 2021 SUSE LLC
+# Copyright (c) 2022 SUSE LLC
 #
 # All modifications and additions to the file contributed by third parties
 # remain the property of their copyright owners, unless otherwise agreed
@@ -45,6 +45,10 @@
 #PATCH-FIX-SUSE: SUSE did not bump epoch of openjdk packages, whereas Fedora 
did
 #               Avoid generating unresolvable requires
 Patch2:         suse-no-epoch.patch
+#PATCH-FIX-SUSE: Let maven_depmap.py generate metadata with dependencies under 
certain circumstances
+Patch3:         0001-Let-maven_depmap.py-generate-metadata-with-dependenc.patch
+Patch4:         0002-Do-not-try-to-construct-POM-from-maven-coordinate-st.patch
+Patch5:         0003-Fix-tests-after-the-recent-maven_depmap.py-changes.patch
 BuildRequires:  asciidoc
 BuildRequires:  fdupes
 BuildRequires:  perl

++++++ 0001-Let-maven_depmap.py-generate-metadata-with-dependenc.patch ++++++
>From d2db56cd30a48bd2ece7a5112e67c80417406a36 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Fridrich=20=C5=A0trba?= <fridrich.st...@bluewin.ch>
Date: Fri, 29 Apr 2022 12:08:21 +0200
Subject: [PATCH 1/3] Let maven_depmap.py generate metadata with dependencies
 under certain circumstances

---
 java-utils/maven_depmap.py | 43 +++++++++++++++++++++++++++++++++++++-
 1 file changed, 42 insertions(+), 1 deletion(-)

diff --git a/java-utils/maven_depmap.py b/java-utils/maven_depmap.py
index 92e0ff4c..f684a728 100644
--- a/java-utils/maven_depmap.py
+++ b/java-utils/maven_depmap.py
@@ -49,6 +49,7 @@ from copy import deepcopy
 from javapackages.maven.pom import POM
 from javapackages.metadata.artifact import MetadataArtifact
 from javapackages.metadata.alias import MetadataAlias
+from javapackages.metadata.dependency import MetadataDependency
 from javapackages.metadata.metadata import Metadata
 
 from javapackages.common.exception import JavaPackagesToolsException
@@ -132,6 +133,38 @@ def _make_files_versioned(versions, pom_path, jar_path, 
pom_base, jar_base):
     # return paths to versioned, but regular files (not symlinks)
     return ret_pom_path, ret_jar_path
 
+def _resolve_deps(pom):
+    deps = []
+    depm = []
+    props = {}
+
+    deps.extend([x for x in pom.dependencies])
+    depm.extend([x for x in pom.dependencyManagement])
+    props = pom.properties
+    if pom.groupId:
+        props["project.groupId"] = pom.groupId
+    if pom.artifactId:
+        props["project.artifactId"] = pom.artifactId
+    if pom.version:
+        props["project.version"] = pom.version
+
+    for d in deps:
+        d.interpolate(props)
+
+    for dm in depm:
+        dm.interpolate(props)
+
+    # apply dependencyManagement on deps
+    for d in deps:
+        for dm in depm:
+            if d.compare_to(dm):
+                d.merge_with(dm)
+                break
+
+    # only deps with scope "compile" or "runtime" are interesting
+    deps = [x for x in deps if x.scope in ["", "compile", "runtime"]]
+
+    return deps
 
 # Add a file to a ZIP archive (or JAR, WAR, ...) unless the file
 # already exists in the archive.  Provided by Tomas Radej.
@@ -279,7 +312,15 @@ def _main():
     if namespace:
         artifact.namespace = namespace
 
-    artifact.properties["xmvn.resolver.disableEffectivePom"] = "true"
+    pom = POM(pom_path)
+    if pom.parent or pom.packaging == "pom":
+        artifact.properties["xmvn.resolver.disableEffectivePom"] = "true"
+    else:
+        deps = []
+        for d in _resolve_deps(pom):
+            deps.append(MetadataDependency.from_mvn_dependency(d))
+        if deps:
+            artifact.dependencies = set(deps)
 
 
     buildroot = os.environ.get('RPM_BUILD_ROOT')
-- 
2.36.0


++++++ 0002-Do-not-try-to-construct-POM-from-maven-coordinate-st.patch ++++++
>From c0a4b557eb9a3b20c36fabe67ffac953ba78d2a5 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Fridrich=20=C5=A0trba?= <fridrich.st...@bluewin.ch>
Date: Fri, 29 Apr 2022 13:27:10 +0200
Subject: [PATCH 2/3] Do not try to construct POM from maven coordinate string

---
 java-utils/maven_depmap.py | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/java-utils/maven_depmap.py b/java-utils/maven_depmap.py
index f684a728..367632e6 100644
--- a/java-utils/maven_depmap.py
+++ b/java-utils/maven_depmap.py
@@ -312,8 +312,10 @@ def _main():
     if namespace:
         artifact.namespace = namespace
 
-    pom = POM(pom_path)
-    if pom.parent or pom.packaging == "pom":
+    pom = None
+    if have_pom:
+        pom = POM(pom_path)
+    if not pom or pom.parent or pom.packaging == "pom":
         artifact.properties["xmvn.resolver.disableEffectivePom"] = "true"
     else:
         deps = []
-- 
2.36.0


++++++ 0003-Fix-tests-after-the-recent-maven_depmap.py-changes.patch ++++++
>From 527108b4d064417ab41da7abd70a47f58fcf91d6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Fridrich=20=C5=A0trba?= <fridrich.st...@bluewin.ch>
Date: Fri, 29 Apr 2022 13:27:33 +0200
Subject: [PATCH 3/3] Fix tests after the recent maven_depmap.py changes

---
 test/data/maven_depmap/test_alias_extension-want.xml    | 6 ------
 test/data/maven_depmap/test_basic-want.xml              | 6 ------
 test/data/maven_depmap/test_buildroot-want.xml          | 6 ------
 test/data/maven_depmap/test_versioned_with_pom-want.xml | 6 ------
 test/data/maven_depmap/test_war-want.xml                | 6 ------
 5 files changed, 30 deletions(-)

diff --git a/test/data/maven_depmap/test_alias_extension-want.xml 
b/test/data/maven_depmap/test_alias_extension-want.xml
index 6b0d1aee..d502fdf8 100644
--- a/test/data/maven_depmap/test_alias_extension-want.xml
+++ b/test/data/maven_depmap/test_alias_extension-want.xml
@@ -13,9 +13,6 @@
              </ns1:alias>
          </ns1:aliases>
          <ns1:path>%s/usr/share/java/commons-io.jar</ns1:path>
-         <ns1:properties>
-            
<xmvn.resolver.disableEffectivePom>true</xmvn.resolver.disableEffectivePom>
-         </ns1:properties>
       </ns1:artifact>
       <ns1:artifact>
          <ns1:groupId>alias</ns1:groupId>
@@ -30,9 +27,6 @@
              </ns1:alias>
          </ns1:aliases>
          <ns1:path>%s/JPP-alias.pom</ns1:path>
-         <ns1:properties>
-            
<xmvn.resolver.disableEffectivePom>true</xmvn.resolver.disableEffectivePom>
-         </ns1:properties>
       </ns1:artifact>
    </ns1:artifacts>
 </ns1:metadata>
diff --git a/test/data/maven_depmap/test_basic-want.xml 
b/test/data/maven_depmap/test_basic-want.xml
index 540f076a..200044bf 100644
--- a/test/data/maven_depmap/test_basic-want.xml
+++ b/test/data/maven_depmap/test_basic-want.xml
@@ -6,9 +6,6 @@
          <ns1:artifactId>bndlib</ns1:artifactId>
          <ns1:version>1.50.0</ns1:version>
          <ns1:path>%s/usr/share/java/bndlib.jar</ns1:path>
-         <ns1:properties>
-            
<xmvn.resolver.disableEffectivePom>true</xmvn.resolver.disableEffectivePom>
-         </ns1:properties>
       </ns1:artifact>
       <ns1:artifact>
          <ns1:groupId>biz.aQute</ns1:groupId>
@@ -16,9 +13,6 @@
          <ns1:extension>pom</ns1:extension>
          <ns1:version>1.50.0</ns1:version>
          <ns1:path>%s/JPP-bndlib.pom</ns1:path>
-         <ns1:properties>
-            
<xmvn.resolver.disableEffectivePom>true</xmvn.resolver.disableEffectivePom>
-         </ns1:properties>
       </ns1:artifact>
    </ns1:artifacts>
 </ns1:metadata>
diff --git a/test/data/maven_depmap/test_buildroot-want.xml 
b/test/data/maven_depmap/test_buildroot-want.xml
index 604046a5..9fa588d4 100644
--- a/test/data/maven_depmap/test_buildroot-want.xml
+++ b/test/data/maven_depmap/test_buildroot-want.xml
@@ -7,9 +7,6 @@
          <ns1:version>17</ns1:version>
          <ns1:extension>war</ns1:extension>
          <ns1:path>%s/usr/share/java/commons-war.war</ns1:path>
-         <ns1:properties>
-            
<xmvn.resolver.disableEffectivePom>true</xmvn.resolver.disableEffectivePom>
-         </ns1:properties>
       </ns1:artifact>
       <ns1:artifact>
          <ns1:groupId>commons</ns1:groupId>
@@ -17,9 +14,6 @@
          <ns1:version>17</ns1:version>
          <ns1:extension>pom</ns1:extension>
          <ns1:path>/usr/share/maven-poms/JPP-commons-war.pom</ns1:path>
-         <ns1:properties>
-            
<xmvn.resolver.disableEffectivePom>true</xmvn.resolver.disableEffectivePom>
-         </ns1:properties>
       </ns1:artifact>
    </ns1:artifacts>
 </ns1:metadata>
diff --git a/test/data/maven_depmap/test_versioned_with_pom-want.xml 
b/test/data/maven_depmap/test_versioned_with_pom-want.xml
index c2b344e1..8d259ab9 100644
--- a/test/data/maven_depmap/test_versioned_with_pom-want.xml
+++ b/test/data/maven_depmap/test_versioned_with_pom-want.xml
@@ -9,9 +9,6 @@
             <ns1:version>2013.10</ns1:version>
          </ns1:compatVersions>
          <ns1:path>%s/usr/share/java/testversioned-2013.10.jar</ns1:path>
-         <ns1:properties>
-            
<xmvn.resolver.disableEffectivePom>true</xmvn.resolver.disableEffectivePom>
-         </ns1:properties>
       </ns1:artifact>
       <ns1:artifact>
          <ns1:groupId>alias</ns1:groupId>
@@ -22,9 +19,6 @@
             <ns1:version>2013.10</ns1:version>
          </ns1:compatVersions>
          <ns1:path>%s/JPP-testversioned-2013.10.pom</ns1:path>
-         <ns1:properties>
-            
<xmvn.resolver.disableEffectivePom>true</xmvn.resolver.disableEffectivePom>
-         </ns1:properties>
       </ns1:artifact>
    </ns1:artifacts>
 </ns1:metadata>
diff --git a/test/data/maven_depmap/test_war-want.xml 
b/test/data/maven_depmap/test_war-want.xml
index cac27a84..e8e2e56b 100644
--- a/test/data/maven_depmap/test_war-want.xml
+++ b/test/data/maven_depmap/test_war-want.xml
@@ -7,9 +7,6 @@
          <ns1:version>17</ns1:version>
          <ns1:extension>war</ns1:extension>
          <ns1:path>%s/usr/share/java/commons-war.war</ns1:path>
-         <ns1:properties>
-            
<xmvn.resolver.disableEffectivePom>true</xmvn.resolver.disableEffectivePom>
-         </ns1:properties>
       </ns1:artifact>
       <ns1:artifact>
          <ns1:groupId>commons</ns1:groupId>
@@ -17,9 +14,6 @@
          <ns1:extension>pom</ns1:extension>
          <ns1:version>17</ns1:version>
          <ns1:path>%s/JPP-commons-war.pom</ns1:path>
-         <ns1:properties>
-            
<xmvn.resolver.disableEffectivePom>true</xmvn.resolver.disableEffectivePom>
-         </ns1:properties>
       </ns1:artifact>
    </ns1:artifacts>
 </ns1:metadata>
-- 
2.36.0

Reply via email to