Skip to content
Snippets Groups Projects
Commit 14d157c9 authored by Hans Fangohr's avatar Hans Fangohr Committed by Ashwin Kumar Karnad
Browse files

Apply suggestions from Hans

parent 091e3853
No related branches found
No related tags found
1 merge request!82Remove cmd
Pipeline #385497 passed
This commit is part of merge request !82. Comments created here will be created in the context of that merge request.
...@@ -1082,7 +1082,7 @@ def remove_environment(mpsd_release, root_dir, package_sets="NONE", force_remove ...@@ -1082,7 +1082,7 @@ def remove_environment(mpsd_release, root_dir, package_sets="NONE", force_remove
"""Remove release from installation. """Remove release from installation.
Handle 3 situations : Handle 3 situations :
1. remove dosent specify what to remove 1. remove does not specify what to remove
-> warn and exit -> warn and exit
2. remove all package_sets from release 2. remove all package_sets from release
-> remove release folder except logs -> remove release folder except logs
...@@ -1097,7 +1097,7 @@ def remove_environment(mpsd_release, root_dir, package_sets="NONE", force_remove ...@@ -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 A Path object representing the path to the directory where
the release and package_sets will be installed. the release and package_sets will be installed.
package_sets : list of str 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"). (e.g., "foss2021a-mpi", "global_generic", "ALL").
force_remove : bool, optional force_remove : bool, optional
A boolean indicating whether to force remove the release. 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): ...@@ -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 uninstall all packages exclusive to the environment,
then deactivate the environment, then deactivate the environment,
and finally remove the environment. and finally remove the environment.
Parameters Parameters
---------- ----------
spack_dir : pathlib.Path spack_dir : pathlib.Path
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment