Re: remove ancient pre-dlopen dynloader code - Mailing list pgsql-hackers

From Thomas Munro
Subject Re: remove ancient pre-dlopen dynloader code
Date
Msg-id CAEepm=2+CF3PshNRAs-r8jtPLKj0K6UEACeRSqBi5Cf74L=w7Q@mail.gmail.com
Whole thread Raw
In response to Re: remove ancient pre-dlopen dynloader code  (Peter Eisentraut <peter.eisentraut@2ndquadrant.com>)
List pgsql-hackers
On Thu, Sep 6, 2018 at 1:16 AM Peter Eisentraut
<peter.eisentraut@2ndquadrant.com> wrote:
> I'm going to use this thread for a moment to work out some details with
> the cfbot.
>
> The v2 patch I sent previously was created using git format-patch with
> default settings.  This detected a rename:
>
>  rename src/{backend/port/dynloader/win32.c => port/dlopen.c} (51%)
>
> which is fair enough.  However, whatever method the cfbot uses to apply
> patches fails to handle that.

Interesting.  Its version of "patch" doesn't understand that.  I am
not sure if other versions of patch do.  Currently cfbot doesn't use
git am because not everyone is posting patches created with
format-patch, and I thought good old patch could handle basically
anything.  I wasn't aware of this quirk.   I'll see if there is some
way I can convince patch to respect renames, or I should try to apply
with git first and then fall back to patch only if that fails.

-- 
Thomas Munro
http://www.enterprisedb.com


pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: remove ancient pre-dlopen dynloader code
Next
From: Ildus Kurbangaliev
Date:
Subject: Re: [HACKERS] Custom compression methods