Skip to content
Snippets Groups Projects
Commit 02e316e7 authored by Todd Gamblin's avatar Todd Gamblin
Browse files

Convert ValueErrors to SpackError subclass.

parent 2374eb4d
Branches
Tags
No related merge requests found
......@@ -941,16 +941,17 @@ def _check_extendable(self):
def _sanity_check_extension(self):
if not self.is_extension:
raise ValueError("This package is not an extension.")
raise ActivationError("This package is not an extension.")
extendee_package = self.extendee_spec.package
extendee_package._check_extendable()
if not extendee_package.installed:
raise ValueError("Can only (de)activate extensions for installed packages.")
raise ActivationError("Can only (de)activate extensions for installed packages.")
if not self.installed:
raise ValueError("Extensions must first be installed.")
raise ActivationError("Extensions must first be installed.")
if not self.extendee_spec.name in self.extendees:
raise ValueError("%s does not extend %s!" % (self.name, self.extendee.name))
raise ActivationError("%s does not extend %s!" % (self.name, self.extendee.name))
def do_activate(self, **kwargs):
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment