Re: Can we amend gitignore so git postgresql works with git on windows using Msys/Mingw64 - Mailing list pgsql-hackers

From Regina Obe
Subject Re: Can we amend gitignore so git postgresql works with git on windows using Msys/Mingw64
Date
Msg-id 000401d1878d$8b271d70$a1755850$@pcorp.us
Whole thread Raw
In response to Re: Can we amend gitignore so git postgresql works with git on windows using Msys/Mingw64  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
> You can change the setting with:
> git config --global core.autocrlf input

> Still, it's fair to wonder if we shouldn't add an entry for this to our
.gitattributes.  I'm actually wondering why we wouldn't apply it to ALL text
files in git, not just the extensions Regina mentioned.
        > regards, tom lane

David,

Thanks for the tip.  I'll give that a try.  Still I think it's safer to use
.gitattributes.

to Tom's comment,
Here is why.  For certain repos, certain files are designated for certain
OS.  So you don't want your developer committer's settings controlling the
output.

Take for example windows specific files.  I do have lines like

*.vcproj    eol=crlf
*.bat        eol=crlf


I don't actually think it much matters even for those if they have LF,
except for batch scripts it looks a bit wonky in some windows editors if
they don't have CRLF breaks.


I'm also not clear what Mac does.  Does Mac still do CR (instead of LF, I
thought it used to).  Does that mean if they switch their ending I'd
suddenly be getting Mac endings?

https://help.github.com/articles/dealing-with-line-endings/

As that above links says - "This file is committed into the repository and
overrides an individual's core.autocrlf setting, ensuring consistent
behavior for all users, regardless of their Git settings." 

I think consistency is more important in this case.


Thanks,
Regina







pgsql-hackers by date:

Previous
From: Piotr Stefaniak
Date:
Subject: Re: Two division by 0 errors in optimizer/plan/planner.c and optimizer/path/costsize.c
Next
From: Alvaro Herrera
Date:
Subject: Re: multivariate statistics v14