From: | Noah Misch <noah(at)leadboat(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>, Michael Paquier <michael(at)paquier(dot)xyz>, Jacob Champion <pchampion(at)vmware(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>, Daniel Gustafsson <daniel(at)yesql(dot)se>, info(at)cspug(dot)cz |
Subject: | Re: Preventing abort() and exit() calls in libpq |
Date: | 2021-07-01 01:23:28 |
Message-ID: | 20210701012328.GA2288851@rfd.leadboat.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Wed, Jun 30, 2021 at 12:06:47PM -0400, Tom Lane wrote:
> I wrote:
> > OK, thanks, will push a fix momentarily.
>
> Did so, and look what popped up on wrasse [1]:
>
> ! nm -A -g -u libpq.so.5.15 2>/dev/null | grep -v __cxa_atexit | grep exit
> libpq.so.5.15: [765] | 232544| 248|FUNC |GLOB |3 |14 |PQexitPipelineMode
>
> This makes no sense, because (a) wrasse was happy with the previous
> version, and (b) surely the "-u" switch should prevent nm from
> printing PQexitPipelineMode. Noah, did you change anything about
> wrasse's configuration today?
No, and wrasse still succeeds at "git checkout e45b0df^". Solaris
/usr/bin/grep doesn't support "-e":
[nm(at)gcc-solaris11 5:0 2021-06-30T22:23:29 postgresql 0]$ echo exit | grep -e exit
grep: illegal option -- e
Usage: grep [-c|-l|-q] -bhinsvw pattern file . . .
[nm(at)gcc-solaris11 5:0 2021-06-30T22:23:43 postgresql 2]$ echo exit | grep exit
exit
[nm(at)gcc-solaris11 5:0 2021-06-30T22:24:16 postgresql 0]$ echo exit | /usr/xpg4/bin/grep -e exit
exit
That concealed things in the previous version. You can see those "illegal
option" messages in the last passing run:
https://buildfarm.postgresql.org/cgi-bin/show_stage_log.pl?nm=wrasse&dt=2021-06-30%2008%3A57%3A27&stg=make
From | Date | Subject | |
---|---|---|---|
Next Message | tanghy.fnst@fujitsu.com | 2021-07-01 01:25:23 | RE: Added schema level support for publication. |
Previous Message | David Rowley | 2021-07-01 01:16:52 | Re: Use simplehash.h instead of dynahash in SMgr |