[debhelper-devel] [debhelper] 02/02: dh_builddeb: Use default comp. settings for dbgsyms

Niels Thykier nthykier at moszumanska.debian.org
Thu Jul 27 05:45:03 UTC 2017


This is an automated email from the git hooks/post-receive script.

nthykier pushed a commit to branch master
in repository debhelper.

commit ddee9c80acffef3da31f620607439a17cb99792b
Author: Niels Thykier <niels at thykier.net>
Date:   Thu Jul 27 05:44:20 2017 +0000

    dh_builddeb: Use default comp. settings for dbgsyms
    
    Signed-off-by: Niels Thykier <niels at thykier.net>
---
 debian/changelog |  4 ++++
 dh_builddeb      | 10 ++++------
 2 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/debian/changelog b/debian/changelog
index 4bc95d6..3ae0864 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -55,6 +55,10 @@ debhelper (10.7) UNRELEASED; urgency=medium
   * dh_gencontrol: Rewrite code that ensures that misc:Depends and
     misc:Pre-Depends are present to avoid relying external processes
     for this.
+  * dh_builddeb: Let dpkg-deb choose the default compression level for
+    dbgsym packages instead of using xz -1e.  The current default (xz -6)
+    is likely to be a better default value for dbgsym packages.  Thanks to
+    Adrian Bunk for the suggestion.  (Closes: #868674)
 
   [ Colin Watson ]
   * Quote assignment to $recommended_compat in case it isn't a valid Perl
diff --git a/dh_builddeb b/dh_builddeb
index 90dd472..d091682 100755
--- a/dh_builddeb
+++ b/dh_builddeb
@@ -131,12 +131,10 @@ on_items_in_parallel(\@items, sub {
 
 		if ($dbgsym) {
 			my $dbgsym_tmpdir = "debian/.debhelper/${package}/dbgsym-root";
-			# XXX: Should we blindly overrule the maintainer here?  It
-			# is not apparent that their explicit -z was intended for
-			# the dbgsym package.
-			my @args = default_compressor_args(["-z1", "-Zxz", "-Sextreme"],
-				@{$dh{U_PARAMS}});
-			doit("dpkg-deb", @args,
+			# XXX: Should we blindly overrule the maintainer's choice
+			# of compressor arguments here?  It is not apparent that
+			# their explicit -z was intended for the dbgsym package.
+			doit("dpkg-deb", @{$dh{U_PARAMS}}
 				"--build", $dbgsym_tmpdir, $dh{DESTDIR});
 			next;
 		}

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/debhelper/debhelper.git




More information about the debhelper-devel mailing list