maurice/snix
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

chore(3p/nix): Remove support for plugins

Browse source 
Plugins seem to not really be used anywhere (I can find one plugin
that's actually defined, and it doesn't seem very useful, especially
since we got rid of builtins.exec) and their presence is adding
additional complexity and potential sources of bugs to an already
unsteady refactor. At some point we may want to bring back
something *like* plugins, but their design will likely be different and
it will definitely be after we have a functioning Nix again.

Change-Id: I3bc40e55917f70bf260fbc208c1705e2e6a7c626
Reviewed-on: https://cl.tvl.fyi/c/depot/+/1291
Tested-by: BuildkiteCI
Reviewed-by: Alyssa Ross <hi@alyssa.is>
Reviewed-by: isomer <isomer@tvl.fyi>
This commit is contained in:
Griffin Smith 2020-07-19 14:35:05 -04:00 • committed by glittershark
parent 31516eb9cd
commit 2ef1060361
18 changed files with 1 additions and 128 deletions
Download patch file Download diff file Expand all files Collapse all files

2
third_party/nix/src/build-remote/build-remote.cc vendored
View file

@ -68,8 +68,6 @@ static int _main(int argc, char* argv[]) {
settings.maxBuildJobs.set("1"); // hack to make tests with local?root= work
initPlugins();
auto store = openStore().cast<LocalStore>();
/* It would be more appropriate to use $XDG_RUNTIME_DIR, since

4
third_party/nix/src/libexpr/primops.hh vendored
View file

@ -14,9 +14,7 @@ struct RegisterPrimOp {
RegisterPrimOp(const std::string& name, size_t arity, PrimOpFun fun);
};
/* These primops are disabled without enableNativeCode, but plugins
may wish to use them in limited contexts without globally enabling
them. */
/* These primops are disabled without enableNativeCode */
/* Load a ValueInitializer from a DSO and return whatever it initializes */
void prim_importNative(EvalState& state, const Pos& pos, Value** args,
Value& v);

1
third_party/nix/src/libmain/shared.hh vendored
View file

@ -22,7 +22,6 @@ class Exit : public std::exception {
int handleExceptions(const std::string& programName,
const std::function<void()>& fun);
/* Don't forget to call initPlugins() after settings are initialized! */
void initNix();
void parseCmdLine(

31
third_party/nix/src/libstore/globals.cc vendored
View file

@ -167,35 +167,4 @@ void MaxBuildJobsSetting::set(const std::string& str) {
}
}
void initPlugins() {
for (const auto& pluginFile : settings.pluginFiles.get()) {
Paths pluginFiles;
try {
auto ents = readDirectory(pluginFile);
for (const auto& ent : ents) {
pluginFiles.emplace_back(pluginFile + "/" + ent.name);
}
} catch (SysError& e) {
if (e.errNo != ENOTDIR) {
throw;
}
pluginFiles.emplace_back(pluginFile);
}
for (const auto& file : pluginFiles) {
/* handle is purposefully leaked as there may be state in the
DSO needed by the action of the plugin. */
void* handle = dlopen(file.c_str(), RTLD_LAZY | RTLD_LOCAL);
if (handle == nullptr) {
throw Error("could not dynamically open plugin file '%s': %s", file,
dlerror());
}
}
}
/* Since plugins can add settings, try to re-apply previously
unknown settings. */
globalConfig.reapplyUnknownSettings();
globalConfig.warnUnknownSettings();
}
} // namespace nix

10
third_party/nix/src/libstore/globals.hh vendored
View file

@ -452,21 +452,11 @@ class Settings : public Config {
Setting<uint64_t> minFreeCheckInterval{
this, 5, "min-free-check-interval",
"Number of seconds between checking free disk space."};
Setting<Paths> pluginFiles{
this,
{},
"plugin-files",
"Plugins to dynamically load at nix initialization time."};
};
// FIXME: don't use a global variable.
extern Settings settings;
/* This should be called after settings are initialized, but before
anything else */
void initPlugins();
void loadConfFile();
extern const std::string nixVersion;

2
third_party/nix/src/nix-build/nix-build.cc vendored
View file

@ -255,8 +255,6 @@ static void _main(int argc, char** argv) {
myArgs.parseCmdline(args);
initPlugins();
if (packages && fromArgs) {
throw UsageError("'-p' and '-E' are mutually exclusive");
}

2
third_party/nix/src/nix-channel/nix-channel.cc vendored
View file

@ -214,8 +214,6 @@ static int _main(int argc, char** argv) {
return true;
});
initPlugins();
switch (cmd) {
case cNone:
throw UsageError("no command specified");

2
third_party/nix/src/nix-collect-garbage/nix-collect-garbage.cc vendored
View file

@ -82,8 +82,6 @@ static int _main(int argc, char** argv) {
return true;
});
initPlugins();
auto profilesDir = settings.nixStateDir + "/profiles";
if (removeOld) {
removeOldGenerations(profilesDir);

2
third_party/nix/src/nix-copy-closure/nix-copy-closure.cc vendored
View file

@ -48,8 +48,6 @@ static int _main(int argc, char** argv) {
return true;
});
initPlugins();
if (sshHost.empty()) {
throw UsageError("no host name specified");
}

2
third_party/nix/src/nix-daemon/nix-daemon.cc vendored
View file

@ -1107,8 +1107,6 @@ static int _main(int argc, char** argv) {
return true;
});
initPlugins();
if (stdio) {
if (getStoreType() == tDaemon) {
/* Forward on this connection to the real daemon */

2
third_party/nix/src/nix-env/nix-env.cc vendored
View file

@ -1496,8 +1496,6 @@ static int _main(int argc, char** argv) {
myArgs.parseCmdline(argvToStrings(argc, argv));
initPlugins();
if (op == nullptr) {
throw UsageError("no operation specified");
}

2
third_party/nix/src/nix-instantiate/nix-instantiate.cc vendored
View file

@ -153,8 +153,6 @@ static int _main(int argc, char** argv) {
myArgs.parseCmdline(argvToStrings(argc, argv));
initPlugins();
if (evalOnly && !wantsReadWrite) {
settings.readOnlyMode = true;
}

2
third_party/nix/src/nix-prefetch-url/nix-prefetch-url.cc vendored
View file

@ -100,8 +100,6 @@ static int _main(int argc, char** argv) {
myArgs.parseCmdline(argvToStrings(argc, argv));
initPlugins();
if (args.size() > 2) {
throw UsageError("too many arguments");
}

2
third_party/nix/src/nix-store/nix-store.cc vendored
View file

@ -1276,8 +1276,6 @@ static int _main(int argc, char** argv) {
return true;
});
initPlugins();
if (op == nullptr) {
throw UsageError("no operation specified");
}

2
third_party/nix/src/nix/main.cc vendored
View file

@ -144,8 +144,6 @@ void mainWrapped(int argc, char** argv) {
args.parseCmdline(argvToStrings(argc, argv));
initPlugins();
if (!args.command) {
args.showHelpAndExit();
}