)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"715ae65a83ebca82ae919476b51e36f1f5d5dbb8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0f5d026b_ed99a63a","updated":"2025-05-07 19:53:51.000000000","message":"afaict none of the CI failures are related","commit_id":"04bb21f7f3614c302ef93b16775811d01d276593"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"1346a960727776410515bc20c1f24f952c0d5eff","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"8d144335_e65975e8","updated":"2025-05-08 15:01:45.000000000","message":"I\u0027m testing this with [an osc-lib patch](https://review.opendev.org/c/openstack/osc-lib/+/949059) and the following additional change:\n\n```\ndiff --git pyproject.toml pyproject.toml\nindex 79bab92..06efa27 100644\n--- pyproject.toml\n+++ pyproject.toml\n@@ -1,5 +1,6 @@\n [build-system]\n-requires \u003d [\"pbr\u003e\u003d6.1.1\"]\n+requires \u003d [\"/home/stephenfin/openstack/pbr\"]\n build-backend \u003d \"pbr.build\"\n \n [project]\ndiff --git setup.cfg setup.cfg\ndeleted file mode 100644\nindex 4581430..0000000\n--- setup.cfg\n+++ /dev/null\n@@ -1,2 +0,0 @@\n-[metadata]\n-name \u003d osc-lib\n```\n\nIt currently fails on the `build_wheel` step with the following traceback:\n\n```\n* Building wheel...                                                                                                                                                                           \nError parsing                                                                                                                                                                                 \nTraceback (most recent call last):                                                                                                                                                            \n  File \"/tmp/build-env-yjnkpikd/lib64/python3.13/site-packages/pbr/core.py\", line 104, in pbr                                                                                                 \n    attrs \u003d util.cfg_to_args(path, dist.script_args)                                                                                                                                          \n  File \"/tmp/build-env-yjnkpikd/lib64/python3.13/site-packages/pbr/util.py\", line 272, in cfg_to_args                                                                                         \n    pbr.hooks.setup_hook(config)                                                                                                                                                              \n    ~~~~~~~~~~~~~~~~~~~~^^^^^^^^                                                                                                                                                              \n  File \"/tmp/build-env-yjnkpikd/lib64/python3.13/site-packages/pbr/hooks/__init__.py\", line 25, in setup_hook                                                                                 \n    metadata_config.run()                                                                                                                                                                     \n    ~~~~~~~~~~~~~~~~~~~^^                                                                                                                                                                     \n  File \"/tmp/build-env-yjnkpikd/lib64/python3.13/site-packages/pbr/hooks/base.py\", line 27, in run                                                                                            \n    self.hook()                                                                                                                                                                               \n    ~~~~~~~~~^^                                                                                                                                                                               \n  File \"/tmp/build-env-yjnkpikd/lib64/python3.13/site-packages/pbr/hooks/metadata.py\", line 26, in hook                                                                                       \n    self.config[\u0027name\u0027], self.config.get(\u0027version\u0027, None))                                                                                                                                    \n    ~~~~~~~~~~~^^^^^^^^                                                                                                                                                                       \nKeyError: \u0027name\u0027                                                                                                                                                                              \nerror in setup command: Error parsing /tmp/build-via-sdist-1bdqgb_j/osc_lib-0.0.0/setup.cfg: KeyError: \u0027name\u0027                                                                                 \n                                                                                                                                                                                              \nERROR Backend subprocess exited when trying to invoke build_whee\n```\n\nLooks like there\u0027s more to do here.","commit_id":"d943487d54dbe4b3423312c5321de7d791a843ce"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"e1d5c9b67ad798152e6b474b2b09b287733726ca","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"5ff7566c_a31c4a7f","updated":"2025-05-08 14:48:53.000000000","message":"thx","commit_id":"d943487d54dbe4b3423312c5321de7d791a843ce"}],"pbr/core.py":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"715ae65a83ebca82ae919476b51e36f1f5d5dbb8","unresolved":true,"context_lines":[{"line_number":70,"context_line":"    If given as a string, the value of pbr is assumed to be the relative path"},{"line_number":71,"context_line":"    to the setup.cfg file to use.  Otherwise, if it evaluates to true, it"},{"line_number":72,"context_line":"    simply assumes that pbr should be used, and the default \u0027setup.cfg\u0027 is"},{"line_number":73,"context_line":"    used."},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    This works by reading the setup.cfg file, parsing out the supported"},{"line_number":76,"context_line":"    metadata and command options, and using them to rebuild the"}],"source_content_type":"text/x-python","patch_set":1,"id":"a8090a8b_693a95e5","line":73,"updated":"2025-05-07 19:53:51.000000000","message":"Maybe also amend this accordingly?","commit_id":"04bb21f7f3614c302ef93b16775811d01d276593"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"14d0503c9f40c2cffeee9892bb76ca885e1e4d52","unresolved":false,"context_lines":[{"line_number":70,"context_line":"    If given as a string, the value of pbr is assumed to be the relative path"},{"line_number":71,"context_line":"    to the setup.cfg file to use.  Otherwise, if it evaluates to true, it"},{"line_number":72,"context_line":"    simply assumes that pbr should be used, and the default \u0027setup.cfg\u0027 is"},{"line_number":73,"context_line":"    used."},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    This works by reading the setup.cfg file, parsing out the supported"},{"line_number":76,"context_line":"    metadata and command options, and using them to rebuild the"}],"source_content_type":"text/x-python","patch_set":1,"id":"10b10726_53faa1f1","line":73,"in_reply_to":"a8090a8b_693a95e5","updated":"2025-05-08 13:24:04.000000000","message":"Done","commit_id":"04bb21f7f3614c302ef93b16775811d01d276593"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"715ae65a83ebca82ae919476b51e36f1f5d5dbb8","unresolved":true,"context_lines":[{"line_number":124,"context_line":"        # Repeat some of the Distribution initialization code with the newly"},{"line_number":125,"context_line":"        # provided attrs"},{"line_number":126,"context_line":"        if attrs:"},{"line_number":127,"context_line":"            # Skips \u0027options\u0027 and \u0027licence\u0027 support which are rarely used; may"},{"line_number":128,"context_line":"            # add back in later if demanded"},{"line_number":129,"context_line":"            for key, val in attrs.items():"},{"line_number":130,"context_line":"                if hasattr(dist.metadata, \u0027set_\u0027 + key):"}],"source_content_type":"text/x-python","patch_set":1,"id":"b1b24616_2dffc90c","line":127,"updated":"2025-05-07 19:53:51.000000000","message":"I know this comment is just copied, but it also looks no longer to be correct, or am I missing something?","commit_id":"04bb21f7f3614c302ef93b16775811d01d276593"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"14d0503c9f40c2cffeee9892bb76ca885e1e4d52","unresolved":false,"context_lines":[{"line_number":124,"context_line":"        # Repeat some of the Distribution initialization code with the newly"},{"line_number":125,"context_line":"        # provided attrs"},{"line_number":126,"context_line":"        if attrs:"},{"line_number":127,"context_line":"            # Skips \u0027options\u0027 and \u0027licence\u0027 support which are rarely used; may"},{"line_number":128,"context_line":"            # add back in later if demanded"},{"line_number":129,"context_line":"            for key, val in attrs.items():"},{"line_number":130,"context_line":"                if hasattr(dist.metadata, \u0027set_\u0027 + key):"}],"source_content_type":"text/x-python","patch_set":1,"id":"fd5231fa_7b6e0560","line":127,"in_reply_to":"b1b24616_2dffc90c","updated":"2025-05-08 13:24:04.000000000","message":"Done","commit_id":"04bb21f7f3614c302ef93b16775811d01d276593"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"715ae65a83ebca82ae919476b51e36f1f5d5dbb8","unresolved":true,"context_lines":[{"line_number":141,"context_line":"    else:"},{"line_number":142,"context_line":"        if not os.path.exists(os.path.abspath(\u0027pyproject.toml\u0027)):"},{"line_number":143,"context_line":"            raise errors.DistutilsFileError("},{"line_number":144,"context_line":"                \u0027The setup.cfg file %s does not exist.\u0027 % path"},{"line_number":145,"context_line":"            )"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":"    # Re-finalize the underlying Distribution"}],"source_content_type":"text/x-python","patch_set":1,"id":"a993ed9f_2ce748c9","line":144,"updated":"2025-05-07 19:53:51.000000000","message":"Not sure if the text of this error must stay untouch or could also mention pyproject.toml? like\n\n```suggestion\n                \u0027Neither the setup.cfg file %s nor pyproject.toml do exist.\u0027 % path\n```","commit_id":"04bb21f7f3614c302ef93b16775811d01d276593"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"14d0503c9f40c2cffeee9892bb76ca885e1e4d52","unresolved":false,"context_lines":[{"line_number":141,"context_line":"    else:"},{"line_number":142,"context_line":"        if not os.path.exists(os.path.abspath(\u0027pyproject.toml\u0027)):"},{"line_number":143,"context_line":"            raise errors.DistutilsFileError("},{"line_number":144,"context_line":"                \u0027The setup.cfg file %s does not exist.\u0027 % path"},{"line_number":145,"context_line":"            )"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":"    # Re-finalize the underlying Distribution"}],"source_content_type":"text/x-python","patch_set":1,"id":"d546358a_577914bd","line":144,"in_reply_to":"a993ed9f_2ce748c9","updated":"2025-05-08 13:24:04.000000000","message":"Done","commit_id":"04bb21f7f3614c302ef93b16775811d01d276593"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"bb7ad9ba812ca0d0a3b79c2264a92aeaa53cfc68","unresolved":true,"context_lines":[{"line_number":101,"context_line":"    if os.path.exists(path):"},{"line_number":102,"context_line":"        # If we have a setup.cfg file, convert it to setup() arguments"},{"line_number":103,"context_line":"        try:"},{"line_number":104,"context_line":"            attrs \u003d util.cfg_to_args(path, dist.script_args)"},{"line_number":105,"context_line":"        except Exception:"},{"line_number":106,"context_line":"            e \u003d sys.exc_info()[1]"},{"line_number":107,"context_line":"            # NB: This will output to the console if no explicit logging has"}],"source_content_type":"text/x-python","patch_set":2,"id":"340a4456_7341b7c7","line":104,"updated":"2025-05-12 11:12:27.000000000","message":"By placing all of this under a conditional, this method never gets called. It has side-effects on the `dist` object, so that\u0027s an issue. I\u0027ll have to rework this.","commit_id":"d943487d54dbe4b3423312c5321de7d791a843ce"}]}
