Script 'mail_helper' called by obssrc
Hello community,

here is the log from the commit of package python-pbr for openSUSE:Factory 
checked in at 2022-10-29 20:16:06
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/python-pbr (Old)
 and      /work/SRC/openSUSE:Factory/.python-pbr.new.2275 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "python-pbr"

Sat Oct 29 20:16:06 2022 rev:61 rq:1032046 version:5.11.0

Changes:
--------
--- /work/SRC/openSUSE:Factory/python-pbr/python-pbr.changes    2022-08-22 
11:04:51.245664987 +0200
+++ /work/SRC/openSUSE:Factory/.python-pbr.new.2275/python-pbr.changes  
2022-10-29 20:17:07.534216683 +0200
@@ -1,0 +2,7 @@
+Fri Oct 28 19:46:28 UTC 2022 - Yogalakshmi Arunachalam <yarunacha...@suse.com>
+
+- Update to 5.11.0 
+  * Fix symbol identification in multiline message
+  * Replace deprecated readfp method with read_file
+
+-------------------------------------------------------------------

Old:
----
  pbr-5.10.0.tar.gz

New:
----
  pbr-5.11.0.tar.gz

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

Other differences:
------------------
++++++ python-pbr.spec ++++++
--- /var/tmp/diff_new_pack.nGJAdy/_old  2022-10-29 20:17:07.994219134 +0200
+++ /var/tmp/diff_new_pack.nGJAdy/_new  2022-10-29 20:17:07.998219155 +0200
@@ -26,7 +26,7 @@
 %bcond_with test
 %endif
 Name:           python-pbr%{psuffix}
-Version:        5.10.0
+Version:        5.11.0
 Release:        0
 Summary:        Python Build Reasonableness
 License:        Apache-2.0

++++++ pbr-5.10.0.tar.gz -> pbr-5.11.0.tar.gz ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/pbr-5.10.0/AUTHORS new/pbr-5.11.0/AUTHORS
--- old/pbr-5.10.0/AUTHORS      2022-08-11 16:02:39.000000000 +0200
+++ new/pbr-5.11.0/AUTHORS      2022-10-20 13:02:42.000000000 +0200
@@ -146,6 +146,7 @@
 dineshbhor <dinesh.b...@nttdata.com>
 jiansong <jian.s...@easystack.cn>
 lifeless <robe...@robertcollins.net>
+ljhuang <huang.liu...@99cloud.net>
 manchandavishal <manchandavishal...@gmail.com>
 melanie witt <melw...@yahoo-inc.com>
 melissaml <ma....@99cloud.net>
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/pbr-5.10.0/ChangeLog new/pbr-5.11.0/ChangeLog
--- old/pbr-5.10.0/ChangeLog    2022-08-11 16:02:39.000000000 +0200
+++ new/pbr-5.11.0/ChangeLog    2022-10-20 13:02:42.000000000 +0200
@@ -1,11 +1,18 @@
 CHANGES
 =======
 
+5.11.0
+------
+
+* Fix symbol identification in multiline message
+* Replace deprecated readfp method with read\_file
+
 5.10.0
 ------
 
 * Specify Changelog procedure
 * Allow leading spaces when determining symbols
+* Use stdlib importlib.metadata where possible
 * Adding python classifiers py38 & py39
 
 5.9.0
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/pbr-5.10.0/PKG-INFO new/pbr-5.11.0/PKG-INFO
--- old/pbr-5.10.0/PKG-INFO     2022-08-11 16:02:39.758154200 +0200
+++ new/pbr-5.11.0/PKG-INFO     2022-10-20 13:02:42.879536000 +0200
@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: pbr
-Version: 5.10.0
+Version: 5.11.0
 Summary: Python Build Reasonableness
 Home-page: https://docs.openstack.org/pbr/latest/
 Author: OpenStack
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/pbr-5.10.0/pbr/packaging.py 
new/pbr-5.11.0/pbr/packaging.py
--- old/pbr-5.10.0/pbr/packaging.py     2022-08-11 16:01:20.000000000 +0200
+++ new/pbr-5.11.0/pbr/packaging.py     2022-10-20 13:02:15.000000000 +0200
@@ -692,12 +692,14 @@
     # git log output affecting out ability to have working sem ver headers.
     changelog = git._run_git_command(['log', '--pretty=%B', version_spec],
                                      git_dir)
-    header_len = len('sem-ver:')
-    commands = [line[header_len:].strip() for line in changelog.split('\n')
-                if line.lower().strip().startswith('sem-ver:')]
     symbols = set()
-    for command in commands:
-        symbols.update([symbol.strip() for symbol in command.split(',')])
+    header = 'sem-ver:'
+    for line in changelog.split("\n"):
+        line = line.lower().strip()
+        if not line.lower().strip().startswith(header):
+            continue
+        new_symbols = line[len(header):].strip().split(",")
+        symbols.update([symbol.strip() for symbol in new_symbols])
 
     def _handle_symbol(symbol, symbols, impact):
         if symbol in symbols:
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/pbr-5.10.0/pbr/tests/test_packaging.py 
new/pbr-5.11.0/pbr/tests/test_packaging.py
--- old/pbr-5.10.0/pbr/tests/test_packaging.py  2022-08-11 16:01:20.000000000 
+0200
+++ new/pbr-5.11.0/pbr/tests/test_packaging.py  2022-10-20 13:02:15.000000000 
+0200
@@ -672,6 +672,27 @@
         version = packaging._get_version_from_git()
         self.assertThat(version, matchers.StartsWith('2.0.0.dev1'))
 
+    def test_multi_inline_symbols_no_space(self):
+        self.repo.commit()
+        self.repo.tag('1.2.3')
+        self.repo.commit('Sem-ver: feature,api-break')
+        version = packaging._get_version_from_git()
+        self.assertThat(version, matchers.StartsWith('2.0.0.dev1'))
+
+    def test_multi_inline_symbols_spaced(self):
+        self.repo.commit()
+        self.repo.tag('1.2.3')
+        self.repo.commit('Sem-ver: feature, api-break')
+        version = packaging._get_version_from_git()
+        self.assertThat(version, matchers.StartsWith('2.0.0.dev1'))
+
+    def test_multi_inline_symbols_reversed(self):
+        self.repo.commit()
+        self.repo.tag('1.2.3')
+        self.repo.commit('Sem-ver: api-break,feature')
+        version = packaging._get_version_from_git()
+        self.assertThat(version, matchers.StartsWith('2.0.0.dev1'))
+
     def test_leading_space(self):
         self.repo.commit()
         self.repo.tag('1.2.3')
@@ -679,6 +700,18 @@
         version = packaging._get_version_from_git()
         self.assertThat(version, matchers.StartsWith('2.0.0.dev1'))
 
+    def test_leading_space_multiline(self):
+        self.repo.commit()
+        self.repo.tag('1.2.3')
+        self.repo.commit(
+            (
+                '   Some cool text\n'
+                '   sem-ver: api-break'
+            )
+        )
+        version = packaging._get_version_from_git()
+        self.assertThat(version, matchers.StartsWith('2.0.0.dev1'))
+
     def test_leading_characters_symbol_not_found(self):
         self.repo.commit()
         self.repo.tag('1.2.3')
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/pbr-5.10.0/pbr/tests/test_util.py 
new/pbr-5.11.0/pbr/tests/test_util.py
--- old/pbr-5.10.0/pbr/tests/test_util.py       2022-08-11 16:01:20.000000000 
+0200
+++ new/pbr-5.11.0/pbr/tests/test_util.py       2022-10-20 13:02:15.000000000 
+0200
@@ -27,12 +27,13 @@
 
 def config_from_ini(ini):
     config = {}
+    ini = textwrap.dedent(six.u(ini))
     if sys.version_info >= (3, 2):
         parser = configparser.ConfigParser()
+        parser.read_file(io.StringIO(ini))
     else:
         parser = configparser.SafeConfigParser()
-    ini = textwrap.dedent(six.u(ini))
-    parser.readfp(io.StringIO(ini))
+        parser.readfp(io.StringIO(ini))
     for section in parser.sections():
         config[section] = dict(parser.items(section))
     return config
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/pbr-5.10.0/pbr/version.py 
new/pbr-5.11.0/pbr/version.py
--- old/pbr-5.10.0/pbr/version.py       2022-08-11 16:01:20.000000000 +0200
+++ new/pbr-5.11.0/pbr/version.py       2022-10-20 13:02:15.000000000 +0200
@@ -22,11 +22,16 @@
 import operator
 import sys
 
-try:
-    import importlib_metadata
+# TODO(stephenfin): Remove this once we drop support for Python < 3.8
+if sys.version_info >= (3, 8):
+    from importlib import metadata as importlib_metadata
     use_importlib = True
-except ImportError:
-    use_importlib = False
+else:
+    try:
+        import importlib_metadata
+        use_importlib = True
+    except ImportError:
+        use_importlib = False
 
 
 def _is_int(string):
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/pbr-5.10.0/pbr.egg-info/PKG-INFO 
new/pbr-5.11.0/pbr.egg-info/PKG-INFO
--- old/pbr-5.10.0/pbr.egg-info/PKG-INFO        2022-08-11 16:02:39.000000000 
+0200
+++ new/pbr-5.11.0/pbr.egg-info/PKG-INFO        2022-10-20 13:02:42.000000000 
+0200
@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: pbr
-Version: 5.10.0
+Version: 5.11.0
 Summary: Python Build Reasonableness
 Home-page: https://docs.openstack.org/pbr/latest/
 Author: OpenStack
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/pbr-5.10.0/pbr.egg-info/SOURCES.txt 
new/pbr-5.11.0/pbr.egg-info/SOURCES.txt
--- old/pbr-5.10.0/pbr.egg-info/SOURCES.txt     2022-08-11 16:02:39.000000000 
+0200
+++ new/pbr-5.11.0/pbr.egg-info/SOURCES.txt     2022-10-20 13:02:42.000000000 
+0200
@@ -103,6 +103,7 @@
 releasenotes/notes/fix-keywords-as-cfg-list-6cadc5141429d7f5.yaml
 
releasenotes/notes/fix-mapping-value-explode-with-equal-sign-41bf822fa4dd0e68.yaml
 releasenotes/notes/fix-pep517-metadata-regression-bc287e60e45b2732.yaml
+releasenotes/notes/fix-symbols-leading-spaces-f68928d75a8f0997.yaml
 releasenotes/notes/ignore-find-links-07cf54f465aa33a6.yaml
 releasenotes/notes/long-descr-content-type-f9a1003acbb8740f.yaml
 releasenotes/notes/pep517-support-89189ce0bab15845.yaml
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/pbr-5.10.0/releasenotes/notes/fix-symbols-leading-spaces-f68928d75a8f0997.yaml
 
new/pbr-5.11.0/releasenotes/notes/fix-symbols-leading-spaces-f68928d75a8f0997.yaml
--- 
old/pbr-5.10.0/releasenotes/notes/fix-symbols-leading-spaces-f68928d75a8f0997.yaml
  1970-01-01 01:00:00.000000000 +0100
+++ 
new/pbr-5.11.0/releasenotes/notes/fix-symbols-leading-spaces-f68928d75a8f0997.yaml
  2022-10-20 13:02:15.000000000 +0200
@@ -0,0 +1,5 @@
+---
+fixes:
+  - |
+    Fix an issue where symbols that were indented
+    would produce an incorrect version.
\ No newline at end of file

Reply via email to