pmbootstrap: pmb.config.sudo: Use type union compatible with Python 3.7 v3 APPLIED

Newbyte: 1
 pmb.config.sudo: Use type union compatible with Python 3.7

 1 files changed, 2 insertions(+), 1 deletions(-)

[PATCH pmbootstrap v3] pmb.config.sudo: Use type union compatible with Python 3.7 Export this patch

The Type | OtherType syntax for writing unions was introduced in Python
3.10. We want to support Python 3.7, so use an Optional type hint
instead which declares the same thing and is supported by 3.7.

Fixes d31313f7dc8018aeb92eeb8caf2dccf71a443a43

Luca Weiss <luca@z3ntu.xyz>

1 year, 3 months ago

Fixes: d31313f7 ("Don't use 'sudo' when running as root")

With that fixed:

Reviewed-by: Luca Weiss <luca@z3ntu.xyz>

---
 pmb/config/sudo.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/pmb/config/sudo.py b/pmb/config/sudo.py
index 1e227534..baa5963e 100644
--- a/pmb/config/sudo.py
+++ b/pmb/config/sudo.py
@@ -3,10 +3,11 @@
 import os
 import shutil
 from functools import lru_cache
+from typing import Optional
 
 
 @lru_cache()
-def which_sudo() -> str | None:
+def which_sudo() -> Optional[str]:
     """Returns a command required to run commands as root, if any.
 
     Find whether sudo or doas is installed for commands that require root.
-- 
2.40.1

builds.sr.ht <builds@sr.ht>

1 year, 3 months ago

pmbootstrap/patches/.build.yml: FAILED in 5m24s

[pmb.config.sudo: Use type union compatible with Python 3.7][0] v3 from [Newbyte][1]

[0]: https://lists.sr.ht/~postmarketos/pmbootstrap-devel/patches/41651
[1]: mailto:newbyte@postmarketos.org

✗ #1001625 FAILED pmbootstrap/patches/.build.yml https://builds.sr.ht/~postmarketos/job/1001625

Oliver Smith <ollieparanoid@postmarketos.org>

1 year, 3 months ago

Oliver Smith <ollieparanoid@postmarketos.org>

1 year, 3 months ago

Applied, thanks!

[1/1] pmb.config.sudo: Use type union compatible with Python 3.7
      commit: c6a8a2614d34148176a737887dbb56d66bdedcb6

Best regards