aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
Diffstat
-rw-r--r--perl/lib/Nix/Store.xs4
-rw-r--r--src/libexpr/primops.cc2
-rw-r--r--src/libstore/binary-cache-store.cc2
-rw-r--r--src/libstore/build.cc10
-rw-r--r--src/libstore/derivations.cc4
-rw-r--r--src/libstore/derivations.hh2
-rw-r--r--src/libstore/download.cc4
-rw-r--r--src/libstore/local-store.cc8
-rw-r--r--src/libstore/store-api.cc11
-rw-r--r--src/libstore/store-api.hh2
-rw-r--r--src/nix-prefetch-url/nix-prefetch-url.cc4
-rw-r--r--src/nix-store/nix-store.cc3
12 files changed, 27 insertions, 29 deletions
diff --git a/perl/lib/Nix/Store.xs b/perl/lib/Nix/Store.xs
index 697fd79f9..6b137a13c 100644
--- a/perl/lib/Nix/Store.xs
+++ b/perl/lib/Nix/Store.xs
@@ -288,8 +288,8 @@ SV * makeFixedOutputPath(int recursive, char * algo, char * hash, char * name)
PPCODE:
try {
HashType ht = parseHashType(algo);
- Path path = store()->makeFixedOutputPath(recursive, ht,
- parseHash16or32(ht, hash), name);
+ Hash h = parseHash16or32(ht, hash);
+ Path path = store()->makeFixedOutputPath(recursive, h, name);
XPUSHs(sv_2mortal(newSVpv(path.c_str(), 0)));
} catch (Error & e) {
croak("%s", e.what());
diff --git a/src/libexpr/primops.cc b/src/libexpr/primops.cc
index 25736ebff..c456e9b96 100644
--- a/src/libexpr/primops.cc
+++ b/src/libexpr/primops.cc
@@ -624,7 +624,7 @@ static void prim_derivationStrict(EvalState & state, const Pos & pos, Value * *
outputHash = printHash(h);
if (outputHashRecursive) outputHashAlgo = "r:" + outputHashAlgo;
- Path outPath = state.store->makeFixedOutputPath(outputHashRecursive, ht, h, drvName);
+ Path outPath = state.store->makeFixedOutputPath(outputHashRecursive, h, drvName);
drv.env["out"] = outPath;
drv.outputs["out"] = DerivationOutput(outPath, outputHashAlgo, outputHash);
}
diff --git a/src/libstore/binary-cache-store.cc b/src/libstore/binary-cache-store.cc
index 668f1e566..801ecd368 100644
--- a/src/libstore/binary-cache-store.cc
+++ b/src/libstore/binary-cache-store.cc
@@ -209,7 +209,7 @@ Path BinaryCacheStore::addToStore(const string & name, const Path & srcPath,
}
ValidPathInfo info;
- info.path = makeFixedOutputPath(recursive, hashAlgo, h, name);
+ info.path = makeFixedOutputPath(recursive, h, name);
addToStore(info, *sink.s, repair);
diff --git a/src/libstore/build.cc b/src/libstore/build.cc
index af2c908c3..10ae574f9 100644
--- a/src/libstore/build.cc
+++ b/src/libstore/build.cc
@@ -2706,8 +2706,8 @@ void DerivationGoal::registerOutputs()
hash). */
if (i.second.hash != "") {
- bool recursive; HashType ht; Hash h;
- i.second.parseHashInfo(recursive, ht, h);
+ bool recursive; Hash h;
+ i.second.parseHashInfo(recursive, h);
if (!recursive) {
/* The output path should be a regular file without
@@ -2719,11 +2719,11 @@ void DerivationGoal::registerOutputs()
/* Check the hash. In hash mode, move the path produced by
the derivation to its content-addressed location. */
- Hash h2 = recursive ? hashPath(ht, actualPath).first : hashFile(ht, actualPath);
+ Hash h2 = recursive ? hashPath(h.type, actualPath).first : hashFile(h.type, actualPath);
if (buildMode == bmHash) {
- Path dest = worker.store.makeFixedOutputPath(recursive, ht, h2, drv->env["name"]);
+ Path dest = worker.store.makeFixedOutputPath(recursive, h2, drv->env["name"]);
printMsg(lvlError, format("build produced path ‘%1%’ with %2% hash ‘%3%’")
- % dest % printHashType(ht) % printHash16or32(h2));
+ % dest % printHashType(h.type) % printHash16or32(h2));
if (worker.store.isValidPath(dest))
return;
Path actualDest = worker.store.toRealPath(dest);
diff --git a/src/libstore/derivations.cc b/src/libstore/derivations.cc
index 5590b99b3..7dcf71d46 100644
--- a/src/libstore/derivations.cc
+++ b/src/libstore/derivations.cc
@@ -9,7 +9,7 @@
namespace nix {
-void DerivationOutput::parseHashInfo(bool & recursive, HashType & hashType, Hash & hash) const
+void DerivationOutput::parseHashInfo(bool & recursive, Hash & hash) const
{
recursive = false;
string algo = hashAlgo;
@@ -19,7 +19,7 @@ void DerivationOutput::parseHashInfo(bool & recursive, HashType & hashType, Hash
algo = string(algo, 2);
}
- hashType = parseHashType(algo);
+ HashType hashType = parseHashType(algo);
if (hashType == htUnknown)
throw Error(format("unknown hash algorithm ‘%1%’") % algo);
diff --git a/src/libstore/derivations.hh b/src/libstore/derivations.hh
index e27c93713..974de78c5 100644
--- a/src/libstore/derivations.hh
+++ b/src/libstore/derivations.hh
@@ -29,7 +29,7 @@ struct DerivationOutput
this->hashAlgo = hashAlgo;
this->hash = hash;
}
- void parseHashInfo(bool & recursive, HashType & hashType, Hash & hash) const;
+ void parseHashInfo(bool & recursive, Hash & hash) const;
};
typedef std::map<string, DerivationOutput> DerivationOutputs;
diff --git a/src/libstore/download.cc b/src/libstore/download.cc
index f09926895..cf3929cad 100644
--- a/src/libstore/download.cc
+++ b/src/libstore/download.cc
@@ -232,7 +232,7 @@ Path Downloader::downloadCached(ref<Store> store, const string & url_, bool unpa
Path expectedStorePath;
if (expectedHash) {
- expectedStorePath = store->makeFixedOutputPath(unpack, expectedHash.type, expectedHash, name);
+ expectedStorePath = store->makeFixedOutputPath(unpack, expectedHash, name);
if (store->isValidPath(expectedStorePath))
return expectedStorePath;
}
@@ -282,7 +282,7 @@ Path Downloader::downloadCached(ref<Store> store, const string & url_, bool unpa
StringSink sink;
dumpString(*res.data, sink);
Hash hash = hashString(expectedHash ? expectedHash.type : htSHA256, *res.data);
- info.path = store->makeFixedOutputPath(false, hash.type, hash, name);
+ info.path = store->makeFixedOutputPath(false, hash, name);
info.narHash = hashString(htSHA256, *sink.s);
store->addToStore(info, *sink.s, false, true);
storePath = info.path;
diff --git a/src/libstore/local-store.cc b/src/libstore/local-store.cc
index 822d5fce3..96ce6a0d8 100644
--- a/src/libstore/local-store.cc
+++ b/src/libstore/local-store.cc
@@ -486,9 +486,9 @@ void LocalStore::checkDerivationOutputs(const Path & drvPath, const Derivation &
if (out == drv.outputs.end())
throw Error(format("derivation ‘%1%’ does not have an output named ‘out’") % drvPath);
- bool recursive; HashType ht; Hash h;
- out->second.parseHashInfo(recursive, ht, h);
- Path outPath = makeFixedOutputPath(recursive, ht, h, drvName);
+ bool recursive; Hash h;
+ out->second.parseHashInfo(recursive, h);
+ Path outPath = makeFixedOutputPath(recursive, h, drvName);
StringPairs::const_iterator j = drv.env.find("out");
if (out->second.path != outPath || j == drv.env.end() || j->second != outPath)
@@ -940,7 +940,7 @@ Path LocalStore::addToStoreFromDump(const string & dump, const string & name,
{
Hash h = hashString(hashAlgo, dump);
- Path dstPath = makeFixedOutputPath(recursive, hashAlgo, h, name);
+ Path dstPath = makeFixedOutputPath(recursive, h, name);
addTempRoot(dstPath);
diff --git a/src/libstore/store-api.cc b/src/libstore/store-api.cc
index 2a062b9b2..af002dcc8 100644
--- a/src/libstore/store-api.cc
+++ b/src/libstore/store-api.cc
@@ -191,13 +191,13 @@ Path Store::makeOutputPath(const string & id,
Path Store::makeFixedOutputPath(bool recursive,
- HashType hashAlgo, Hash hash, string name) const
+ const Hash & hash, const string & name) const
{
- return hashAlgo == htSHA256 && recursive
+ return hash.type == htSHA256 && recursive
? makeStorePath("source", hash, name)
: makeStorePath("output:out", hashString(htSHA256,
"fixed:out:" + (recursive ? (string) "r:" : "") +
- printHashType(hashAlgo) + ":" + printHash(hash) + ":"),
+ printHashType(hash.type) + ":" + printHash(hash) + ":"),
name);
}
@@ -205,10 +205,9 @@ Path Store::makeFixedOutputPath(bool recursive,
std::pair<Path, Hash> Store::computeStorePathForPath(const Path & srcPath,
bool recursive, HashType hashAlgo, PathFilter & filter) const
{
- HashType ht(hashAlgo);
- Hash h = recursive ? hashPath(ht, srcPath, filter).first : hashFile(ht, srcPath);
+ Hash h = recursive ? hashPath(hashAlgo, srcPath, filter).first : hashFile(hashAlgo, srcPath);
string name = baseNameOf(srcPath);
- Path dstPath = makeFixedOutputPath(recursive, hashAlgo, h, name);
+ Path dstPath = makeFixedOutputPath(recursive, h, name);
return std::pair<Path, Hash>(dstPath, h);
}
diff --git a/src/libstore/store-api.hh b/src/libstore/store-api.hh
index 75caab7ea..0b80312d6 100644
--- a/src/libstore/store-api.hh
+++ b/src/libstore/store-api.hh
@@ -232,7 +232,7 @@ public:
const Hash & hash, const string & name) const;
Path makeFixedOutputPath(bool recursive,
- HashType hashAlgo, Hash hash, string name) const;
+ const Hash & hash, const string & name) const;
/* This is the preparatory part of addToStore() and
addToStoreFixed(); it computes the store path to which srcPath
diff --git a/src/nix-prefetch-url/nix-prefetch-url.cc b/src/nix-prefetch-url/nix-prefetch-url.cc
index 44a16743d..00f5ae28d 100644
--- a/src/nix-prefetch-url/nix-prefetch-url.cc
+++ b/src/nix-prefetch-url/nix-prefetch-url.cc
@@ -146,7 +146,7 @@ int main(int argc, char * * argv)
Path storePath;
if (args.size() == 2) {
expectedHash = parseHash16or32(ht, args[1]);
- storePath = store->makeFixedOutputPath(unpack, ht, expectedHash, name);
+ storePath = store->makeFixedOutputPath(unpack, expectedHash, name);
if (store->isValidPath(storePath))
hash = expectedHash;
else
@@ -197,7 +197,7 @@ int main(int argc, char * * argv)
into the Nix store. */
storePath = store->addToStore(name, tmpFile, unpack, ht);
- assert(storePath == store->makeFixedOutputPath(unpack, ht, hash, name));
+ assert(storePath == store->makeFixedOutputPath(unpack, hash, name));
}
if (!printPath)
diff --git a/src/nix-store/nix-store.cc b/src/nix-store/nix-store.cc
index 78b2aa102..8532101a1 100644
--- a/src/nix-store/nix-store.cc
+++ b/src/nix-store/nix-store.cc
@@ -213,8 +213,7 @@ static void opPrintFixedPath(Strings opFlags, Strings opArgs)
string name = *i++;
cout << format("%1%\n") %
- store->makeFixedOutputPath(recursive, hashAlgo,
- parseHash16or32(hashAlgo, hash), name);
+ store->makeFixedOutputPath(recursive, parseHash16or32(hashAlgo, hash), name);
}
generated by cgit v1.2.3 (git 2.25.1) at 2024-11-22 04:20:06 +0000