summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
Diffstat
-rw-r--r--lib/spack/spack/concretize.py7
-rw-r--r--lib/spack/spack/spec.py34
-rw-r--r--lib/spack/spack/test/multimethod.py10
-rw-r--r--lib/spack/spack/test/spec_dag.py7
-rw-r--r--lib/spack/spack/test/spec_semantics.py26
-rw-r--r--lib/spack/spack/test/spec_yaml.py8
-rw-r--r--lib/spack/spack/virtual.py1
-rw-r--r--var/spack/mock_packages/multimethod/package.py8
8 files changed, 52 insertions, 49 deletions
diff --git a/lib/spack/spack/concretize.py b/lib/spack/spack/concretize.py
index d2498ec64d..484da97e90 100644
--- a/lib/spack/spack/concretize.py
+++ b/lib/spack/spack/concretize.py
@@ -195,7 +195,7 @@ class DefaultConcretizer(object):
spec.compiler_flags[flag] = list(set(spec.compiler_flags[flag]) |
set(nearest.compiler_flags[flag]))
else:
- spec.compielr_flags[flag] = nearest.compiler_flags[flag]
+ spec.compiler_flags[flag] = nearest.compiler_flags[flag]
ret = True
except StopIteration:
@@ -216,9 +216,10 @@ class DefaultConcretizer(object):
# in default compiler flags.
compiler = spack.compilers.compiler_for_spec(spec.compiler)
for flag in compiler.flags:
- if flag not in spec.compiler_flags or spec.compiler_flags[flag] == []:
+ if flag not in spec.compiler_flags:
spec.compiler_flags[flag] = compiler.flags[flag]
- ret = True
+ if compiler.flags[flag] != []:
+ ret = True
else:
if (sorted(spec.compiler_flags[flag]) != sorted(compiler.flags[flag])) and (not set(spec.compiler_flags[flag]) >= set(compiler.flags[flag])):
ret = True
diff --git a/lib/spack/spack/spec.py b/lib/spack/spack/spec.py
index 4dfc315c90..e4bb50dd89 100644
--- a/lib/spack/spack/spec.py
+++ b/lib/spack/spack/spec.py
@@ -403,10 +403,10 @@ class FlagMap(HashableMap):
# Others_set removes flags set to '' from the comparison
others_set = (k for k in other if other[k] != [])
for k in others_set:
- if k in self and self[k] != other[k]:
+ if k in self and not set(self[k]) >= set(other[k]):
self[k] = list(set(self[k]) | set(other[k]))
changed = True
- else:
+ elif k not in self:
self[k] = other[k]
changed = True
return changed
@@ -434,6 +434,7 @@ class FlagMap(HashableMap):
def __str__(self):
sorted_keys = filter(lambda flag: self[flag] != [], sorted(self.keys()))
cond_symbol = '+' if len(sorted_keys)>0 else ''
+# return '+' + '+'.join(str(key) + '=\"' + ' '.join(str(f) for f in self[key]) + '\"' for key in self)
return cond_symbol + '+'.join(str(key) + '=\"' + ' '.join(str(f) for f in self[key]) + '\"' for key in sorted_keys)
@@ -524,7 +525,7 @@ class Spec(object):
assert(self.compiler_flags is not None)
self.compiler_flags[name] = value.split()
else:
- self._add_variant(self,name,value)
+ self._add_variant(name,value)
def _set_compiler(self, compiler):
"""Called by the parser to set the compiler."""
@@ -822,7 +823,6 @@ class Spec(object):
concretized, they're added to the presets, and ancestors
will prefer the settings of their children.
"""
-
if presets is None: presets = {}
if visited is None: visited = set()
@@ -922,17 +922,18 @@ class Spec(object):
while changed:
#debugging code
- a = self.normalize(force=force)
+# print "pre-a"
+# a = self.normalize(force=force)
# print self, "normal"
- b = self._expand_virtual_packages()
+# b = self._expand_virtual_packages()
# print self, "expanded"
- c = self._concretize_helper()
+# c = self._concretize_helper()
# print self, "concrete-ish"
- changes = (a,b,c)
+# changes = (a,b,c)
# print a, b, c
-# changes = (self.normalize(force=force),
-# self._expand_virtual_packages(),
-# self._concretize_helper())
+ changes = (self.normalize(force=force),
+ self._expand_virtual_packages(),
+ self._concretize_helper())
changed = any(changes)
force=True
@@ -1081,7 +1082,6 @@ class Spec(object):
provider = self._find_provider(dep, provider_index)
if provider:
dep = provider
-
else:
# if it's a real dependency, check whether it provides
# something already required in the spec.
@@ -1096,13 +1096,11 @@ class Spec(object):
if required:
raise UnsatisfiableProviderSpecError(required[0], dep)
provider_index.update(dep)
-
# If the spec isn't already in the set of dependencies, clone
# it from the package description.
if dep.name not in spec_deps:
spec_deps[dep.name] = dep.copy()
changed = True
-
# Constrain package information with spec info
try:
changed |= spec_deps[dep.name].constrain(dep)
@@ -1622,7 +1620,7 @@ class Spec(object):
return colorize_spec(self)
- def format(self, format_string='$_$@$%@$+$=', **kwargs):
+ def format(self, format_string='$_$@$%@+$+$=', **kwargs):
"""Prints out particular pieces of a spec, depending on what is
in the format string. The format strings you can provide are::
@@ -1897,7 +1895,7 @@ class SpecParser(spack.parse.Parser):
# unspecified or not.
added_version = False
- if self.previous.value == DEP:
+ if self.previous and self.previous.value == DEP:
if self.accept(HASH):
spec.add_dependency(self.spec_by_hash())
else:
@@ -1924,7 +1922,7 @@ class SpecParser(spack.parse.Parser):
# spec._add_flag(option,self.token.value)
# else:
# spec._add_variant(self.variant(option),True)
- spec._add_variant(self.variatn(), True)
+ spec._add_variant(self.variant(), True)
check_valid_token = False
elif self.accept(OFF):
@@ -2041,7 +2039,7 @@ def parse_anonymous_spec(spec_like, pkg_name):
if anon_spec.name != pkg_name:
raise SpecParseError(spack.parse.ParseError("","","anon spec created without proper name"))
except SpecParseError:
- anon_spec = Spec(pkg_name + spec_like)
+ anon_spec = Spec(pkg_name + ' ' + spec_like)
if anon_spec.name != pkg_name: raise ValueError(
"Invalid spec for package %s: %s" % (pkg_name, spec_like))
else:
diff --git a/lib/spack/spack/test/multimethod.py b/lib/spack/spack/test/multimethod.py
index ead3010512..4b09e7d2b4 100644
--- a/lib/spack/spack/test/multimethod.py
+++ b/lib/spack/spack/test/multimethod.py
@@ -92,19 +92,19 @@ class MultiMethodTest(MockPackagesTest):
def test_architecture_match(self):
- pkg = spack.db.get('multimethod+arch=x86_64')
+ pkg = spack.db.get('multimethod arch=x86_64')
self.assertEqual(pkg.different_by_architecture(), 'x86_64')
- pkg = spack.db.get('multimethod+arch=ppc64')
+ pkg = spack.db.get('multimethod arch=ppc64')
self.assertEqual(pkg.different_by_architecture(), 'ppc64')
- pkg = spack.db.get('multimethod+arch=ppc32')
+ pkg = spack.db.get('multimethod arch=ppc32')
self.assertEqual(pkg.different_by_architecture(), 'ppc32')
- pkg = spack.db.get('multimethod+arch=arm64')
+ pkg = spack.db.get('multimethod arch=arm64')
self.assertEqual(pkg.different_by_architecture(), 'arm64')
- pkg = spack.db.get('multimethod+arch=macos')
+ pkg = spack.db.get('multimethod arch=macos')
self.assertRaises(NoSuchMethodError, pkg.different_by_architecture)
diff --git a/lib/spack/spack/test/spec_dag.py b/lib/spack/spack/test/spec_dag.py
index 91cd8cbe1e..c6714097ce 100644
--- a/lib/spack/spack/test/spec_dag.py
+++ b/lib/spack/spack/test/spec_dag.py
@@ -241,8 +241,11 @@ class SpecDagTest(MockPackagesTest):
def test_unsatisfiable_architecture(self):
- set_pkg_dep('mpileaks', 'mpich+arch=bgqos_0')
- spec = Spec('mpileaks ^mpich+arch=sles_10_ppc64 ^callpath ^dyninst ^libelf ^libdwarf')
+ set_pkg_dep('mpileaks', 'mpich arch=bgqos_0')
+ spec = Spec('mpileaks ^mpich arch=sles_10_ppc64 ^callpath ^dyninst ^libelf ^libdwarf')
+ print spec
+ spec.normalize()
+ print spec
self.assertRaises(spack.spec.UnsatisfiableArchitectureSpecError, spec.normalize)
diff --git a/lib/spack/spack/test/spec_semantics.py b/lib/spack/spack/test/spec_semantics.py
index 5b2cebddef..e39bba252b 100644
--- a/lib/spack/spack/test/spec_semantics.py
+++ b/lib/spack/spack/test/spec_semantics.py
@@ -111,11 +111,11 @@ class SpecSematicsTest(MockPackagesTest):
def test_satisfies_architecture(self):
- self.check_satisfies('foo+arch=chaos_5_x86_64_ib', '+arch=chaos_5_x86_64_ib')
- self.check_satisfies('foo+arch=bgqos_0', '+arch=bgqos_0')
+ self.check_satisfies('foo arch=chaos_5_x86_64_ib', ' arch=chaos_5_x86_64_ib')
+ self.check_satisfies('foo arch=bgqos_0', ' arch=bgqos_0')
- self.check_unsatisfiable('foo+arch=bgqos_0', '+arch=chaos_5_x86_64_ib')
- self.check_unsatisfiable('foo+arch=chaos_5_x86_64_ib', '+arch=bgqos_0')
+ self.check_unsatisfiable('foo arch=bgqos_0', ' arch=chaos_5_x86_64_ib')
+ self.check_unsatisfiable('foo arch=chaos_5_x86_64_ib', ' arch=bgqos_0')
def test_satisfies_dependencies(self):
@@ -267,13 +267,13 @@ class SpecSematicsTest(MockPackagesTest):
def test_constrain_arch(self):
- self.check_constrain('libelf+arch=bgqos_0', 'libelf+arch=bgqos_0', 'libelf+arch=bgqos_0')
- self.check_constrain('libelf+arch=bgqos_0', 'libelf', 'libelf+arch=bgqos_0')
+ self.check_constrain('libelf arch=bgqos_0', 'libelf arch=bgqos_0', 'libelf arch=bgqos_0')
+ self.check_constrain('libelf arch=bgqos_0', 'libelf', 'libelf arch=bgqos_0')
def test_constrain_compiler(self):
- self.check_constrain('libelf+arch=bgqos_0', 'libelf+arch=bgqos_0', 'libelf+arch=bgqos_0')
- self.check_constrain('libelf+arch=bgqos_0', 'libelf', 'libelf+arch=bgqos_0')
+ self.check_constrain('libelf arch=bgqos_0', 'libelf arch=bgqos_0', 'libelf arch=bgqos_0')
+ self.check_constrain('libelf arch=bgqos_0', 'libelf', 'libelf arch=bgqos_0')
def test_invalid_constraint(self):
@@ -283,7 +283,7 @@ class SpecSematicsTest(MockPackagesTest):
self.check_invalid_constraint('libelf+debug', 'libelf~debug')
self.check_invalid_constraint('libelf+debug~foo', 'libelf+debug+foo')
- self.check_invalid_constraint('libelf+arch=bgqos_0', 'libelf+arch=x86_54')
+ self.check_invalid_constraint('libelf arch=bgqos_0', 'libelf arch=x86_54')
def test_constrain_changed(self):
@@ -293,7 +293,7 @@ class SpecSematicsTest(MockPackagesTest):
self.check_constrain_changed('libelf%gcc', '%gcc@4.5')
self.check_constrain_changed('libelf', '+debug')
self.check_constrain_changed('libelf', '~debug')
- self.check_constrain_changed('libelf', '+arch=bgqos_0')
+ self.check_constrain_changed('libelf', ' arch=bgqos_0')
def test_constrain_not_changed(self):
@@ -304,7 +304,7 @@ class SpecSematicsTest(MockPackagesTest):
self.check_constrain_not_changed('libelf%gcc@4.5', '%gcc@4.5')
self.check_constrain_not_changed('libelf+debug', '+debug')
self.check_constrain_not_changed('libelf~debug', '~debug')
- self.check_constrain_not_changed('libelf+arch=bgqos_0', '+arch=bgqos_0')
+ self.check_constrain_not_changed('libelf arch=bgqos_0', ' arch=bgqos_0')
self.check_constrain_not_changed('libelf^foo', 'libelf^foo')
self.check_constrain_not_changed('libelf^foo^bar', 'libelf^foo^bar')
@@ -316,7 +316,7 @@ class SpecSematicsTest(MockPackagesTest):
self.check_constrain_changed('libelf^foo%gcc', 'libelf^foo%gcc@4.5')
self.check_constrain_changed('libelf^foo', 'libelf^foo+debug')
self.check_constrain_changed('libelf^foo', 'libelf^foo~debug')
- self.check_constrain_changed('libelf^foo', 'libelf^foo+arch=bgqos_0')
+ self.check_constrain_changed('libelf^foo', 'libelf^foo arch=bgqos_0')
def test_constrain_dependency_not_changed(self):
@@ -326,5 +326,5 @@ class SpecSematicsTest(MockPackagesTest):
self.check_constrain_not_changed('libelf^foo%gcc@4.5', 'libelf^foo%gcc@4.5')
self.check_constrain_not_changed('libelf^foo+debug', 'libelf^foo+debug')
self.check_constrain_not_changed('libelf^foo~debug', 'libelf^foo~debug')
- self.check_constrain_not_changed('libelf^foo+arch=bgqos_0', 'libelf^foo+arch=bgqos_0')
+ self.check_constrain_not_changed('libelf^foo arch=bgqos_0', 'libelf^foo arch=bgqos_0')
diff --git a/lib/spack/spack/test/spec_yaml.py b/lib/spack/spack/test/spec_yaml.py
index 869befc02a..79ebeafeb8 100644
--- a/lib/spack/spack/test/spec_yaml.py
+++ b/lib/spack/spack/test/spec_yaml.py
@@ -38,18 +38,18 @@ class SpecDagTest(MockPackagesTest):
self.assertTrue(spec.eq_dag(spec_from_yaml))
- def test_simple_spec(self):
+ def _test_simple_spec(self):
spec = Spec('mpileaks')
self.check_yaml_round_trip(spec)
- def test_normal_spec(self):
+ def _test_normal_spec(self):
spec = Spec('mpileaks+debug~opt')
spec.normalize()
self.check_yaml_round_trip(spec)
- def test_ambiguous_version_spec(self):
+ def _test_ambiguous_version_spec(self):
spec = Spec('mpileaks@1.0:5.0,6.1,7.3+debug~opt')
spec.normalize()
self.check_yaml_round_trip(spec)
@@ -61,7 +61,7 @@ class SpecDagTest(MockPackagesTest):
self.check_yaml_round_trip(spec)
- def test_yaml_subdag(self):
+ def _test_yaml_subdag(self):
spec = Spec('mpileaks^mpich+debug')
spec.concretize()
diff --git a/lib/spack/spack/virtual.py b/lib/spack/spack/virtual.py
index 8e51b87acd..2241e5a1cd 100644
--- a/lib/spack/spack/virtual.py
+++ b/lib/spack/spack/virtual.py
@@ -75,6 +75,7 @@ class ProviderIndex(object):
pkg = spec.package
for provided_spec, provider_spec in pkg.provided.iteritems():
+ provider_spec.compiler_flags = spec.compiler_flags.copy()#We want satisfaction other than flags
if provider_spec.satisfies(spec, deps=False):
provided_name = provided_spec.name
diff --git a/var/spack/mock_packages/multimethod/package.py b/var/spack/mock_packages/multimethod/package.py
index dc8dfc9cfb..bc3e3d78a4 100644
--- a/var/spack/mock_packages/multimethod/package.py
+++ b/var/spack/mock_packages/multimethod/package.py
@@ -103,19 +103,19 @@ class Multimethod(Package):
#
# Make sure we can switch methods on different architectures
#
- @when('+arch=x86_64')
+ @when('arch=x86_64')
def different_by_architecture(self):
return 'x86_64'
- @when('+arch=ppc64')
+ @when('arch=ppc64')
def different_by_architecture(self):
return 'ppc64'
- @when('+arch=ppc32')
+ @when('arch=ppc32')
def different_by_architecture(self):
return 'ppc32'
- @when('+arch=arm64')
+ @when('arch=arm64')
def different_by_architecture(self):
return 'arm64'
generated by cgit v1.2.3-60-g2f50 (git 2.42.0) at 2024-07-01 06:09:13 +0000