Re: CVS import [SOLVED]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, February 16, 2009 14:20, Ferry Huberts (Pelagic) wrote:
> I solved it:
>
> it has to do with the
> core.autocrlf=input
> core.safecrlf=true
>
> settings I had in my global config.
> Maybe the manual page should warn against having these defined?
>

I'm working on it now, and did some more testing: it's actually the safecrlf setting, not the autocrlf option.
Which leaves me with some questions of what to do exactly:

 autocrlf safecrlf
1 false    false
2 false     warn
3 false     true
4 input    false
5 input     warn
6 input     true
7 true     false
8 true      warn
9 true      true


1- git ignores the safecrlf flag; obviously acceptable
2- git ignores the safecrlf flag; so acceptable
3- git ignores the safecrlf flag, so acceptable
4- seems acceptable to me
5- seems acceptable to me
6- unacceptable
7- seems acceptable to me
8- seems acceptable to me
9- unacceptable

So, 6 and 9 (safecrlf==true) are definitely unacceptable. 1-3 are definitely acceptable.
How about the others?
Should these produce warnings?
Should the user use a 'force' option to make the import work (and acknowledge that he's calling for trouble)
Should we enforce some setting? Which flags/setting?

Input appreciated :-)

Ferry
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux