remove builtins.exec #3355
remove builtins.exec #3355
Conversation
…tion." This reverts commit 0bb8db2.
|
builtins.exec is already hidden behind a very verbose flag, and allows for a much nicer extension experience without any native code. What is the concern? |
|
People see |
|
Are there real examples of people shooting themselves in the foot or failing to do things the right way due to this? |
|
It's a massive opportunity to shoot yourself in the foot. It's undocumented, 0bb8db2 mentioned it was meant as an intermediate thing, so I think if people want it, moving it to an optional plugin would be the right move. |
|
Would removing it solve a actual current problem? This provides an escape hatch to do funny and crazy things. It's properly protected behind a scary flag. |
|
Closing since it doesn't really harm anybody as it's hidden behind a flag. |
|
I would put this behind an experimental feature. |
IMO that feature is very dangerous to have in Nix and probably doesn't fit the entire model at all.
It currently isn't documented so there shouldn't be many users. Those that need it can probably add this via a plugin. The plugin example (
tests/plugins/plugintest.c) already shows how to add such a plugin and the removed code can probably just pasted into that file to get started.