Opened 12 years ago

Closed 2 years ago

#1621 closed task (fixed)

Get rid of upgrade intermediary files

Reported by: robe Owned by: strk
Priority: low Milestone: PostGIS Fund Me
Component: build Version: master
Keywords: Cc:

Description

right now files like rtpostgis_upgrade.sql , topology_drop_before.sql topology_upgrade.sql topology_drop_after.sql are created as intermediaries for creating the final upgrade_minor files.

These should be deleted after use and never be installed. I think some of these get installed.

Change History (8)

comment:1 by strk, 12 years ago

It doesnt' look like we're installign them. Here's what gets installed:

legacy_compatibility_layer.sql  rtpostgis_upgrade_cleanup.sql
legacy.sql                      spatial_ref_sys.sql
postgis.sql                     topology.sql
postgis_upgrade_20_minor.sql    topology_upgrade_20_minor.sql
raster_comments.sql             uninstall_legacy.sql
rtpostgis_drop.sql              uninstall_postgis.sql
rtpostgis_legacy.sql            uninstall_rtpostgis.sql
rtpostgis.sql                   uninstall_topology.sql
rtpostgis_upgrade_20_minor.sql

Are them all meant to be installed ?

comment:2 by robe, 12 years ago

These should not get installed since they are intermediaries

rtpostgis_upgrade_cleanup.sql
rtpostgis_drop.sql

The others look fine.

comment:3 by strk, 12 years ago

Milestone: PostGIS 2.0.0PostGIS 2.1.0
Priority: mediumlow
Type: defecttask

r9349 fixes the bogus install. Intermediary files cleanup inside the build tree is really not an issue. Just an annoyance for them not to be easily recognizable as temps ( a rename could do ). In any case nothing left to do for 2.0.

comment:4 by robe, 11 years ago

Milestone: PostGIS 2.1.0PostGIS 2.0.4

comment:5 by robe, 11 years ago

Milestone: PostGIS 2.0.4PostGIS 2.2.0

comment:6 by strk, 9 years ago

Milestone: PostGIS 2.2.0PostGIS Future

comment:7 by robe, 7 years ago

Milestone: PostGIS FuturePostGIS Fund Me

Milestone renamed

comment:8 by strk, 2 years ago

Resolution: fixed
Status: newclosed

This looks to be fixed. Also the annoyance part.

Note: See TracTickets for help on using tickets.