Package: debhelper Version: 9.20160403 Severity: normal Tags: patch Hi,
Thanks for maintaining debhelper! What do you think about the attached patch that makes dh_strip only match static libraries with "lib" at the start of the filename, not just anywhere in there? It makes dh_strip's behavior consistent when a Debian package ships *.a files with weird names, like e.g. gforth's /usr/lib/*/gforth/0.7.2/cstr.a and /usr/lib/*/gforth/0.7.2./fflib.a - the first one is not stripped, but the second one is :) [1] On an unrelated note, the second patch is a bit of clean-up for regular expressions that start with /.*/ and are not anchored (don't start with /^.*/) - this is pretty much a no-op and, in the worst case, may make the Perl virtual machine handle the regexp a bit inefficiently. Well, I do believe that the regexp compiler is smart enough to drop the ".*" at the start anyway, but there's no need to make it do even that much :) Thanks again for taking care of debhelper, and keep up the great work! G'luck, Peter [1] Yes, with this patch neither one will be stripped, but in this case this is not really a problem - these are dynamically generated (compiled from dynamically generated *.c files) helpers for loading native system libraries, and we don't care about debug symbols for them. Moreover, stripping the ones shipped with GForth would lead to an inconsistency when the user later generates new ones that shall not be stripped; not that anyone would ever notice, but oh well :) -- System Information: Debian Release: stretch/sid APT prefers oldoldstable APT policy: (500, 'oldoldstable'), (500, 'testing') Architecture: amd64 (x86_64) Foreign Architectures: i386 Kernel: Linux 4.4.0-1-amd64 (SMP w/4 CPU cores) Locale: LANG=bg_BG.UTF-8, LC_CTYPE=bg_BG.UTF-8 (charmap=UTF-8) Shell: /bin/sh linked to /bin/dash Init: systemd (via /run/systemd/system) Versions of packages debhelper depends on: ii autotools-dev 20150820.1 ii binutils 2.26-8 ii dh-autoreconf 12 ii dh-strip-nondeterminism 0.016-1 ii dpkg 1.18.4 ii dpkg-dev 1.18.4 ii file 1:5.25-2 ii libdpkg-perl 1.18.4 ii man-db 2.7.5-1 ii perl 5.22.1-9 ii po-debconf 1.0.19 debhelper recommends no packages. Versions of packages debhelper suggests: ii dh-make 2.201605 -- no debconf information
From 63b79a7d50ed3e89de4b093b44da25ba899d86e1 Mon Sep 17 00:00:00 2001
From: Peter Pentchev <r...@ringlet.net>
Date: Fri, 8 Apr 2016 17:31:13 +0300
Subject: [PATCH 1/2] dh_strip: Only match *.a files that start with "lib"
---
dh_strip | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/dh_strip b/dh_strip
index 752176e..a7206db 100755
--- a/dh_strip
+++ b/dh_strip
@@ -230,7 +230,7 @@ sub testfile {
}
# Is it a static library, and not a debug library?
- if (m/lib.*\.a$/ && ! m/.*_g\.a$/) {
+ if (m/\/lib[^\/]*\.a$/ && ! m/.*_g\.a$/) {
# Is it a binary file, or something else (maybe a linker
# script on Hurd, for example? I don't use file, because
# file returns a variety of things on static libraries.
--
2.8.0.rc3
From 3aa558d06a82b2c86dafaf9efac88e6f786d3e86 Mon Sep 17 00:00:00 2001
From: Peter Pentchev <r...@ringlet.net>
Date: Fri, 8 Apr 2016 17:42:25 +0300
Subject: [PATCH 2/2] Drop .* from the beginning of unanchored regular
expressions
---
Debian/Debhelper/Dh_Lib.pm | 2 +-
dh_installman | 4 ++--
dh_installmanpages | 4 ++--
dh_strip | 6 +++---
4 files changed, 8 insertions(+), 8 deletions(-)
diff --git a/Debian/Debhelper/Dh_Lib.pm b/Debian/Debhelper/Dh_Lib.pm
index bc2bd43..2f028d3 100644
--- a/Debian/Debhelper/Dh_Lib.pm
+++ b/Debian/Debhelper/Dh_Lib.pm
@@ -567,7 +567,7 @@ sub pkgfilename {
}
# Is this a native Debian package?
- if ($dh{VERSION}=~m/.*-/) {
+ if ($dh{VERSION}=~m/-/) {
return $isnative_cache{$package}=0;
}
else {
diff --git a/dh_installman b/dh_installman
index 46b0366..826eeae 100755
--- a/dh_installman
+++ b/dh_installman
@@ -140,7 +140,7 @@ foreach my $package (@{$dh{DOPACKAGES}}) {
close(IN);
# Failing that, we can try to get it from the filename.
if (! $section) {
- ($section)=$basename=~m/.*\.([1-9]\S*)/;
+ ($section)=$basename=~m/\.([1-9]\S*)/;
}
# Now get the numeric component of the section.
@@ -158,7 +158,7 @@ foreach my $package (@{$dh{DOPACKAGES}}) {
# Translated man pages are typically specified by
adding the
# language code to the filename, so detect that and
# redirect to appropriate directory, stripping the code.
-
($langcode)=$basename=~m/.*\.([a-z][a-z](?:_[A-Z][A-Z])?)\.(?:[1-9]|man)/;
+
($langcode)=$basename=~m/\.([a-z][a-z](?:_[A-Z][A-Z])?)\.(?:[1-9]|man)/;
}
elsif ($dh{LANGUAGE} ne 'C') {
$langcode=$dh{LANGUAGE};
diff --git a/dh_installmanpages b/dh_installmanpages
index d393500..e58415a 100755
--- a/dh_installmanpages
+++ b/dh_installmanpages
@@ -158,13 +158,13 @@ foreach my $package (@{$dh{DOPACKAGES}}) {
if ($install) {
my $extdir="share";
- my ($section)=$basename=~m/.*\.([1-9])/;
+ my ($section)=$basename=~m/\.([1-9])/;
my $destdir="$tmp/usr/$extdir/man/man$section/";
# Handle translated man pages.
my $instname=$basename;
- my ($langcode)=$basename=~m/.*\.([a-z][a-z])\.([1-9])/;
+ my ($langcode)=$basename=~m/\.([a-z][a-z])\.([1-9])/;
if (defined $langcode && $langcode ne '') {
$destdir="$tmp/usr/$extdir/man/$langcode/man$section/";
$instname=~s/\.$langcode\./\./;
diff --git a/dh_strip b/dh_strip
index a7206db..fb5d74b 100755
--- a/dh_strip
+++ b/dh_strip
@@ -209,10 +209,10 @@ sub testfile {
# Does its filename look like a shared library?
# - *.cmxs are OCaml native code shared libraries
# - *.node are also native ELF binaries (for node-js)
- if (m/.*\.(?:so.*?|cmxs|node)$/) {
+ if (m/\.(?:so.*?|cmxs|node)$/) {
# Ok, do the expensive test.
my $type=get_file_type($_);
- if ($type=~m/.*ELF.*shared.*/) {
+ if ($type=~m/ELF.*shared.*/) {
push @shared_libs, $fn;
return;
}
@@ -223,7 +223,7 @@ sub testfile {
if ($mode & 0111) {
# Ok, expensive test.
my $type=get_file_type($_);
- if ($type=~m/.*ELF.*(executable|shared).*/) {
+ if ($type=~m/ELF.*(executable|shared).*/) {
push @executables, $fn;
return;
}
--
2.8.0.rc3
signature.asc
Description: PGP signature
