Add warning about using SHELLOPTS to set igncr#22
Add warning about using SHELLOPTS to set igncr#22jon-turney merged 1 commit intocygwin:masterfrom tobil4sk:readme-SHELLOPTS-warning
Conversation
|
Thanks. It sounds like really we should only be recommending using '-o igncr', since having SHELLOPTS in the environment causes this propagation, so I might do a bit of tweaking on the wording... |
|
Thanks!
Yes, this sounds like the best idea to me |
|
FYI: I use |
|
Well, ideally, GitHub would give us control of the line endings written into the file whose name is substituted for If I recall correctly, I was musing if the brief usage at the top of the README needs an example to demonstrate how to write it with a multiline YAML literal block like that. |
By default,
SHELLOPTSis a shell variable and moving it to the environment causes all shell options to propagate to child shells. This may break scripts in a confusing way, so it is not a good idea to recommend it without at least providing a warning.Perhaps cygwin-install-action could provide an option to set up
BASH_ENVwith a file containingset -o igncr. setup-ocaml does this when configuring cygwin.See: