]> granicus.if.org Git - postgresql/commit
Convert contrib/hstore_plpython to not use direct linking to other modules.
authorTom Lane <tgl@sss.pgh.pa.us>
Tue, 4 Oct 2016 02:27:11 +0000 (22:27 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Tue, 4 Oct 2016 02:27:11 +0000 (22:27 -0400)
commitd51924be886c2a05e691fa05b16cb6b30ab8370f
tree3d8d29335b0e9e35145eae3c9cec164a015e1e3a
parent6bc811c992a804bdb8d228ce0be9f0a8e7198df6
Convert contrib/hstore_plpython to not use direct linking to other modules.

Previously, on most platforms, we allowed hstore_plpython's references
to hstore and plpython to be unresolved symbols at link time, trusting
the dynamic linker to resolve them when the module is loaded.  This
has a number of problems, the worst being that the dynamic linker
does not know where the references come from and can do nothing but
fail if those other modules haven't been loaded.  We've more or less
gotten away with that for the limited use-case of datatype transform
modules, but even there, it requires some awkward hacks, most recently
commit 83c249200.

Instead, let's not treat these references as linker-resolvable at all,
but use function pointers that are manually filled in by the module's
_PG_init function.  There are few enough contact points that this
doesn't seem unmaintainable, at least for these use-cases.  (Note that
the same technique wouldn't work at all for decoupling from libpython
itself, but fortunately that's just a standard shared library and can
be linked to normally.)

This is an initial patch that just converts hstore_plpython.  If the
buildfarm doesn't find any fatal problems, I'll work on the other
transform modules soon.

Tom Lane, per an idea of Andres Freund's.

Discussion: <2652.1475512158@sss.pgh.pa.us>
contrib/hstore_plpython/Makefile
contrib/hstore_plpython/hstore_plpython.c
contrib/hstore_plpython/hstore_plpython2u--1.0.sql
contrib/hstore_plpython/hstore_plpython3u--1.0.sql
contrib/hstore_plpython/hstore_plpythonu--1.0.sql
src/tools/msvc/Mkvcbuild.pm