Opened 6 years ago
Closed 6 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)
Change History (5)
by , 6 years ago
Attachment: | postgresql-v12-build.patch added |
---|
comment:1 by , 6 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).
An attempt to fix the build