From ac1314308254519850ae2efbe4dd868e362c0e43b2c266908ce5ed9ae8a89823 Mon Sep 17 00:00:00 2001 From: Matej Cepl Date: Sun, 2 May 2021 17:10:11 +0000 Subject: [PATCH] - Make sure to close the import_failed.map file after the exception has been raised in order to avoid ResourceWarnings when the failing import is part of a try...except block. OBS-URL: https://build.opensuse.org/package/show/devel:languages:python:Factory/python39?expand=0&rev=65 --- import_failed.py | 11 ++++++----- python39.changes | 7 +++++++ 2 files changed, 13 insertions(+), 5 deletions(-) diff --git a/import_failed.py b/import_failed.py index 7e83460..258b5a5 100644 --- a/import_failed.py +++ b/import_failed.py @@ -8,11 +8,12 @@ if __spec__: else: failed_name = __name__ -for line in open(failed_map_path): - package = line.split(':')[0] - imports = line.split(':')[1] - if failed_name in imports: - raise ImportError(f"""Module '{failed_name}' is not installed. +with open(failed_map_path) as fd: + for line in fd: + package = line.split(':')[0] + imports = line.split(':')[1] + if failed_name in imports: + raise ImportError(f"""Module '{failed_name}' is not installed. Use: sudo zypper install {package} to install it.""") diff --git a/python39.changes b/python39.changes index 9946f93..7c2fce9 100644 --- a/python39.changes +++ b/python39.changes @@ -1,3 +1,10 @@ +------------------------------------------------------------------- +Sun May 2 09:20:06 UTC 2021 - Ben Greiner + +- Make sure to close the import_failed.map file after the exception + has been raised in order to avoid ResourceWarnings when the + failing import is part of a try...except block. + ------------------------------------------------------------------- Wed Apr 28 16:39:54 UTC 2021 - Matej Cepl