Add a repo_name to Source, use that instead of repo in most places.

This is because we will need to integrate a RemoteRepoCache and we don't
always want to create a CachedRepo object (in fact, we only want that
for the sources of artifacts we actually need to build). So we'll use
either the local or the remote repo cache to resolve refs and morphology
texts and then later, when actually caching a repo, we'll set the
source.repo member to a CachedRepo object that we can then use to
unpack the sources.

1 files changed, 5 insertions, 3 deletions

diff --git a/morphlib/source.py b/morphlib/source.py
index dc535a55..ff440f0b 100644
--- a/morphlib/source.py
+++ b/morphlib/source.py
@@ -24,6 +24,7 @@ class Source(object):
     Has the following properties:
     
     * ``repo`` -- the git repository which contains the source
+    * ``repo_name`` -- name of the git repository which contains the source
     * ``original_ref`` -- the git ref provided by the user or a morphology
     * ``sha1`` -- the absolute git commit id for the revision we use
     * ``morphology`` -- the in-memory representation of the morphology we use
@@ -36,9 +37,10 @@ class Source(object):
     
     '''
     
-    def __init__(self, repo, original_ref, sha1, morphology, filename):
+    def __init__(self, repo_name, original_ref, sha1, morphology, filename):
         assert isinstance(morphology, morphlib.morph2.Morphology)
-        self.repo = repo
+        self.repo = None
+        self.repo_name = repo_name
         self.original_ref = original_ref
         self.sha1 = sha1
         self.morphology = morphology
@@ -57,6 +59,6 @@ class Source(object):
         return source in self.dependencies
 
     def __str__(self): # pragma: no cover
-        return '%s|%s|%s' % (self.repo.original_name,
+        return '%s|%s|%s' % (self.repo_name,
                              self.original_ref,
                              self.filename)