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.

Sign up for GitHub

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

Jump to bottom

[shell] Override CDPATH in brew script. #16179

Merged
merged 1 commit into from
Nov 5, 2023
Merged

[shell] Override CDPATH in brew script. #16179

merged 1 commit into from
Nov 5, 2023

Commits on Nov 4, 2023

  1. [shell] Override CDPATH in brew script. 

    The CDPATH environment variable can affect the behavior of
`cd`, and `cd` takes the `-P`, `-L` and `-e` flags.
(I didn't know about `-e` until looking at the source)
Make quietcd more robust by setting the CDPATH to ''.

I tested this change by running the following commands from a nonstandard
directory, which caused some of tcsh's and perl's dependencies to be recompiled.

$ [path to brew] install tcsh
$ [path to brew] install perl

Here's a link to the source code of `cd` in a mirror of the bash repo.

https://github.com/bminor/bash/blob/ec8113b9861375e4e17b3307372569d429dec814/builtins/cd.def#L267
    @gregorynisbet-google
    gregorynisbet-google committed Nov 4, 2023
    Configuration menu
    Copy the full SHA
    077eee8 View commit details
    Browse the repository at this point in the history