Re: git imap-send converting my patches to CRLF line endings?

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

 



On 06/17/2011 09:14 AM, Jeff King wrote:
> On Fri, Jun 17, 2011 at 03:35:04PM +0200, Michael Mc Donnell wrote:
> 
>> I'm using git imap-send to send patches to wine-patches, and it seems
>> like it converts all my patches to have CRLF line endings?
> 
> The canonical line ending for mail is CRLF. So yes, it will convert your
> patch to CRLF for storage. But anything pulling it out of the IMAP
> folder should convert it back to native line endings.

Not always.  Modern thunderbird (3.1.10, is that modern? I haven't
checked), saves mail using CRLF.  I don't have access to gmail at the
moment, but I'm pretty sure gmail does the same thing, i.e. when you
select "view original", and then use your browser to "save as...".

mailsplit was modified to strip CRLF when splitting mail here:

  c2ca1d7 Allow mailsplit (and hence git-am) to handle mails with CRLF line-endings

which should have first appeared in git v1.6.5.

>> I can see it when I download the patch from the Gmail drafts folder.
>> Git complains about white space when I apply the downloaded patch. It
>> works fine if I just use git to create the patch and then apply it on
>> a new branch. Is it git imap-send or just Gmail that's the problem?
> 
> How do you download and apply the patch exactly? If you are speaking
> imap to gmail, generally the client would strip out the CR's from the
> mail.

Michael, how are you applying the "email"?  Are you using 'git am'? or
possibly are you trying to use 'git apply'?  You need to use 'git am'.

Also, as I mentioned above, you should be using git more recent than
v1.6.5 so that you have a mailsplit that will strip out the CRLF line
endings.

-Brandon
--
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]