-
Notifications
You must be signed in to change notification settings - Fork 140
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use `sh' as default shell #161
Comments
I solved my own problem! I had an ever growing user path environment variable definition in my config.fish, causing it to traverse the same path n times where n is the amount of times I had instantiated fish since adding that definition. https://stackoverflow.com/questions/26208231/modifying-path-with-fish-shell#34401308 |
Would like to suggest reopening this issue to look into using sh/bash by default since I was running into the problem that sxhkd was feeling rather sluggish for me and was fixed by setting the SXHKD_SHELL to point to something more light-weight. I am much happier after the change was made and I imagine I am not the only one that has felt this sluggishness and only serendipitously stumbled upon the solution. |
I disagree. You will only have this "problem" if you have set your login shell to something other than Most Also, most EXTRA: |
Far point, but I am not advocating for any particular shell be picked necessarily. However, I would imagine that many that use The only reason I am suggesting something like this is that I was dealing with this problem that is essentially invisible and undebuggable as it did not even occur to me that it was the shell that was being used that was causing the sluggishness of Totally think your points about forcing a default are valid and it would be hard to pick something that works on all target systems, but maybe at least making this significant optimization to the experience being somehow visible to users that may not even realize that they are having a degraded experience? |
If the problem is to let users that have set their login shell to a shell with an high overhead (at least higher than bash's) and have installed Benchmark comparison between
copiable snippet of the benchmarking code
for shell in bash fish zsh ksh dash; do
type "$shell" &> /dev/null || continue
TIMEFORMAT=$shell$'\t%R %U %S'
time for (( i = 1; i <= 2000; ++i )); do "$shell" -c '/bin/echo hi'; done > /dev/null
done |& sort -k2,2g | nl |
Use a shell with fast start-up time: baskerville/sxhkd#161
I ran into the issue with fish solved by this comment after much trial and error messing around with startup items in my xinitrc.
I tested other shells startup times and fish seems uniquely slow in starting up compared to others (zsh, tcsh, bash, sh).
My suggestion would be to default to `sh' using the SXHKD_SHELL variable, or at the very least warn in the README or somewhere that fish users may experience significant delays in executing commands due to this behaviour
The text was updated successfully, but these errors were encountered: