Opened 7 years ago

Closed 7 years ago

#3921 closed defect (invalid)

ALTER EXTENSION upgrade failing at ST_CurveToLine

Reported by: robe Owned by: robe
Priority: blocker Milestone: PostGIS 2.5.0
Component: QA/buildbots Version: master
Keywords: Cc:

Description

After I resolved the make install issue at #3916

this issue was uncovered on winnie

Preparing db 'postgis_reg' using: CREATE EXTENSION postgis
Upgrading PostGIS in 'postgis_reg' using: ALTER EXTENSION postgis UPDATE TO '2.5.0devnext'
Died at run_test.pl line 1423.
 failed (Error encountered altering EXTENSION POSTGIS: /projects/postgis/tmp/2.5.0dev_pg10_geos3.7.0dev_gdal2.2.2w64/regress_log)
-----------------------------------------------------------------------------
sh: createlang: command not found
CREATE EXTENSION
CREATE EXTENSION
ERROR:  cannot remove parameter defaults from existing function
HINT:  Use DROP FUNCTION st_curvetoline(geometry,double precision,integer,integer) first.

Change History (9)

comment:1 by robe, 7 years ago

Component: build/upgrade/installbuildbots
Owner: changed from strk to robe

Might be a winnie specific thing since I'm not getting same failure. Though I may not be running same upgrade tests.

comment:2 by robe, 7 years ago

Ahah I see r16045 is the culprit. I think it's an accident that winnie was picking up on this though (though it is probably something that will be an issue upgrading from 2.4.* to 2.5, because you can't change default args without dropping old first)

comment:3 by robe, 7 years ago

Component: buildbotsbuild/upgrade/install

comment:4 by robe, 7 years ago

Component: build/upgrade/installbuildbots

I think this is my bad. The postgis.sql scripts weren't being overwritten so copy issue since winnie copies her install scripts and I missed the postgis.sql one which is used when no version is specified I think.

Version 0, edited 7 years ago by robe (next)

comment:5 by robe, 7 years ago

In 16074:

revise winnie's regess and package script copying to edb install to copy over non-versioned .sql file as well.
references #3921

comment:6 by robe, 7 years ago

In 16075:

revise winnie's regess and package script copying to edb install to copy over non-versioned .sql file as well.
Actually unversioned .sql aren't used at all, it was the next.sql scripts that weren't being overridden
references #3921

comment:7 by strk, 7 years ago

Why are those scripts not just using make install ?

comment:8 by robe, 7 years ago

It actually does use make install, but winnie has to test against another PostgreSQL instance (the VC++ build) so the install location is no good since it's the MingW PostgreSQL install location and since install location can't be controlled with --prefix, it's not really useful.

Also the installed items along with some other binaries (e.g. GDAL etc) need to be packaged in a zip file for windows users to consume.

comment:9 by robe, 7 years ago

Resolution: invalid
Status: newclosed
Note: See TracTickets for help on using tickets.