Changed recipe_modules base paths to be abstract entities in their own right.

scripts/slave/recipe_config_types.py

Index: scripts/slave/recipe_config_types.py
diff --git a/scripts/slave/recipe_config_types.py b/scripts/slave/recipe_config_types.py
index 0c298d2fdb5bea6b3af5b0ce63437dcb39d59318..a92a2106c8f2ed5e2317496d3fb075513cf79fb4 100644
--- a/scripts/slave/recipe_config_types.py
+++ b/scripts/slave/recipe_config_types.py
@@ -2,8 +2,11 @@
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
+import abc
 import re
 
+from collections import namedtuple
+
 from infra.libs import infra_types
 
 def ResetTostringFns():
@@ -60,6 +63,37 @@ class RecipeConfigType(object):
     return self.tostring_fn(self) # pylint: disable=not-callable
 
 
+class BasePath(object):
+  __metaclass__ = abc.ABCMeta
+
+
+class NamedBasePath(BasePath, namedtuple('NamedBasePath', 'name')):
+  # Restrict basenames to '[ALL_CAPS]'. This will help catch
+  # errors if someone attempts to provide an actual string path '/some/example'
+  # as the 'base'.
+  BASE_RE = re.compile(r'\[([A-Z][A-Z_]*)\]')
+
+  @staticmethod
+  def parse(base):
+    base_match = NamedBasePath.BASE_RE.match(base)
+    assert base_match, 'Base should be [ALL_CAPS], got %r' % base
+    return NamedBasePath(base_match.group(1).lower())
+
+  def __repr__(self):
+    return '[%s]' % self.name.upper()
+
+
+class ModuleBasePath(BasePath, namedtuple('ModuleBasePath', ('module',))):

[iannucci, 2015/03/07 02:55:02]

I think this r an oops. Let's stick with line 70 style

[luqui, 2015/03/09 18:49:28]

Done.

+  def __repr__(self):
+    name = re.sub(r'^RECIPE_MODULES\.', '', self.module.__name__)

[iannucci, 2015/03/07 02:55:03]

let's pull the regex out too, and add a comment as appropriate (similar to the
other one)

[luqui, 2015/03/09 18:49:28]

Done.

+    return 'MODULE[%s]' % name

[iannucci, 2015/03/07 02:55:02]

let's do "RECIPE_MODULE" instead, since MODULE is overloaded (esp if you see
MODULE[os], etc)

[luqui, 2015/03/09 18:49:28]

Done.

+
+
+class ConcreteBasePath(BasePath, namedtuple('ConcreteBasePath', 'path')):
+  def __repr__(self):
+    return self.path
+
+
 class Path(RecipeConfigType):
   """Represents a path which is relative to a semantically-named base.
 
@@ -68,10 +102,6 @@ class Path(RecipeConfigType):
   absolute path, we only store three context-free attributes in this Path
   object.
   """
-  # Restrict basenames to '[ALL_CAPS]'. This will help catch
-  # errors if someone attempts to provide an actual string path '/some/example'
-  # as the 'base'.
-  BASE_RE = re.compile(r'\[([A-Z][A-Z_]*)\]')
 
   def __init__(self, base, *pieces, **kwargs):
     """Creates a Path
@@ -85,20 +115,18 @@ class Path(RecipeConfigType):
     Kwargs:
       platform_ext (dict(str, str)) - A mapping from platform name (as defined
         by the 'platform' module), to a suffix for the path.
-      _bypass (bool) - Bypass the type checking and use |base| directly. Don't
-        use this outside of the 'path' module or this class.
     """
     super(Path, self).__init__()
     assert all(isinstance(x, basestring) for x in pieces), pieces
     assert not any(x in ('..', '.', '/', '\\') for x in pieces)
     self.pieces = pieces
 
-    if kwargs.get('_bypass'):
+    if isinstance(base, BasePath):
       self.base = base
+    elif isinstance(base, basestring):
+      self.base = NamedBasePath.parse(base)
     else:
-      base_match = self.BASE_RE.match(base)
-      assert base_match, 'Base should be [ALL_CAPS], got %r' % base
-      self.base = base_match.group(1).lower()
+      raise ValueError('%s is not a valid base path' % base)
 
     self.platform_ext = kwargs.get('platform_ext', {})
 
@@ -112,7 +140,6 @@ class Path(RecipeConfigType):
 
   def join(self, *pieces, **kwargs):
     kwargs.setdefault('platform_ext', self.platform_ext)
-    kwargs['_bypass'] = True
     return Path(self.base, *filter(bool, self.pieces + pieces), **kwargs)
 
   def is_parent_of(self, child):
@@ -133,4 +160,4 @@ class Path(RecipeConfigType):
     pieces = ''
     if self.pieces:
       pieces = ', ' + (', '.join(map(repr, self.pieces)))
-    return 'Path(\'[%s]\'%s%s)' % (self.base.upper(), pieces, suffix)
+    return 'Path(\'%s\'%s%s)' % (self.base, pieces, suffix)