diff options
| author | Sam Thursfield <sam.thursfield@codethink.co.uk> | 2013-02-22 19:07:13 +0000 |
|---|---|---|
| committer | Sam Thursfield <sam.thursfield@codethink.co.uk> | 2013-03-13 15:20:02 +0000 |
| commit | ece7f823de6bd61a0676edf71a9525697848824e (patch) | |
| tree | 98c408d5960915f0cb5b98a941e2dc6508047f8b /morphlib/artifact.py | |
| parent | 6a61dd9cc1fe8a3ccd2128fb628ed929fd496ad0 (diff) | |
| download | morph-ece7f823de6bd61a0676edf71a9525697848824e.tar.gz | |
Refactor build process
Reorganise the build_artifact() and build_artifacts() functions to
allow more complex work when setting up chunk builds in
build_artifact().
The staging area now holds the BuildEnvironment object (the
environment variables that should be set during build). This makes sense
because all build commands should be run inside the staging area and
therefore through the StagingArea object.
The BuildEnvironment object is now considered immutable after it is
created. The environment is used in cache key computation when
computing what artifacts are required; if it changes after that point
we risk either computing different artifact keys for the same artifact
or missing data in the cache key that should be included in the hash.
Better to force changes into a separate 'extra_env' variable.
Diffstat (limited to 'morphlib/artifact.py')
| -rw-r--r-- | morphlib/artifact.py | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/morphlib/artifact.py b/morphlib/artifact.py index aef48d76..5f084384 100644 --- a/morphlib/artifact.py +++ b/morphlib/artifact.py @@ -83,4 +83,3 @@ class Artifact(object): yield a return list(depth_first(self)) - |