-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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
Getting Started issues #26855
Getting Started issues #26855
Comments
Hi @ericfrederich. I think this could be fixed easily by just adjusting their install script. I also filed a duplicate issue at #26824 |
I will close mine in favour of this since it also seems it has another issue written here. |
Thanks @uncomfyhalomacro. Perhaps it should be discussed the proper place on Linux to put these. This isn't my area of expertise to say that it should be in When I look at my system I actually don't have an existing
Perhaps the completions should be put in the user's directory under |
I don't remember bash reading those directories (i also think it requires an environmental variable). i do believe that the proposal i have is where bash usually looks for completions
i might be wrong so it still requires discussion. i think just putting them in "standard" bash completion directories should be okay |
Version: Deno 2.0.6
OS: Ubuntu 24.04 via WSL2
Couple of issues I noticed simply running
curl -fsSL https://deno.land/install.sh | sh
as instructed in the Getting Started page. "/home/username/.deno/env"
to my ~/.bashrc and ~/.profile but does not add a trailing newline. Obviously a bug because it means subsequent scripts that want to do the same thing would end up appending to that line.warning: Failed to install completions for bash: PermissionDenied: Permission denied (os error 13): mkdir '/usr/local/etc/bash_completion.d'
. The docs do not mention the need to run this as root. Perhaps those files should be installed in the user's directory and not globally.The text was updated successfully, but these errors were encountered: