Opened 3 days ago

Closed 40 hours ago

#5870 closed defect (fixed)

liblwgeom compile failure on PG17 and PG18 GHA runs

Reported by: robe Owned by: lbartoletti
Priority: blocker Milestone: PostGIS SFCGAL
Component: postgis Version: master
Keywords: Cc:

Description

https://github.com/postgis/postgis/actions/runs/14012537588/job/39234117154

 libtool: link: ( cd ".libs" && rm -f "libryu.la" && ln -s "../libryu.la" "libryu.la" )
make[2]: Leaving directory '/src/postgis/deps/ryu'
libtool: compile:  gcc -I./../deps/ryu/.. -I/usr/local/include -I/usr/local/include -I/usr/include/x86_64-linux-gnu -I/usr/include/libxml2 -I/usr/local/include -I/usr/include/json-c -DNDEBUG -I. -I. -std=gnu11 -g -O0 --coverage -fno-math-errno -fno-signed-zeros -Wall -g -fPIC -DPIC -I/usr/local/include -c lwgeodetic.c -o lwgeodetic.o >/dev/null 2>&1
libtool: compile:  gcc -I./../deps/ryu/.. -I/usr/local/include -I/usr/local/include -I/usr/include/x86_64-linux-gnu -I/usr/include/libxml2 -I/usr/local/include -I/usr/include/json-c -DNDEBUG -I. -I. -std=gnu11 -g -O0 --coverage -fno-math-errno -fno-signed-zeros -Wall -g -fPIC -DPIC -I/usr/local/include -c topo/lwgeom_topo.c -o topo/lwgeom_topo.o >/dev/null 2>&1
make[1]: Leaving directory '/src/postgis/liblwgeom'
make: *** [GNUmakefile:36: all] Error 1

Change History (6)

comment:2 by robe, 3 days ago

Okay I missed the offending part

math-errno -fno-signed-zeros -Wall -g -fPIC -DPIC -I/usr/local/include -c lwpoly.c -o lwpoly.o >/dev/null 2>&1
In file included from lwgeom_sfcgal.h:26,
                 from lwgeom_sfcgal.c:25:
/usr/local/include/SFCGAL/capi/sfcgal_c.h:1287:69: error: expected ';', ',' or ')' before '=' token
 1287 |                                  double                   tolerance = 0.0);
      |                                                                     ^

comment:3 by robe, 3 days ago

Milestone: PostGIS 3.6.0PostGIS SFCGAL

Okay this appears to be probably because of upgrade in sfcgal and so makes sense only the cis following sfcgal master failed.

comment:4 by robe, 3 days ago

Owner: changed from pramsey to lbartoletti
Note: See TracTickets for help on using tickets.