search

matklad / xshell

Apache License 2.0
696 stars 30 forks source link

Feature Request: Add option to print the enviroment with the command #83

Open elichai opened 7 months ago

elichai commented 7 months ago

something like Cmd::set_verbose(self, bool) that if set will print to stderr $ ENV1=KEY1 ENV2=KEY2 ... CMD ARGS this will help debug errors when setting enviroment variables

matklad commented 7 months ago

Makes sense to me! We already have Cmd::quiet / set_quiet which controls echoing of the command itself. One design question is the specific naming of methods. I think fn quiet is already wrong, as it is very unclear what that does. So perhaps we can deprecate that and come up with some clearer API

elichai commented 7 months ago

@matklad Maybe something like echo_command() and echo_env()?