[postgis-tickets] r14632 - Further simplify extension wrapping

Sandro Santilli strk at keybit.net
Tue Feb 9 00:35:03 PST 2016


Author: strk
Date: 2016-02-09 00:35:02 -0800 (Tue, 09 Feb 2016)
New Revision: 14632

Modified:
   trunk/extensions/postgis/Makefile.in
Log:
Further simplify extension wrapping

See #3450

Modified: trunk/extensions/postgis/Makefile.in
===================================================================
--- trunk/extensions/postgis/Makefile.in	2016-02-09 08:28:56 UTC (rev 14631)
+++ trunk/extensions/postgis/Makefile.in	2016-02-09 08:35:02 UTC (rev 14632)
@@ -73,24 +73,14 @@
 sql_bits/spatial_ref_sys_config_dump.sql: ../../spatial_ref_sys.sql ../../utils/create_spatial_ref_sys_config_dump.pl
 	: ../../utils/create_spatial_ref_sys_config_dump.pl $< > $@
 
-# we need to also drop this temporary function from the extension
-# for casts that are being dropped we need to drop them 
-# from extension only if they are in the existension so we use our postgis_extension_drop..
-# so that it will silently fail if cast is not in extension
 sql_bits/rtpostgis_upgrade.sql: ../../raster/rt_pg/rtpostgis_upgrade.sql
 	sed -e 's/BEGIN;//g' -e 's/COMMIT;//g' \
-		-e 's/DROP FUNCTION _rename_raster_tables();/ALTER EXTENSION ${EXTENSION} DROP FUNCTION _rename_raster_tables();DROP FUNCTION _rename_raster_tables();/g' \
-		-e 's/DROP FUNCTION _drop_st_samealignment();/ALTER EXTENSION ${EXTENSION} DROP FUNCTION _drop_st_samealignment();DROP FUNCTION _drop_st_samealignment();/g' \
-		-e 's/DROP CAST\(.*\)/SELECT postgis_extension_drop_if_exists('\''$(EXTENSION)'\'', '\''DROP CAST \1'\'');DROP CAST \1/' \
+		  -e "s/^\(DROP .*\);/SELECT postgis_extension_drop_if_exists('${EXTENSION}', '\1');\n\1;/" \
 	$< > $@
 
-#don't drop casts just yet since we don't have provision to remove from extension yet
-#TODO: check the above statement, as I do see DROP CAST handled for raster instead
 sql_bits/postgis_upgrade.sql: ../../postgis/postgis_upgrade.sql 
 		 sed -e 's/BEGIN;//g' -e 's/COMMIT;//g' \
-		 -e '/^\(DROP\|CREATE\) \(CAST\).*;/d' \
-		 -e '/^\(DROP\|CREATE\) \(CAST\)/,/\;/d' \
-		 -e "s/^\(DROP .*\);/SELECT postgis_extension_drop_if_exists('${EXTENSION}', '\1');\1;/" \
+		     -e "s/^\(DROP .*\);/SELECT postgis_extension_drop_if_exists('${EXTENSION}', '\1');\n\1;/" \
 	 	 $< > $@
 
 



More information about the postgis-tickets mailing list