Horton

Thread

Commits

  1. Rename JsonIsPredicate.value_type, fix JSON backend/nodes/ infrastructure.

  1. typedefs.list glitches

    Tom Lane <tgl@sss.pgh.pa.us> — 2022-05-12T20:00:13Z

    I just completed the v15 pre-beta pgindent run.  It went reasonably
smoothly, but I had to hack up typedefs.list a little bit compared
to the version downloaded from the buildfarm.

* The buildfarm's list is missing
    pg_md5_ctx
    pg_sha1_ctx
    pg_sha224_ctx
    pg_sha256_ctx
    pg_sha384_ctx
    pg_sha512_ctx
which are certainly used, but only in some src/common files
that are built only in non-OpenSSL builds.  So evidently,
every buildfarm member that's contributing to the typedefs list
builds with OpenSSL.  That wouldn't surprise me, except that
my own animal sifaka should be filling that gap.  Looking at
its latest attempt[1], it seems to be generating an empty list,
which I guess means that our recipe for extracting typedefs
doesn't work on macOS/arm64.  I shall investigate.

* The buildfarm's list includes "value_type", which is surely
not typedef'd anywhere in our code, and that is messing up
some formatting involving JsonIsPredicate.value_type.
I suppose that is coming from some system header where it is
a typedef on some machines (komodoensis and lorikeet report it,
which seems like an odd pairing).  I think the best thing to
do here is rename that field while we still can, perhaps to
item_type.  Thoughts?

			regards, tom lane

[1] https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=sifaka&dt=2022-05-11%2020%3A21%3A15

  2. Re: typedefs.list glitches

    Tom Lane <tgl@sss.pgh.pa.us> — 2022-05-12T21:21:43Z

    I wrote:
> every buildfarm member that's contributing to the typedefs list
> builds with OpenSSL.  That wouldn't surprise me, except that
> my own animal sifaka should be filling that gap.  Looking at
> its latest attempt[1], it seems to be generating an empty list,
> which I guess means that our recipe for extracting typedefs
> doesn't work on macOS/arm64.  I shall investigate.

Found it.  Current macOS produces

$ objdump -W
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/objdump: error: unknown argument '-W'

where last year's vintage produced

$ objdump -W
objdump: Unknown command line argument '-W'.  Try: '/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/objdump --help'
objdump: Did you mean '-C'?

This confuses run_build.pl into taking the "Linux and sometimes windows"
code path instead of the $using_osx one.  I think simplest fix is to
move the $using_osx branch ahead of the heuristic ones, as attached.

			regards, tom lane

  3. Re: typedefs.list glitches

    Tristan Partin <tristan@neon.tech> — 2023-12-12T15:37:29Z

    On Thu May 12, 2022 at 4:21 PM CDT, Tom Lane wrote:
> I wrote:
> > every buildfarm member that's contributing to the typedefs list
> > builds with OpenSSL.  That wouldn't surprise me, except that
> > my own animal sifaka should be filling that gap.  Looking at
> > its latest attempt[1], it seems to be generating an empty list,
> > which I guess means that our recipe for extracting typedefs
> > doesn't work on macOS/arm64.  I shall investigate.
>
> Found it.  Current macOS produces
>
> $ objdump -W
> /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/objdump: error: unknown argument '-W'
>
> where last year's vintage produced
>
> $ objdump -W
> objdump: Unknown command line argument '-W'.  Try: '/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/objdump --help'
> objdump: Did you mean '-C'?
>
> This confuses run_build.pl into taking the "Linux and sometimes windows"
> code path instead of the $using_osx one.  I think simplest fix is to
> move the $using_osx branch ahead of the heuristic ones, as attached.

Hey Tom,

Was this patch ever committed?

-- 
Tristan Partin
Neon (https://neon.tech)

  4. Re: typedefs.list glitches

    Tom Lane <tgl@sss.pgh.pa.us> — 2023-12-12T15:48:33Z

    "Tristan Partin" <tristan@neon.tech> writes:
> Was this patch ever committed?

Yes, though not till

commit dcca861554e90d6395c3c153317b0b0e3841f103
Author: Andrew Dunstan <andrew@dunslane.net>
Date:   Sun Jan 15 07:32:50 2023 -0500

    Improve typedef logic for MacOS

sifaka is currently generating typedefs, and I'm pretty certain
it's using unpatched REL_17 BF code.

			regards, tom lane