18799: Make discovery2pydoc status check Py3.7-compatible
authorBrett Smith <brett.smith@curii.com>
Thu, 13 Apr 2023 15:42:09 +0000 (11:42 -0400)
committerBrett Smith <brett.smith@curii.com>
Thu, 13 Apr 2023 15:42:09 +0000 (11:42 -0400)
The `status` attribute was added in Python 3.9.

Arvados-DCO-1.1-Signed-off-by: Brett Smith <brett.smith@curii.com>

sdk/python/discovery2pydoc.py

index 889ee93293a5edd75b3858e9747a1cb18cd00e3a..57ae3161330a42ef7629917c450801a4cff94c42 100755 (executable)
@@ -317,7 +317,8 @@ If not provided, retrieved dynamically from Arvados client configuration.
 def main(arglist: Optional[Sequence[str]]=None) -> int:
     args = parse_arguments(arglist)
     with urllib.request.urlopen(args.discovery_url) as discovery_file:
-        if not (discovery_file.status is None or 200 <= discovery_file.status < 300):
+        status = discovery_file.getcode()
+        if not (status is None or 200 <= status < 300):
             print(
                 f"error getting {args.discovery_url}: server returned {discovery_file.status}",
                 file=sys.stderr,