From: Justin Pryzby <pryzby@telsasoft.com>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Andres Freund <andres@anarazel.de>
Cc: Laurenz Albe <laurenz.albe@cybertec.at>
Cc: pgsql-hackers@postgresql.org
Subject: Re: Use -fvisibility=hidden for shared libraries
Date: Mon, 10 Jan 2022 21:11:07 -0600
Message-ID: <20220111031107.GU14051@telsasoft.com> (raw)
In-Reply-To: <2499085.1641859296@sss.pgh.pa.us>
References: <20211101020311.av6hphdl6xbjbuif@alap3.anarazel.de>
<b8103d16bf26cff7a393d1de690474dbd9afaa19.camel@cybertec.at>
<20211101054700.z3jzmc5lnxlad4y7@alap3.anarazel.de>
<2499085.1641859296@sss.pgh.pa.us>
On Mon, Jan 10, 2022 at 07:01:36PM -0500, Tom Lane wrote:
> Andres Freund <andres@anarazel.de> writes:
> > [ v2-0001-Use-hidden-visibility-for-shared-libraries-where-.patch ]
>
> This seems like a good idea, but it's failing to apply right now,
> mainly because of some cleanup I did in e04a8059a. As far as I can
> tell given the now-clarified meanings of PGDLLIMPORT/PGDLLEXPORT,
> this patch shouldn't be touching PGDLLIMPORT. The attached revision
> works for me under gcc 8.5 and clang 13.
>
> Also, it seemed like you'd maybe been more enthusiastic than necessary
> about plastering PGDLLEXPORT on things. I got through check-world
> cleanly without the diffs in either ltree.h or worker_spi.c (although
> I confess being confused why I didn't need the latter). I didn't try
> individually removing other diffs. Those diffs are still in v3 below,
> but we should clarify exactly which functions need marking.
Without the patch, it fails under windows like:
https://cirrus-ci.com/task/6171477065072640
[01:44:45.399] c:\cirrus\contrib\ltree\ltree.h(193): message : see declaration of '_ltree_isparent' [c:\cirrus\ltree.vcxproj]
[01:44:45.399] c:\cirrus\contrib\ltree\_ltree_op.c(16,1): error C2375: '_ltree_risparent': redefinition; different linkage [c:\cirrus\ltree.vcxproj]
--
Justin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-hackers@postgresql.org
Cc: pryzby@telsasoft.com, tgl@sss.pgh.pa.us, andres@anarazel.de, laurenz.albe@cybertec.at
Subject: Re: Use -fvisibility=hidden for shared libraries
In-Reply-To: <20220111031107.GU14051@telsasoft.com>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox