Skip to content
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

Add support for tcsh shell #3413

Open
Rageoholic opened this issue Jul 22, 2023 · 2 comments
Open

Add support for tcsh shell #3413

Rageoholic opened this issue Jul 22, 2023 · 2 comments
Labels
E-easy E-mentor enhancement help wanted O-shell Not really OS-related, but shell-specific

Comments

@Rageoholic
Copy link

Problem

On FreeBSD, rustup will generate a .cargo/env that is sourced in .profile. There are two issues here.

  1. tcsh does not source .profile on run, it sources .login
  2. Manually sourcing .cargo/env will give a syntax error that too many arguments were given to case

Steps

  1. Install FreeBSD on a fresh machine with tcsh as the set shell (I used a VM)
  2. Install rustup using the given directions on the website
  3. You can either try to source .cargo/env or exit and reenter the shell
  4. Try running any rustup commands

Possible Solution(s)

  • Modify .cargo/env to not use BASH specific extensions (I wouldn't quite know what specific syntax you should be using)
  • Add sourcing .cargo/env to .login if present,

Notes

No response

Rustup version

Rustup 1.26.0

Installed toolchains

rustc 1.71.0
@rbtcollins
Copy link
Contributor

rustup scatters a bunch of entries to source the env file, because we don't know what shell the user will be using (it may not be the one they installed with). We have some code that represents this and I don't think we have tcsh support - if you'd like to add that that would be great.

@rami3l
Copy link
Member

rami3l commented Aug 18, 2023

AFAIK tcsh isn't even POSIX, so supporting that falls into the same category as supporting fish, meaning that:

@rami3l rami3l changed the title Rustup install on freebsd and tcsh does not put cargo in PATH Add support for tcsh shell Jul 10, 2024
@rami3l rami3l added enhancement E-easy E-mentor O-shell Not really OS-related, but shell-specific and removed bug labels Jul 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
E-easy E-mentor enhancement help wanted O-shell Not really OS-related, but shell-specific
Projects
None yet
Development

No branches or pull requests

3 participants