patch-manager 0.0.10__tar.gz → 0.0.11__tar.gz

{patch_manager-0.0.10 → patch_manager-0.0.11}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: patch-manager
-Version: 0.0.10
+Version: 0.0.11
 Summary: Create, check and email patch series, with Patchwork integration
 Author-email: Simon Glass <sjg@chromium.org>
 License-Expression: GPL-2.0-or-later
@@ -35,6 +35,10 @@ Requires-Dist: claude-agent-sdk; extra == "all"
 Requires-Dist: concurrencytest; extra == "all"
 Dynamic: license-file
 
+.. image:: https://raw.githubusercontent.com/nxtboot/patman/master/logo/patman-logo-primary.png
+   :alt: Patman
+   :width: 400px
+
 Patman patch manager
 ====================
 

{patch_manager-0.0.10 → patch_manager-0.0.11}/README.rst

@@ -1,3 +1,7 @@
+.. image:: https://raw.githubusercontent.com/nxtboot/patman/master/logo/patman-logo-primary.png
+   :alt: Patman
+   :width: 400px
+
 Patman patch manager
 ====================
 

{patch_manager-0.0.10 → patch_manager-0.0.11}/patch_manager.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: patch-manager
-Version: 0.0.10
+Version: 0.0.11
 Summary: Create, check and email patch series, with Patchwork integration
 Author-email: Simon Glass <sjg@chromium.org>
 License-Expression: GPL-2.0-or-later
@@ -35,6 +35,10 @@ Requires-Dist: claude-agent-sdk; extra == "all"
 Requires-Dist: concurrencytest; extra == "all"
 Dynamic: license-file
 
+.. image:: https://raw.githubusercontent.com/nxtboot/patman/master/logo/patman-logo-primary.png
+   :alt: Patman
+   :width: 400px
+
 Patman patch manager
 ====================
 

{patch_manager-0.0.10 → patch_manager-0.0.11}/patman/cseries.py

@@ -341,9 +341,15 @@ class Cseries(cser_helper.CseriesHelper):
         _, ser, version, _, _, _, _, _ = self._get_patches(series, version)
         svinfo = self.get_ser_ver(ser.idnum, version)
 
-        # Use the per-version description if available, since the
-        # series-level desc may be stale (e.g. patch order changed)
-        desc = svinfo.desc or ser.desc
+        # Prefer the live cover-letter title from the branch: it is what
+        # patchwork indexes the series by, and it reflects a rename made
+        # after the version was created (the stored description would be
+        # stale in that case). Fall back to the stored per-version
+        # description for series with no cover letter, where it is more
+        # stable than the first commit subject (e.g. if patches are
+        # reordered).
+        desc = ser.cover[0] if ser.cover else None
+        desc = desc or svinfo.desc or ser.desc
         if not desc:
             raise ValueError(f"Series '{ser.name}' has an empty description")
         ser.desc = desc

{patch_manager-0.0.10 → patch_manager-0.0.11}/patman/test_cseries.py

@@ -1117,6 +1117,36 @@ Tested-by: Mary Smith <msmith@wibble.com>   # yak
                 (457, None, 'second', 2, 'Series for my board'),
                 cser.link_search(pwork, 'second', 2))
 
+    def test_series_link_auto_renamed(self):
+        """autolink uses the live cover title, not a stale stored desc
+
+        When a later version renames the series (changes the cover-letter
+        title) after the version was created, the per-version description
+        stored at creation time goes stale. link_search must use the
+        current cover title from the branch so it searches patchwork for
+        the right name.
+        """
+        cser = self.get_cser()
+        with terminal.capture():
+            cser.add('second', allow_unmarked=True)
+            cser.link_set('second', None, f'{self.SERIES_ID_SECOND_V1}', True)
+            cser.increment('second')
+
+        pwork = Patchwork.for_testing(self._fake_patchwork_cser)
+        pwork.project_set(self.PROJ_ID, self.PROJ_LINK_NAME)
+        cser.project_set(pwork, 'U-Boot', quiet=True)
+
+        # Simulate the rename: the stored v2 description no longer matches
+        # the cover-letter title on the branch
+        ser = cser.get_series_by_name('second')
+        svid = cser.get_series_svid(ser.idnum, 2)
+        cser.db.ser_ver_set_desc(svid, 'Stale old title')
+        cser.commit()
+
+        with terminal.capture():
+            _, _, _, _, desc = cser.link_search(pwork, 'second', 2)
+        self.assertEqual(self.TITLE_SECOND, desc)
+
     def test_series_link_auto_name(self):
         """Test finding the patchwork link for a cseries with auto name"""
         cser = self.get_cser()

{patch_manager-0.0.10 → patch_manager-0.0.11}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "patch-manager"
-version = "0.0.10"
+version = "0.0.11"
 authors = [
   { name="Simon Glass", email="sjg@chromium.org" },
 ]