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

Cygwin support (aka. support systems with core.autocrlf=true set globally) #2

Open
DrVanScott opened this issue Mar 21, 2018 · 1 comment

Comments

@DrVanScott
Copy link

Hi,

On windows systems core.autocrlf is normally set to "true" globally. Unfortunately this will render git projects useless which contain bash code, as git is creating those files with CRLF which bash doesn't like. I see four possible solutions:

  • import.bash should do a "git -c core.autocrlf=input clone ..." and finally set core.autocrlf=input locally
  • import.bash should make use of "set -o igncr" (warning: cygwin bash only!) (Not sure if this option really works...)
  • reject this request and force me to use "set -o igncr", e.g. in my .bash_profile. I admitt that this solutions seems to be clean (and easy from your perspective ;-) ), but i am not sure about the consequences of setting igncr globally (Are there shell scripts where CRLF SHOULD matter?)
  • add .gitattributes file to all of your *.bash projects and force git to handle shell scripts eol=lf

In my opinion the first or the last option are preferable.

@kovetskiy
Copy link
Member

@DrVanScott Hello. I'm glad that you found this project useful and you want to use it on Windows, but we didn't have plans for Cygwin support since we don't use windows at all, if you will send a pull-request with all required changeset, then I will be glad to merge it.

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

No branches or pull requests

2 participants