Re: Context diffs - Mailing list pgsql-hackers

From Andrew Dunstan
Subject Re: Context diffs
Date
Msg-id b288905f-d6f1-4dc0-6be4-17a74088116f@dunslane.net
Whole thread Raw
In response to Re: Context diffs  (Thomas Munro <thomas.munro@gmail.com>)
List pgsql-hackers
On 1/4/21 5:21 PM, Thomas Munro wrote:
> On Tue, Jan 5, 2021 at 8:07 AM Bruce Momjian <bruce@momjian.us> wrote:
>> *  "git apply" and "git am" can't process context diffs (they throw an
>>    error once a context-like section of the diff is hit; simple
>>    adding/removing lines in a block works)
>>
>> *  the commit-fest doesn't recognized context diff attachments as
>> patches:
>>
>>         https://commitfest.postgresql.org/31/2912/
>>
>> *  cfbot can't process file renames/add/delete from context diffs
> For the record, cfbot just uses plain old GNU patch, because that
> seems to accept nearly everything that anyone posts here (after a step
> that tries to unpack tarballs etc).  Several people have suggested I
> change it to use git apply instead (IIRC it works better for patches
> containing binary files such as cryptographic keys?), but then it
> wouldn't accept ye olde context diffs.
>
>

I would try 'git apply' and fall back on 'patch' if it fails.


cheers


andrew


--
Andrew Dunstan
EDB: https://www.enterprisedb.com




pgsql-hackers by date:

Previous
From: Amit Kapila
Date:
Subject: Re: Single transaction in the tablesync worker?
Next
From: Daniele Varrazzo
Date:
Subject: Types info on binary copy