Opened 5 years ago

Closed 5 years ago

#4313 closed defect (fixed)

PostgreSQL commits a9c35cf85ca and fa2cf164aaf break build

Reported by: Laurenz Albe Owned by: strk
Priority: medium Milestone: PostGIS 2.5.2
Component: build Version: master
Keywords: Cc:

Description

Commit fa2cf164aaf91e074be653c28e035f65d80eb666 renamed nodes/relation.h to nodes/pathnodes.h.

This affects postgis/gserialized_estimate.c and is easily fixed by the attached patch.

Commit a9c35cf85ca1ff72f16f0f10d7ddee6e582b62b8 is a more difficult affair.

It removed FunctionCallInfoData and replaced it with a variable-length array that is allocated with the LOCAL_FCINFO macro. The way to access the function arguments has also changed.

My attached patch is an attempt to fix all places (which are unfortunately many). This certainly makes the code harder to read.

There are some files where my changes are pretty invasive, mostly raster/rt_pg/rtpg_mapalgebra.c. This would require an extra pair of eyes and some testing.

Of course it would be nice to reduce the number of sites in the code where direct function calls are made.

Attachments (1)

postgresql-v12-build.patch (21.6 KB ) - added by Laurenz Albe 5 years ago.
An attempt to fix the build

Download all attachments as: .zip

Change History (5)

by Laurenz Albe, 5 years ago

Attachment: postgresql-v12-build.patch added

An attempt to fix the build

comment:1 by Algunenano, 5 years ago

I've created https://github.com/postgis/postgis/pull/368 with the patch to pass it through the CI.

The patch seems straightforward except for the raster part. I'll have a look to make sure I don't see anything wrong but I won't enter in detail to reduce the direct usage of function calls there (mainly because I don't know anything about raster).

comment:2 by Raul Marin, 5 years ago

In 17223:

PostgreSQL 12 compatibility

Initial patch by Laurenz Albe
References #4313

comment:3 by Raul Marin, 5 years ago

In 17224:

PostgreSQL 12 compatibility

Initial patch by Laurenz Albe
References #4313

comment:4 by Raul Marin, 5 years ago

Resolution: fixed
Status: newclosed

In 17225:

PostgreSQL 12 compatibility

Initial patch by Laurenz Albe
Closes #4313
Closes https://github.com/postgis/postgis/pull/368/

Note: See TracTickets for help on using tickets.