Opened 7 months ago

Closed 6 months ago

#5652 closed defect (fixed)

Upgrade raster tests running even if postgis raster is disabled

Reported by: robe Owned by: strk
Priority: blocker Milestone: PostGIS 3.5.0
Component: upgrade Version: master
Keywords: Cc:

Description

Cirrus builds are failing on master

https://cirrus-ci.com/task/4631711644909568

 --------------- Extensions --------------- 
  PostgreSQL EXTENSION support:       enabled
  PostGIS Raster:                     disabled
  PostGIS Topology:                   enabled
  SFCGAL support:                     enabled
  Address Standardizer support:       enabled
/usr/local/bin/perl -lpe "s'MODULE_PATHNAME'\$libdir/postgis_sfcgal-3'g;s'@extschema@\.''g" > sfcgal_after_upgrade.sql
rm -f sfcgal_after_upgrade.sql.tmp
echo "BEGIN;" > sfcgal_upgrade.sql
cat sfcgal_before_upgrade.sql sfcgal_upgrade.sql.in sfcgal_after_upgrade.sql >> sfcgal_upgrade.sql
echo "COMMIT;" >> sfcgal_upgrade.sql
gmake[1]: Leaving directory '/tmp/cirrus-ci-build/sfcgal'
---- Making all in utils
gmake[1]: Entering directory '/tmp/cirrus-ci-build/utils'
sed -e '1,/^__END__/d' postgis_restore.pl.in \
	| sort -u \
	> postgis_restore_data.hardcoded
gmake[1]: *** No rule to make target '../raster/rt_pg/rtpostgis_upgrade.sql', needed by 'postgis_restore_data.generated'.  Stop.
gmake[1]: Leaving directory '/tmp/cirrus-ci-build/utils'
gmake: *** [GNUmakefile:27: all] Error 1

I think it might have something to do with the fact that they don't have raster enabled, so they shouldn't even be trying to build raster extension or upgrades scripts.

Change History (3)

comment:1 by robe, 7 months ago

The commit where this started was [907a7bb4b/git]

comment:2 by robe, 6 months ago

See #5656 a real user being inconvenienced by this bug.

comment:3 by Regina Obe <lr@…>, 6 months ago

Resolution: fixed
Status: newclosed

In 8619e4d/git:

Make sure rtpostgis_ugprade.sql gets built
during upgrade tests
Even if raster is disabled
Closes #5652

Note: See TracTickets for help on using tickets.