Re: [PATCH] Print error when libpq-refs-stamp fails

From: Daniel Gustafsson <daniel(at)yesql(dot)se>
To: Jacob Champion <pchampion(at)vmware(dot)com>
Cc: Rachel Heaton <rachelmheaton(at)gmail(dot)com>, pgsql-hackers(at)lists(dot)postgresql(dot)org, a(dot)voloshin(at)postgrespro(dot)ru
Subject: Re: [PATCH] Print error when libpq-refs-stamp fails
Date: 2021-10-04 18:36:54
Message-ID: E355517C-AA5D-41F5-B4F2-A5534CFD1F48@yesql.se
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

> On 4 Oct 2021, at 19:21, Daniel Gustafsson <daniel(at)yesql(dot)se> wrote:
>
>> On 4 Oct 2021, at 19:02, Jacob Champion <pchampion(at)vmware(dot)com> wrote:
>>
>> On Mon, 2021-10-04 at 23:40 +0700, Anton Voloshin wrote:
>>>
>>> Could you please confirm that the change from -A to -a in nm arguments
>>> in this patch is intentional?
>>
>> That was not intended by us, thank you for the catch! A stray
>> lowercasing in vim, perhaps.
>
> Hmm, I will take care of this shortly.

Right, so I missed this in reviewing and testing, and I know why the latter
didn't catch it. nm -A and -a outputs the same thing *for this input* on my
Debian and macOS boxes, with the small difference that -A prefixes the line
with the name of the input file. -a also include debugger symbols, but for
this usage it didn't alter the results. I will go ahead and fix this, thanks
for catching it!

--
Daniel Gustafsson https://vmware.com/

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Daniel Gustafsson 2021-10-04 18:38:06 Re: 2021-09 Commitfest
Previous Message Peter Eisentraut 2021-10-04 18:36:32 Re: improvements in Unicode tables generation code