diff --git a/src/fromager/build_environment.py b/src/fromager/build_environment.py index 6698aa69..f4719063 100644 --- a/src/fromager/build_environment.py +++ b/src/fromager/build_environment.py @@ -237,7 +237,10 @@ def get_distributions(self) -> typing.Mapping[str, Version]: "json.dump({d.name: d.version for d in distributions()}, sys.stdout)", ] result = self.run([str(self.python), "-c", "; ".join(lines)]) - mapping = json.loads(result.strip()) + try: + mapping = json.loads(result.strip()) + except Exception: + logger.exception("failed to de-serialize JSON: %s", result) return {name: Version(version) for name, version in sorted(mapping.items())} @@ -296,9 +299,13 @@ def prepare_build_environment( dep_req_type=RequirementType.BUILD_SDIST, ) - logger.debug( - "build env %r has packages %r", build_env.path, build_env.get_distributions() - ) + try: + distributions = build_env.get_distributions() + except Exception: + # ignore error for debug call, error reason is logged in get_distributions() + pass + else: + logger.debug("build env %r has packages %r", build_env.path, distributions) return build_env diff --git a/src/fromager/run_network_isolation.sh b/src/fromager/run_network_isolation.sh index dd6011da..6d3543a2 100755 --- a/src/fromager/run_network_isolation.sh +++ b/src/fromager/run_network_isolation.sh @@ -32,8 +32,6 @@ fi # set hostname to "localhost" if command -v hostname 2>&1 >/dev/null; then hostname localhost -else - echo "'hostname' command is missing, cannot set host name to 'localhost' for $@" >&2 fi # replace with command