From 040aeaf3a4fbc780148d725aa666954ad1ab20e7 Mon Sep 17 00:00:00 2001 From: Ola x Nilsson Date: Tue, 26 Aug 2025 15:41:34 +0200 Subject: [PATCH] sstate: Open file with context manager In sstat_install and sstate_clean_cache. Signed-off-by: Ola x Nilsson Signed-off-by: Mathieu Dubois-Briand Signed-off-by: Richard Purdie --- meta/classes-global/sstate.bbclass | 28 +++++++++++++--------------- 1 file changed, 13 insertions(+), 15 deletions(-) diff --git a/meta/classes-global/sstate.bbclass b/meta/classes-global/sstate.bbclass index e3d6373b3f4..2fd29d73232 100644 --- a/meta/classes-global/sstate.bbclass +++ b/meta/classes-global/sstate.bbclass @@ -306,18 +306,17 @@ def sstate_install(ss, d): sharedfiles.append(ss['fixmedir'] + "/fixmepath") # Write out the manifest - f = open(manifest, "w") - for file in sharedfiles: - f.write(file + "\n") - - # We want to ensure that directories appear at the end of the manifest - # so that when we test to see if they should be deleted any contents - # added by the task will have been removed first. - dirs = sorted(shareddirs, key=len) - # Must remove children first, which will have a longer path than the parent - for di in reversed(dirs): - f.write(di + "\n") - f.close() + with open(manifest, "w") as f: + for file in sharedfiles: + f.write(file + "\n") + + # We want to ensure that directories appear at the end of the manifest + # so that when we test to see if they should be deleted any contents + # added by the task will have been removed first. + dirs = sorted(shareddirs, key=len) + # Must remove children first, which will have a longer path than the parent + for di in reversed(dirs): + f.write(di + "\n") # Append to the list of manifests for this PACKAGE_ARCH @@ -481,9 +480,8 @@ def sstate_clean_cachefiles(d): def sstate_clean_manifest(manifest, d, canrace=False, prefix=None): import oe.path - mfile = open(manifest) - entries = mfile.readlines() - mfile.close() + with open(manifest) as mfile: + entries = mfile.readlines() for entry in entries: entry = entry.strip() -- 2.47.3