Fix premature validation for properties.

Properties should validate in _validate(), not in _set(), so that we can
get help, version info, etc. This appears to be the intent because
Executive.verifyConfiguration() will call _validate() and nicely prints
all of the errors, not just the first one.

pyre/inventory/Configurable.py

@@ -68,23 +68,13 @@ def updateConfiguration(self, registry):
     def applyConfiguration(self, context=None):
         """transfer user settings to my inventory"""
 
-        try:
-
-            if context is None:
-                context = self.newConfigContext()
+        if context is None:
+            context = self.newConfigContext()
 
-            context.configureComponent(self)
+        context.configureComponent(self)
 
-            # give descendants a chance to adjust to configuration changes
-            self._configure()
-
-
-        except ValueError, err:
-            aliases = ", ".join(self.aliases)
-            raise ValueError("%s\nBacktrace - Component %s" % (err.message, aliases))
-        except RuntimeError, err:
-            aliases = ", ".join(self.aliases)
-            raise RuntimeError("%s\nBacktrace - Component %s" % (err.message, aliases))
+        # give descendants a chance to adjust to configuration changes
+        self._configure()
 
         return context
 

pyre/inventory/Property.py

@@ -48,9 +48,6 @@ def _getDefaultValue(self, instance):
         if value is not None:
             # convert
             value = self._cast(value)
-            # validate 
-            if self.validator:
-                value = self.validator(value)
 
         import pyre.parsing.locators
         locator = pyre.parsing.locators.default()
@@ -63,6 +60,8 @@ def _cast(self, input):
 
 
     def _validate(self, value):
+        if self.validator:
+            value = self.validator(value)
         return value