nix repl: Print which variables are just loaded #11406
Conversation
github-actions
bot
added
the
repl
kstrafe
force-pushed
the
master
branch
4 times, most recently
from
06bb355 to
667a643
Compare
github-actions
bot
added
the
with-tests
kstrafe
force-pushed
the
master
branch
2 times, most recently
from
e9c815f to
9db7cce
Compare
src/libcmd/repl.cc
Outdated
| @@ -470,6 +472,10 @@ ProcessLineResult NixRepl::processLine(std::string line) | |||
| loadFlake(arg); | |||
| } | |||
|
|
|||
| else if (command == ":ll" || command == ":last-flake") { | |||
There was a problem hiding this comment.
What's the idea behind two ls?
Doesn't this also apply to variables loaded with :l <file>?
Maybe it should be :last-scope or :show-scope.
:ls is tempting, but we might want to use that for :b + readDirectory, so I'm not sure about that.
There was a problem hiding this comment.
ll = last loaded. The name :show-scope would be misleading I believe, since we're not actually showing the global scope, just the items that were most recently loaded. :last-scope or :ls might be a bit too close to the program ls and can cause confusion.
:ll applies to anything that loads an attr set into global scope, that is :a, :l, :lf, and repl cmdline args.
|
Hi @kstrafe, how's it going? Do you plan to continue to work on this, or do you prefer that someone else takes over? |
Hey, I've just come back to this. I've added your suggestions and fixed up the tests. They aren't very easy to debug. I've added dumps to files and Is there a more practical way? It's quite slow. Anyhow, we now show all loaded variables on a single line. I have not handled escaping (for keys containing commas and spaces), but I believe that's quite rare so probably not very important to handle. Is there any way to test the pager as well? I'd like to test |
|
@roberth mentioning you since your profile mentioned it. |
|
What is the state of this? What need be done to get it merged? |
When we run `nix repl nixpkgs` we get "Added 6 variables". This is not useful as it doesn't tell us which variables the flake has exported to our global repl scope. This patch prints the name of each variable that was just loaded. We currently cap printing to 10 variables in order to avoid excessive prints. NixOS#11404
Invoking `:ll` will start a pager with all variables which have just been loaded by `:lf`, `:l`, or by a flake provided to `nix repl` as an argument. NixOS#11404
|
@roberth everything should be good now. Let me know if there's anything else that needs to be done before merging. |
When we run
nix repl nixpkgswe get "Added 6 variables". This is not useful as it doesn't tell us which variables the flake has exported to our global repl scope.This patch prints the name of each variable that was just loaded. We
currently cap printing to 10 variables in order to avoid excessive
prints.
This PR also introduces
:llfor listing all previously loaded variables in a pager. This list is overwritten by the next load.Github issue: 11404
This PR also introduces
:llwhich starts a pager containing all loaded variables.