Re: Installation of regress.so?

Hi,

On 2021-05-20 09:16:50 -0400, Andrew Dunstan wrote:
> We certainly shouldn't want that.  But we do need it for the target
> unless we wipe out everything in the source that refers to it.

Is there a reason not to go for the wipe? I don't think the type of
functions we have in regress.so are necessarily ones we'd even expect to
work in the next version?

Here's references to explicit files I see after an installcheck:

SELECT oid::regproc, prosrc, probin FROM pg_proc WHERE probin IS NOT NULL AND probin NOT LIKE '$libdir%';
┌───────────────────────────┬───────────────────────────┬────────────────────────────────────────────────────────────────────────────┐
│            oid            │          prosrc           │                                   probin                                   │
├───────────────────────────┼───────────────────────────┼────────────────────────────────────────────────────────────────────────────┤
│ check_primary_key         │ check_primary_key         │ /home/andres/build/postgres/dev-optimize/vpath/src/test/regress/refint.so  │
│ check_foreign_key         │ check_foreign_key         │ /home/andres/build/postgres/dev-optimize/vpath/src/test/regress/refint.so  │
│ autoinc                   │ autoinc                   │ /home/andres/build/postgres/dev-optimize/vpath/src/test/regress/autoinc.so │
│ trigger_return_old        │ trigger_return_old        │ /home/andres/build/postgres/dev-optimize/vpath/src/test/regress/regress.so │
│ ttdummy                   │ ttdummy                   │ /home/andres/build/postgres/dev-optimize/vpath/src/test/regress/regress.so │
│ set_ttdummy               │ set_ttdummy               │ /home/andres/build/postgres/dev-optimize/vpath/src/test/regress/regress.so │
│ make_tuple_indirect       │ make_tuple_indirect       │ /home/andres/build/postgres/dev-optimize/vpath/src/test/regress/regress.so │
│ test_atomic_ops           │ test_atomic_ops           │ /home/andres/build/postgres/dev-optimize/vpath/src/test/regress/regress.so │
│ test_fdw_handler          │ test_fdw_handler          │ /home/andres/build/postgres/dev-optimize/vpath/src/test/regress/regress.so │
│ test_support_func         │ test_support_func         │ /home/andres/build/postgres/dev-optimize/vpath/src/test/regress/regress.so │
│ test_opclass_options_func │ test_opclass_options_func │ /home/andres/build/postgres/dev-optimize/vpath/src/test/regress/regress.so │
│ test_enc_conversion       │ test_enc_conversion       │ /home/andres/build/postgres/dev-optimize/vpath/src/test/regress/regress.so │
│ binary_coercible          │ binary_coercible          │ /home/andres/build/postgres/dev-optimize/vpath/src/test/regress/regress.so │
│ widget_in                 │ widget_in                 │ /home/andres/build/postgres/dev-optimize/vpath/src/test/regress/regress.so │
│ widget_out                │ widget_out                │ /home/andres/build/postgres/dev-optimize/vpath/src/test/regress/regress.so │
│ int44in                   │ int44in                   │ /home/andres/build/postgres/dev-optimize/vpath/src/test/regress/regress.so │
│ int44out                  │ int44out                  │ /home/andres/build/postgres/dev-optimize/vpath/src/test/regress/regress.so │
│ pt_in_widget              │ pt_in_widget              │ /home/andres/build/postgres/dev-optimize/vpath/src/test/regress/regress.so │
│ overpaid                  │ overpaid                  │ /home/andres/build/postgres/dev-optimize/vpath/src/test/regress/regress.so │
│ interpt_pp                │ interpt_pp                │ /home/andres/build/postgres/dev-optimize/vpath/src/test/regress/regress.so │
│ reverse_name              │ reverse_name              │ /home/andres/build/postgres/dev-optimize/vpath/src/test/regress/regress.so │
└───────────────────────────┴───────────────────────────┴────────────────────────────────────────────────────────────────────────────┘
(21 rows)

Testing the pg_upgrade path for these doesn't seem to add meaningful
coverage, and several seem likely to cause problems across versions?

Greetings,

Andres Freund