qyriad/lix
1
0
Fork 0
forked from lix-project/lix
Code Pull requests Activity

don't add builtins to extras, initEnv() in regular repl

Browse source
This commit is contained in:
Ben Burdette 2021-12-27 14:06:04 -07:00
parent d0d5890445
commit ff82ba98b4
2 changed files with 4 additions and 19 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/libcmd/repl.cc
View file

@ -901,8 +901,8 @@ void runRepl(
repl->initEnv();
// add 'extra' vars.
std::set<std::string> names;
for (auto & [name, value] : extraEnv) {
// names.insert(ANSI_BOLD + name + ANSI_NORMAL);
names.insert(name);
@ -951,6 +951,7 @@ struct CmdRepl : StoreCommand, MixEvalArgs
auto repl = std::make_unique<NixRepl>(evalState);
repl->autoArgs = getAutoArgs(*repl->state);
repl->initEnv();
repl->mainLoop(files);
}
};

18
src/libexpr/eval.cc
View file

@ -715,6 +715,7 @@ void mapStaticEnvBindings(const StaticEnv &se, const Env &env, valmap & vm)
{
// add bindings for the next level up first, so that the bindings for this level
// override the higher levels.
// The top level bindings (builtins) are skipped since they are added for us by initEnv()
if (env.up && se.up) {
mapStaticEnvBindings(*se.up, *env.up,vm);
@ -725,23 +726,6 @@ void mapStaticEnvBindings(const StaticEnv &se, const Env &env, valmap & vm)
map[iter->first] = env.values[iter->second];
}
vm.merge(map);
}
else
{
std::cout << " -------------------- " << std::endl;
// iterate through staticenv bindings and add them,
// except for the __* ones.
auto map = valmap();
for (auto iter = se.vars.begin(); iter != se.vars.end(); ++iter)
{
std::cout << iter->first << std::endl;
std::string s = iter->first;
if (s.substr(0,2) != "__") {
map[iter->first] = env.values[iter->second];
}
}
vm.merge(map);
}
}