From 6a0a2d559336b3a7a165fb0f87f5e70caa9d8d72 Mon Sep 17 00:00:00 2001 From: Eelco Dolstra Date: Thu, 20 Jan 2005 16:01:07 +0000 Subject: [PATCH] * Terminology fixes. --- src/libstore/build.cc | 18 +++++++++--------- src/libstore/build.hh | 20 ++++++++++---------- src/libstore/derivations.cc | 2 +- src/libstore/store.cc | 2 +- src/libstore/store.hh | 2 +- 5 files changed, 22 insertions(+), 22 deletions(-) diff --git a/src/libstore/build.cc b/src/libstore/build.cc index 9e984a5b3..072880765 100644 --- a/src/libstore/build.cc +++ b/src/libstore/build.cc @@ -15,7 +15,7 @@ #include "globals.hh" -/* !!! TODO storeExprFromPath shouldn't be used here */ +/* !!! TODO derivationFromPath shouldn't be used here */ static string pathNullDevice = "/dev/null"; @@ -129,7 +129,7 @@ private: unsigned int nrChildren; /* Maps used to prevent multiple instantiations of a goal for the - same expression / path. */ + same derivation / path. */ WeakGoalMap derivationGoals; WeakGoalMap substitutionGoals; @@ -290,10 +290,10 @@ const char * * strings2CharPtrs(const Strings & ss) class DerivationGoal : public Goal { private: - /* The path of the derivation store expression. */ + /* The path of the derivation. */ Path drvPath; - /* The derivation store expression stored at drvPath. */ + /* The derivation stored at drvPath. */ Derivation drv; /* The remainder is state held during the build. */ @@ -415,7 +415,7 @@ void DerivationGoal::init() { trace("init"); - /* The first thing to do is to make sure that the store expression + /* The first thing to do is to make sure that the derivation exists. If it doesn't, it may be created through a substitute. */ addWaitee(worker.makeSubstitutionGoal(drvPath)); @@ -735,8 +735,8 @@ DerivationGoal::HookReply DerivationGoal::tryBuildHook() % showPaths(outputPaths(drv.outputs))); /* Write the information that the hook needs to perform the - build, i.e., the set of input paths (including closure - expressions), the set of output paths, and [!!!]. */ + build, i.e., the set of input paths, the set of output + paths, and [!!!]. */ Path inputListFN = tmpDir + "/inputs"; Path outputListFN = tmpDir + "/outputs"; @@ -791,7 +791,7 @@ bool DerivationGoal::prepareBuild() reuse its results. (Strictly speaking the first check can be omitted, but that would be less efficient.) Note that since we now hold the locks on the output paths, no other process can - build this expression, so no further checks are necessary. */ + build this derivation, so no further checks are necessary. */ if (allOutputsValid()) { debug(format("skipping build of derivation `%1%', someone beat us to it") % drvPath); @@ -910,7 +910,7 @@ void DerivationGoal::startBuilder() in the store or in the build directory). */ env["NIX_STORE"] = nixStore; - /* Add all bindings specified in the derivation expression. */ + /* Add all bindings specified in the derivation. */ for (StringPairs::iterator i = drv.env.begin(); i != drv.env.end(); ++i) env[i->first] = i->second; diff --git a/src/libstore/build.hh b/src/libstore/build.hh index bfc015733..ed83d678e 100644 --- a/src/libstore/build.hh +++ b/src/libstore/build.hh @@ -3,20 +3,20 @@ #include "derivations.hh" -/* Perform the specified derivations, if necessary. That is, do - whatever is necessary to create the output paths of the derivation. - If the output paths already exists, we're done. If they have - substitutes, we can use those instead. Otherwise, the build action - described by the derivation is performed, after recursively - building any sub-derivations. */ +/* Ensure that the output paths of the derivation are valid. If they + are already valid, this is a no-op. Otherwise, validity can + be reached in two ways. First, if the output paths have + substitutes, then those can be used. Second, the output paths can + be created by running the builder, after recursively building any + sub-derivations. */ void buildDerivations(const PathSet & drvPaths); -/* Ensure that a path exists, possibly by instantiating it by - realising a substitute. */ +/* Ensure that a path is valid. If it is not currently valid, it may + be made valid by running a substitute (if defined for the path). */ void ensurePath(const Path & storePath); -/* Read a derivation store expression, after ensuring its existence - through ensurePath(). */ +/* Read a derivation, after ensuring its existence through + ensurePath(). */ Derivation derivationFromPath(const Path & drvPath); /* Place in `paths' the set of all store paths in the file system diff --git a/src/libstore/derivations.cc b/src/libstore/derivations.cc index 9008be439..2d776fb74 100644 --- a/src/libstore/derivations.cc +++ b/src/libstore/derivations.cc @@ -22,7 +22,7 @@ Path writeDerivation(const Derivation & drv, const string & name) static void checkPath(const string & s) { if (s.size() == 0 || s[0] != '/') - throw Error(format("bad path `%1%' in store expression") % s); + throw Error(format("bad path `%1%' in derivation") % s); } diff --git a/src/libstore/store.cc b/src/libstore/store.cc index 631984784..97b7f5fe8 100644 --- a/src/libstore/store.cc +++ b/src/libstore/store.cc @@ -51,7 +51,7 @@ static TableId dbReferers = 0; One system can compute a derivate and put it on a website (as a Nix archive), for instance, and then another system can register a substitute for that derivate. The substitute in this case might be - a Nix expression that fetches the Nix archive. + a Nix derivation that fetches the Nix archive. */ static TableId dbSubstitutes = 0; diff --git a/src/libstore/store.hh b/src/libstore/store.hh index 8f5190f80..d59634a83 100644 --- a/src/libstore/store.hh +++ b/src/libstore/store.hh @@ -45,7 +45,7 @@ typedef list > SubstitutePairs; void registerSubstitutes(const Transaction & txn, const SubstitutePairs & subPairs); -/* Return the substitutes expression for the given path. */ +/* Return the substitutes for the given path. */ Substitutes querySubstitutes(const Path & srcPath); /* Deregister all substitutes. */