From 14d157c9bd3687caef8b2648a17f8e968ab7f2f7 Mon Sep 17 00:00:00 2001 From: Hans Fangohr <hans.fangohr@mpsd.mpg.de> Date: Fri, 18 Aug 2023 15:16:31 +0000 Subject: [PATCH] Apply suggestions from Hans --- src/mpsd_software_manager/mpsd_software.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/mpsd_software_manager/mpsd_software.py b/src/mpsd_software_manager/mpsd_software.py index d45e74e..2241b60 100755 --- a/src/mpsd_software_manager/mpsd_software.py +++ b/src/mpsd_software_manager/mpsd_software.py @@ -1082,7 +1082,7 @@ def remove_environment(mpsd_release, root_dir, package_sets="NONE", force_remove """Remove release from installation. Handle 3 situations : - 1. remove dosent specify what to remove + 1. remove does not specify what to remove -> warn and exit 2. remove all package_sets from release -> remove release folder except logs @@ -1097,7 +1097,7 @@ def remove_environment(mpsd_release, root_dir, package_sets="NONE", force_remove A Path object representing the path to the directory where the release and package_sets will be installed. package_sets : list of str - A list of strings representing the package_sets to install + A list of strings representing the package_sets to remove (e.g., "foss2021a-mpi", "global_generic", "ALL"). force_remove : bool, optional A boolean indicating whether to force remove the release. @@ -1190,6 +1190,7 @@ def remove_spack_environment(spack_dir, environment_name, build_log_path=None): then uninstall all packages exclusive to the environment, then deactivate the environment, and finally remove the environment. + Parameters ---------- spack_dir : pathlib.Path -- GitLab