remove arch memoizations for now (no effect but the complaint implies some issue with that

scheibelp · scheibelp · commit 37719a9e99e6 · 2020-04-16T18:10:54.000-07:00
diff --git a/lib/spack/spack/architecture.py b/lib/spack/spack/architecture.py
@@ -441,7 +441,7 @@ def from_dict(d):
         return arch_for_spec(spec)
 
 
-@memoized
+# @memoized
 def get_platform(platform_name):
     """Returns a platform object that corresponds to the given name."""
     platform_list = all_platforms()
@@ -477,7 +477,7 @@ def arch_for_spec(arch_spec):
     return Arch(arch_plat, arch_spec.os, arch_spec.target)
 
 
-@memoized
+# @memoized
 def all_platforms():
     classes = []
     mod_path = spack.paths.platform_path
@@ -498,7 +498,7 @@ def all_platforms():
     return classes
 
 
-@memoized
+# @memoized
 def platform():
     """Detects the platform for this machine.
 
@@ -516,7 +516,7 @@ def platform():
             return platform_cls()
 
 
-@memoized
+# @memoized
 def sys_type():
     """Print out the "default" platform-os-target tuple for this machine.
 
@@ -533,7 +533,7 @@ def sys_type():
     return str(arch)
 
 
-@memoized
+# @memoized
 def compatible_sys_types():
     """Returns a list of all the systypes compatible with the current host."""
     compatible_archs = []
diff --git a/lib/spack/spack/test/concretize.py b/lib/spack/spack/test/concretize.py
@@ -96,7 +96,7 @@ def current_host(request, monkeypatch):
     target = llnl.util.cpu.targets[cpu]
 
     # this function is memoized, so clear its state for testing
-    spack.architecture.get_platform.cache.clear()
+    # spack.architecture.get_platform.cache.clear()
 
     if not is_preference:
         monkeypatch.setattr(llnl.util.cpu, 'host', lambda: target)
@@ -109,7 +109,7 @@ def current_host(request, monkeypatch):
             yield target
 
     # clear any test values fetched
-    spack.architecture.get_platform.cache.clear()
+    # spack.architecture.get_platform.cache.clear()
 
 
 @pytest.mark.usefixtures('config', 'mock_packages')
