[gentoo-commits] proj/pkgcore/snakeoil:master commit in: src/snakeoil/dist/

["Arthur Zamarin" <arthurzam@gentoo.org>]

commit:     07519289999445518f9883eb2329af71d91b0aa5
Author:     Brian Harring <ferringb <AT> gmail <DOT> com>
AuthorDate: Fri Jan 26 04:59:26 2024 +0000
Commit:     Arthur Zamarin <arthurzam <AT> gentoo <DOT> org>
CommitDate: Fri Jan 26 19:17:27 2024 +0000
URL:        https://gitweb.gentoo.org/proj/pkgcore/snakeoil.git/commit/?id=07519289

fix(sphinx_ext): email is optional.

It's not obvious, but both name and email are optional for
authors.  NFC what one does with an author field lacking both,
but that's for the PEP authors to resolve.

We have authors in the pkgcore project that don't have email
listed, which now breaks html generation.  Thus this change.

The only hanky point here is that if it's just email, we output
that w/out the usual `<{email}>` brackets per standards.

Signed-off-by: Brian Harring <ferringb <AT> gmail.com>
Signed-off-by: Arthur Zamarin <arthurzam <AT> gentoo.org>

 src/snakeoil/dist/sphinxext.py | 16 +++++++++++++---
 1 file changed, 13 insertions(+), 3 deletions(-)

diff --git a/src/snakeoil/dist/sphinxext.py b/src/snakeoil/dist/sphinxext.py
index 7d04c49d..36d63e23 100644
--- a/src/snakeoil/dist/sphinxext.py
+++ b/src/snakeoil/dist/sphinxext.py
@@ -30,9 +30,19 @@ def prepare_scripts_man(repo_dir: Path, man_pages: list[tuple]):
     with open(repo_dir / 'pyproject.toml', 'rb') as file:
         pyproj = tomllib.load(file)
 
-    authors_list = [
-        f'{author["name"]} <{author["email"]}>' for author in pyproj['project']['authors']
-    ]
+    authors_list: list[str] = []
+    for author in pyproj['project']['authors']:
+        name, email = author.get('name'), author.get('email')
+        if name:
+            if email:
+                authors_list.append(f'{name} <{email}>')
+            else:
+                authors_list.append(name)
+        elif email:
+            authors_list.append(email)
+        else:
+            # no name or contact info, so ignore it.
+            continue
 
     for i, man_page in enumerate(man_pages):
         if man_page[3] is None: