tag 687109 + patch
thanks

Attached, please review. If noone speaks up I'll NMU this or next week.

-- 
Marcin Owsiany <porri...@debian.org>             http://marcin.owsiany.pl/
GnuPG: 2048R/02F946FC  35E9 1344 9F77 5F43 13DD  6423 DBF4 80C6 02F9 46FC
Index: debian/control
===================================================================
--- debian/control	(wersja 2038)
+++ debian/control	(kopia robocza)
@@ -3,7 +3,7 @@
 Priority: optional
 Maintainer: Debian XML/SGML Group <debian-xml-sgml-p...@lists.alioth.debian.org>
 Uploaders: Ardo van Rangelrooij <a...@debian.org>, Daniel Leidert (dale) <daniel.leid...@wgdd.de>
-Build-Depends: debhelper (>= 5)
+Build-Depends: debhelper (>= 9.20120909)
 Build-Depends-Indep: perl
 Standards-Version: 3.8.3
 Vcs-Browser: http://svn.debian.org/wsvn/debian-xml-sgml/packages/xml-core/
@@ -13,7 +13,7 @@
 Package: xml-core
 Architecture: all
 Depends: ${perl:Depends}, ${misc:Depends}, sgml-base (>= 1.17), sed (>= 4.1.2-8)
-Suggests: debhelper (>= 4.1.75)
+Suggests: debhelper (>= 9.20120909)
 Description: XML infrastructure and XML catalog file support
  This package creates the XML infrastructure directories and provides
  XML catalog file support in compliance with the current Debian XML
Index: debian/changelog
===================================================================
--- debian/changelog	(wersja 2038)
+++ debian/changelog	(kopia robocza)
@@ -1,8 +1,10 @@
-xml-core (0.14) UNRELEASED; urgency=low
+xml-core (0.13+nmu1) unstable; urgency=low
 
-  * NOT RELEASED YET
+  * Non-maintainer upload.
+  * Use the "sub" fourth argument to autoscript that is more robust than the
+    sed snippet way (closes: #687109)
 
- -- Daniel Leidert (dale) <daniel.leid...@wgdd.de>  Sun, 25 Oct 2009 23:59:27 +0100
+ -- Marcin Owsiany <porri...@debian.org>  Mon, 10 Sep 2012 21:56:15 +0100
 
 xml-core (0.13) unstable; urgency=low
 
Index: debhelper/dh_installxmlcatalogs
===================================================================
--- debhelper/dh_installxmlcatalogs	(wersja 2038)
+++ debhelper/dh_installxmlcatalogs	(kopia robocza)
@@ -108,7 +108,13 @@
 
 ## ----------------------------------------------------------------------
 use Debian::Debhelper::Dh_Lib;
+use Debian::Debhelper::Dh_Version;
 
+$Debian::Debhelper::Dh_Version::version =~ /^(\d+)\.(\d+)/
+	or error("Unexpected debhelper version format");
+# For the "sub" argument to autoscript:
+$1 > 9 or ($1 == 9 and $2 >= '20120909') or error('debhelper 9.20120909 or later required');
+
 ## ----------------------------------------------------------------------
 my $xmlcorever	= "0.12";
 
@@ -124,7 +130,7 @@
     my $cmd = 'update-xmlcatalog';
     $cmd .= ' --add';
     $cmd .= " --type $type";
-    $cmd .= " --id \\\"$id\\\"";
+    $cmd .= " --id \"$id\"";
     $cmd .= " --package $pkg";
     if ( $local ) {
 	$cmd .= " --local $local";
@@ -141,7 +147,7 @@
     my $cmd = 'update-xmlcatalog';
     $cmd .= ' --del';
     $cmd .= " --type $type";
-    $cmd .= " --id \\\"$id\\\"";
+    $cmd .= " --id \"$id\"";
     if ( $root ) {
 	$cmd .= " --root";
     } else {
@@ -216,8 +222,8 @@
 						die("error: package command with ID '$id' uses non-existent catalog '$local'\n");
 					}
 					
-					$ADD_PACKAGE .= "\t" . add_xmlcat_cmd($package, $type, $id, $local) . "\\n";
-					$DEL_PACKAGE .= "\t" . del_xmlcat_cmd($package, $type, $id) . "\\n";
+					$ADD_PACKAGE .= "\t" . add_xmlcat_cmd($package, $type, $id, $local) . "\n";
+					$DEL_PACKAGE .= "\t" . del_xmlcat_cmd($package, $type, $id) . "\n";
 
 				}
 			} elsif ( $line->[0] eq 'root' ) {
@@ -225,8 +231,8 @@
 
 					my $type = $line->[1];
 					my $id	 = $line->[2];
-					$ADD_ROOT .= "\t" . add_xmlcat_cmd($package, $type, $id) . "\\n";
-					$DEL_ROOT .= "\t" . del_xmlcat_cmd($package, $type, $id, 1) . "\\n";
+					$ADD_ROOT .= "\t" . add_xmlcat_cmd($package, $type, $id) . "\n";
+					$DEL_ROOT .= "\t" . del_xmlcat_cmd($package, $type, $id, 1) . "\n";
 
 				}
 			} elsif ( $line->[0] eq 'root-and-package' ) {
@@ -242,10 +248,10 @@
 						die("error: root-and-package command with ID '$id' uses non-existent catalog '$local'\n");
 					}
 
-					$ADD_PACKAGE .= "\t" . add_xmlcat_cmd($package, $type, $id, $local) . "\\n";
-					$DEL_PACKAGE .= "\t" . del_xmlcat_cmd($package, $type, $id) . "\\n";
-					$ADD_ROOT    .= "\t" . add_xmlcat_cmd($package, $type, $id) . "\\n";
-					$DEL_ROOT    .= "\t" . del_xmlcat_cmd($package, $type, $id, 1) . "\\n";
+					$ADD_PACKAGE .= "\t" . add_xmlcat_cmd($package, $type, $id, $local) . "\n";
+					$DEL_PACKAGE .= "\t" . del_xmlcat_cmd($package, $type, $id) . "\n";
+					$ADD_ROOT    .= "\t" . add_xmlcat_cmd($package, $type, $id) . "\n";
+					$DEL_ROOT    .= "\t" . del_xmlcat_cmd($package, $type, $id, 1) . "\n";
 
 				}
 			} else {
@@ -268,11 +274,11 @@
 			$DEL_PACKAGE or $DEL_PACKAGE = "\t:";
 			$DEL_ROOT    or $DEL_ROOT    = "\t:";
 			autoscript( $package, "postinst", "postinst-xmlcatalog",
-				    "s%#ADD_PACKAGE#%$ADD_PACKAGE%g; s%#ADD_ROOT#%$ADD_ROOT%g;" );
+				    sub { s{#ADD_PACKAGE#}{$ADD_PACKAGE}g; s{#ADD_ROOT#}{$ADD_ROOT}g; } );
 			autoscript( $package, "prerm", "prerm-xmlcatalog",
-				    "s%#DEL_PACKAGE#%$DEL_PACKAGE%g; s%#DEL_ROOT#%$DEL_ROOT%g;" );
+				    sub { s{#DEL_PACKAGE#}{$DEL_PACKAGE}g; s{#DEL_ROOT#}{$DEL_ROOT}g; } );
 			autoscript( $package, "postrm", "postrm-xmlcatalog",
-				    "s%#PACKAGECAT#%$packagecat%g;" );
+				    sub { s{#PACKAGECAT#}{$packagecat}g; } );
 
 			addsubstvar( $package,
 				     "misc:Depends", "xml-core", ">= $xmlcorever" );

Reply via email to