From 2a1693cee90811ee967a2475d5c136197aa3263a Mon Sep 17 00:00:00 2001 From: Alexander Grund Date: Wed, 23 Apr 2025 08:57:13 +0200 Subject: [PATCH] Remove superflous assignment in _sanity_check_step `sanity_check_load_module` returns `self.fake_mod_data` so there is no need to assign it to itself. Remove it to avoid confusion when e.g. implementing easyblocks using this functions --- easybuild/framework/easyblock.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/easybuild/framework/easyblock.py b/easybuild/framework/easyblock.py index 38fb3b0e7b..3b3676e341 100644 --- a/easybuild/framework/easyblock.py +++ b/easybuild/framework/easyblock.py @@ -3874,7 +3874,7 @@ def xs2str(xs): trace_msg("%s %s found: %s" % (typ, xs2str(xs), ('FAILED', 'OK')[found])) if not self.sanity_check_module_loaded: - self.fake_mod_data = self.sanity_check_load_module(extension=extension, extra_modules=extra_modules) + self.sanity_check_load_module(extension=extension, extra_modules=extra_modules) # allow oversubscription of P processes on C cores (P>C) for software installed on top of Open MPI; # this is useful to avoid failing of sanity check commands that involve MPI