Re: [PATCH v2 1/3] Push the NATIVE_CRLF Makefile variable to C and added a test for native.

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

 



Torsten Bögershausen <tboegi@xxxxxx> writes:

> On 2014-08-23 00.54, Junio C Hamano wrote:
>> Torsten Bögershausen <tboegi@xxxxxx> writes:
>> 
>>> Commit 95f31e9a correctly points out that the NATIVE_CRLF setting is
>>> incorrectly set on Mingw git. However, the Makefile variable is not
>>> propagated to the C preprocessor and results in no change. This patch
>>> pushes the definition to the C code and adds a test to validate that
>>> when core.eol as native is crlf, we actually normalize text files to this
>>> line ending convention when core.autocrlf is false.
>>>
>>> Signed-off-by: Pat Thoyts <patthoyts@xxxxxxxxxxxxxxxxxxxxx>
>>> Signed-off-by: Stepan Kasal <kasal@xxxxxx>
>>> Signed-off-by: Torsten Bögershausen <tboegi@xxxxxx>
>>> ---
>> 
>> Who should I record as the author of this patch?
>> 
>
> Sorry for missing this, here is what Mingw says: 
>
> commit 0caba2cacbb9d8e6a31783b45f1a13e52dec6ce8
> Author: Pat Thoyts <patthoyts@xxxxxxxxxxxxxxxxxxxxx>
> Date:   Mon Nov 26 00:24:00 2012 +0000
>
>     Push the NATIVE_CRLF Makefile variable to C and added a test for native.
>     
>     Commit 95f31e9a correctly points out that the NATIVE_CRLF setting is
> []

When forwarding somebody else's patch, please start the *body* of
your message with the in-body header to force the author, followed
by a blank line and then the message, i.e.

 	From: Pat Thoyts <patthoyts@xxxxxxxxxxxxxxxxxxxxx>

	Commit 95f31e9a correctly points out that the NATIVE_CRLF setting is
	incorrectly set on Mingw git. However, the Makefile variable is not...
	...

The request applies to other patches in the series as well.  I
suspect that using send-email on format-patch output may do the
right thing automatically.

Thanks.
--
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]