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

Comment out CLICOLOR_FORCE=1 with warning #10

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

philCryoport
Copy link

CLICOLOR_FORCE=1 can cause issues where shell output is captured -- such as with opam install topkg, ocamlfind and anywhere else shell output is captured. Included warning about uncommenting this line

CLICOLOR_FORCE=1 can cause issues where shell output is captured -- such as with `opam install topkg`, `ocamlfind` and anywhere else shell output is captured.  Included warning about uncommenting this line
@philCryoport
Copy link
Author

philCryoport commented Mar 23, 2022

See ocaml/ocamlfind#40 and dbuenzli/topkg#119 (comment) about a day wasted trying to figure out why I couldn't opam install csvtool -- which relied on opam install topkg -- which relied on ocamlfind -- which captures output from ls.

I think I got the CLICOLOR_FORCE=1 line in my own .zshrc from here when I forked this repo in conjunction with https://github.com/geerlingguy/mac-dev-playbook

@philCryoport
Copy link
Author

@geerlingguy friendly follow-up...

@geerlingguy
Copy link
Owner

I'm now trying to figure out why I had added CLICOLOR_FORCE in the first place...

@philCryoport
Copy link
Author

I'm now trying to figure out why I had added CLICOLOR_FORCE in the first place...

Anything come to mind?

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.

2 participants