Skip to content

Commit b90293b

Browse files
authored
Revert "Introducing version_naming property that makes modulefile as version …"
This reverts commit 7af1150.
1 parent 0848c87 commit b90293b

File tree

6 files changed

+3
-117
lines changed

6 files changed

+3
-117
lines changed

CHANGELOG.md

-1
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,6 @@ and **Merged pull requests**. Critical items to know are:
1414
The versions coincide with releases on pip. Only major versions will be released as tags on Github.
1515

1616
## [0.0.x](https://github.com/singularityhub/singularity-hpc/tree/main) (0.0.x)
17-
- Adding in version_naming feature (0.1.31)
1817
- Fix `module-info shell` Tcl test for Lmod<=8.7.55 (0.1.30)
1918
- Allow import of LooseVersion from packaging (bug) (0.1.29)
2019
- use quay.io api to list tags since does not conform to oci (0.1.28)

shpc/main/modules/base.py

+2-10
Original file line numberDiff line numberDiff line change
@@ -458,10 +458,7 @@ def install(
458458
module.load_override_file()
459459

460460
# Create the module and container directory
461-
if self.settings.version_naming:
462-
utils.mkdirp([os.path.dirname(module.module_dir), module.container_dir])
463-
else:
464-
utils.mkdirp([module.module_dir, module.container_dir])
461+
utils.mkdirp([module.module_dir, module.container_dir])
465462

466463
# Add a .version file to indicate the level of versioning
467464
self.versionfile.write(
@@ -473,12 +470,7 @@ def install(
473470

474471
# Get the template based on the module and container type
475472
template = self.template.load(self.templatefile)
476-
if self.settings.version_naming:
477-
module_path = os.path.join(
478-
os.path.dirname(module.module_dir), name.split(":")[1]
479-
)
480-
else:
481-
module_path = os.path.join(module.module_dir, self.modulefile)
473+
module_path = os.path.join(module.module_dir, self.modulefile)
482474

483475
# Install the container
484476
# This could be simplified to take the module

shpc/main/schemas.py

-1
Original file line numberDiff line numberDiff line change
@@ -170,7 +170,6 @@
170170
"wrapper_shell": {"type": "string", "enum": shells},
171171
"module_sys": {"type": "string", "enum": ["lmod", "tcl", None]},
172172
"container_features": container_features,
173-
"version_naming": {"type": "boolean"},
174173
}
175174

176175
settings = {

shpc/settings.yml

-4
Original file line numberDiff line numberDiff line change
@@ -38,9 +38,6 @@ module_name: '{{ parsed_name.tool }}'
3838
# first_installed: use the first installed
3939
default_version: module_sys
4040

41-
#Experimental version naming tag so that modulefile is changed to version number . Default value is false. Issue number 26
42-
version_naming: false
43-
4441
# store containers separately from module files
4542
# It's recommended to do this for faster loading
4643
container_base: $root_dir/containers
@@ -71,7 +68,6 @@ singularity_shell: /bin/sh
7168
podman_shell: /bin/sh
7269
docker_shell: /bin/sh
7370

74-
7571
# shell for test.sh file
7672
test_shell: /bin/bash
7773

shpc/tests/test_version_naming.py

-100
This file was deleted.

shpc/version.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
__copyright__ = "Copyright 2021-2025, Vanessa Sochat"
33
__license__ = "MPL 2.0"
44

5-
__version__ = "0.1.31"
5+
__version__ = "0.1.30"
66
AUTHOR = "Vanessa Sochat"
77
88
NAME = "singularity-hpc"

0 commit comments

Comments
 (0)