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

feat(installer): add additional shell config #555

Merged
merged 3 commits into from
Dec 4, 2023

Conversation

mistydemeo
Copy link
Contributor

~/.profile isn't sourced in many circumstances; we need to write this to a large number of locations to make sure it gets considered by bash and zsh under the largest number of circumstances.

Fixes #547.

@mistydemeo mistydemeo requested a review from Gankra November 8, 2023 00:12
@mistydemeo mistydemeo marked this pull request as ready for review November 8, 2023 00:12
Copy link
Contributor

@Gankra Gankra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shippable, but see comment for how this differs from rustup

cargo-dist/templates/installer/installer.sh.j2 Outdated Show resolved Hide resolved
@mistydemeo mistydemeo force-pushed the installer_source_targets branch 4 times, most recently from 26a1535 to f90ea74 Compare November 29, 2023 20:06
~/.profile isn't sourced in many circumstances; we need to write this
to a large number of locations to make sure it gets considered by bash and
zsh under the largest number of circumstances.

We have three sets of configuration files, covering generic POSIX, bash, and zsh.
We pick the first file that exists in each set, and if none exists, we create the
first in the list.

Fixes #547.
@mistydemeo mistydemeo force-pushed the installer_source_targets branch 2 times, most recently from 00fd938 to a96bcc7 Compare December 4, 2023 20:02
@mistydemeo mistydemeo force-pushed the installer_source_targets branch 4 times, most recently from 3155bbf to 5ed9108 Compare December 4, 2023 21:21
@mistydemeo mistydemeo force-pushed the installer_source_targets branch 4 times, most recently from 04151f4 to add2143 Compare December 4, 2023 21:50
@mistydemeo mistydemeo force-pushed the installer_source_targets branch from add2143 to 90ca70d Compare December 4, 2023 22:08
@mistydemeo mistydemeo merged commit 7687ae4 into main Dec 4, 2023
14 checks passed
@mistydemeo mistydemeo deleted the installer_source_targets branch December 4, 2023 22:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Installer.sh sourcing ~/.cargo/bin in ~/.profile is not always sufficient
2 participants