Re: [PATCH] Issue 2995: git-cly: Update to latest upload.py from google

[David Kastrup]

"Phil Holmes" <address@hidden> writes:

> ----- Original Message ----- 
> From: "David Kastrup" <address@hidden>
> To: "Phil Holmes" <address@hidden>
> Cc: <address@hidden>
> Sent: Sunday, December 02, 2012 1:43 PM
> Subject: Re: [PATCH] Issue 2995: git-cly: Update to latest upload.py
> from google
>
>
>> "Phil Holmes" <address@hidden> writes:
>>
>>> I'd suggest pushing this direct to github: do you have the permission
>>> to do this?
>>
>> No.
>>
>>> If you don't, could you attach a patch as an attachment and I'll push
>>> it?
>>
>> What is wrong with the mail called [PATCH]?  You can just save it and
>> apply it using git-am.
>
> I'm concerned that my mail client might wrap some lines which would
> lead to incorrect code and poisoning the repo.

If you save the whole message (rather than doing some copy&paste job on
the mail client window) that should not be an issue in the first place.

And even if it were, the result of wrapped lines would be an invalid
patch that git would not apply.

-- 
David Kastrup