Fixed issue #13

This commit is contained in:
Nick Van den Broeck 2019-03-21 09:30:16 +01:00
parent e6109ec765
commit 24b35bf9e7
6 changed files with 31 additions and 12 deletions

View file

@ -33,6 +33,15 @@ MixEvalArgs::MixEvalArgs()
.handler([&](std::vector<std::string> ss) {
evalSettings.pureEval = false;
});
mkFlag()
.longName("override-flake")
.labels({"original-ref", "resolved-ref"})
.description("override a flake registry value")
.arity(2)
.handler([&](std::vector<std::string> ss) {
registryOverrides.push_back(std::make_pair(ss[0], ss[1]));
});
}
Bindings * MixEvalArgs::getAutoArgs(EvalState & state)

View file

@ -16,6 +16,8 @@ struct MixEvalArgs : virtual Args
Strings searchPath;
std::vector<std::pair<std::string, std::string>> registryOverrides;
private:
std::map<std::string, std::string> autoArgs;

View file

@ -63,6 +63,8 @@ typedef std::list<SearchPathElem> SearchPath;
/* Initialise the Boehm GC, if applicable. */
void initGC();
typedef std::vector<std::pair<std::string, std::string>> RegistryOverrides;
class EvalState
{
@ -89,6 +91,9 @@ public:
const ref<Store> store;
RegistryOverrides registryOverrides;
private:
SrcToStore srcToStore;
@ -211,6 +216,8 @@ public:
path. Nothing is copied to the store. */
Path coerceToPath(const Pos & pos, Value & v, PathSet & context);
void addRegistryOverrides(RegistryOverrides overrides) { registryOverrides = overrides; }
public:
/* The base environment, containing the builtin functions and

View file

@ -140,10 +140,13 @@ std::shared_ptr<FlakeRegistry> getUserRegistry()
return readRegistry(getUserRegistryPath());
}
std::shared_ptr<FlakeRegistry> getFlagRegistry()
std::shared_ptr<FlakeRegistry> getFlagRegistry(RegistryOverrides registryOverrides)
{
// TODO (Nick): Implement this.
return std::make_shared<FlakeRegistry>();
auto flagRegistry = std::make_shared<FlakeRegistry>();
for (auto const & x : registryOverrides) {
flagRegistry->entries.insert_or_assign(FlakeRef(x.first), FlakeRef(x.second));
}
return flagRegistry;
}
// This always returns a vector with flakeRef, userReg, globalReg.
@ -151,7 +154,7 @@ std::shared_ptr<FlakeRegistry> getFlagRegistry()
const Registries EvalState::getFlakeRegistries()
{
Registries registries;
registries.push_back(getFlagRegistry());
registries.push_back(getFlagRegistry(registryOverrides));
registries.push_back(getUserRegistry());
registries.push_back(getGlobalRegistry());
return registries;
@ -357,9 +360,8 @@ NonFlake getNonFlake(EvalState & state, const FlakeRef & flakeRef, FlakeAlias al
ResolvedFlake resolveFlake(EvalState & state, const FlakeRef & topRef,
RegistryAccess registryAccess, bool isTopFlake)
{
Flake flake = getFlake(state, topRef,
registryAccess == AllowRegistry || (registryAccess == AllowRegistryAtTop && isTopFlake));
bool allowRegistries = registryAccess == AllowRegistry || (registryAccess == AllowRegistryAtTop && isTopFlake);
Flake flake = getFlake(state, topRef, allowRegistries);
LockFile lockFile;
if (isTopFlake)
@ -405,10 +407,8 @@ static LockFile makeLockFile(EvalState & evalState, FlakeRef & flakeRef)
void updateLockFile(EvalState & state, const Path & path)
{
// FIXME: don't copy 'path' to the store (especially since we
// dirty it immediately afterwards).
FlakeRef flakeRef = FlakeRef(path); // FIXME: ugly
// FIXME: We are writing the lockfile to the store here! Very bad practice!
FlakeRef flakeRef = FlakeRef(path);
auto lockFile = makeLockFile(state, flakeRef);
writeLockFile(lockFile, path + "/flake.lock");

View file

@ -54,7 +54,7 @@ struct CmdBuild : MixDryRun, InstallablesCommand
auto buildables = build(store, dryRun ? DryRun : Build, installables);
auto evalState = std::make_shared<EvalState>(searchPath, store);
evalState->addRegistryOverrides(registryOverrides);
if (dryRun) return;
for (size_t i = 0; i < buildables.size(); ++i) {

View file

@ -92,6 +92,7 @@ struct CmdFlakeDeps : FlakeCommand, MixJSON, StoreCommand, MixEvalArgs
void run(nix::ref<nix::Store> store) override
{
auto evalState = std::make_shared<EvalState>(searchPath, store);
evalState->addRegistryOverrides(registryOverrides);
FlakeRef flakeRef(flakeUri);