Skip to content

Commit

Permalink
Merge pull request #4094 from jfgrimm/system-template-constant-depend…
Browse files Browse the repository at this point in the history
…ency

use SYSTEM template constant in dependencies instead of True in framework tests
  • Loading branch information
boegel authored Oct 13, 2022
2 parents 54655d3 + 48db126 commit 6cab5e2
Show file tree
Hide file tree
Showing 6 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion test/framework/easyblock.py
Original file line number Diff line number Diff line change
Expand Up @@ -871,7 +871,7 @@ def test_make_module_dep_of_dep_hmns(self):
'description = "test easyconfig"',
"toolchain = {'name': 'foss', 'version': '2018a'}",
'dependencies = [',
" ('GCC', '6.4.0-2.28', '', True),"
" ('GCC', '6.4.0-2.28', '', SYSTEM),"
" ('hwloc', '1.11.8', '', ('GCC', '6.4.0-2.28')),",
" ('OpenMPI', '2.1.2', '', ('GCC', '6.4.0-2.28')),"
']',
Expand Down
2 changes: 1 addition & 1 deletion test/framework/easyconfig.py
Original file line number Diff line number Diff line change
Expand Up @@ -1208,7 +1208,7 @@ def test_java_wrapper_templating(self):
'homepage = "https://example.com"',
'description = "test easyconfig"',
'toolchain = {"name":"GCC", "version": "4.6.3"}',
'dependencies = [("Java", "11", "", True)]',
'dependencies = [("Java", "11", "", SYSTEM)]',
'modloadmsg = "Java: %(javaver)s, %(javamajver)s, %(javashortver)s"',
])
self.prep()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ source_urls = [GNU_SOURCE]
# # zlib required, but being linked in statically, so not a runtime dep
# ('zlib', '1.2.8'),
# # use same binutils version that was used when building GCC toolchain, to 'bootstrap' this binutils
# ('binutils', version, '', True)
# ('binutils', version, '', SYSTEM)
#]

moduleclass = 'tools'
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ source_urls = [GNU_SOURCE]
# # zlib required, but being linked in statically, so not a runtime dep
# ('zlib', '1.2.8'),
# # use same binutils version that was used when building GCC toolchain, to 'bootstrap' this binutils
# ('binutils', version, '', True)
# ('binutils', version, '', SYSTEM)
#]

moduleclass = 'tools'
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ sources = ['%(name)s-%(version)s.tar.gz']
# download location for source files
source_urls = [GNU_SOURCE]

hiddendependencies = [('toy', '0.0', '-deps', True)]
hiddendependencies = [('toy', '0.0', '-deps', SYSTEM)]
dependencies = hiddendependencies # hidden deps must be included in list of deps

# make sure the gzip and gunzip binaries are available after installation
Expand Down
2 changes: 1 addition & 1 deletion test/framework/easyconfigs/test_ecs/t/toy/toy-0.0-deps.eb
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ checksums = [[
patches = ['toy-0.0_fix-silly-typo-in-printf-statement.patch']

dependencies = [
('intel', '2018a', '', True),
('intel', '2018a', '', SYSTEM),
('GCC/6.4.0-2.28', EXTERNAL_MODULE),
]

Expand Down

0 comments on commit 6cab5e2

Please sign in to comment.