X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=dh_installman;h=cc94843e886f100955308a8678f59156fe8757f4;hb=fae5806aeb1a5e7b40a9c149a739f645395e5139;hp=d8585aa2e6e072e5093f560cdc45b2d2987a1a01;hpb=b352136551fb3384722942f0206419ca066dedda;p=debhelper.git

diff --git a/dh_installman b/dh_installman
index d8585aa..cc94843 100755
--- a/dh_installman
+++ b/dh_installman
@@ -12,15 +12,26 @@ use Debian::Debhelper::Dh_Lib;
 
 =head1 SYNOPSIS
 
-  dh_installman [debhelper options] [manpage ...]
+B<dh_installman> [S<I<debhelper options>>] [S<I<manpage ...>>]
 
 =head1 DESCRIPTION
 
-dh_installman is a debhelper program that handles for installing
+dh_installman is a debhelper program that handles installing
 man pages into the correct locations in package build directories. You tell
 it what man pages go in your packages, and it figures out where to
 install them based on the section field in their .TH line and their filename
-extention.
+extension. It also supports translated man pages, by looking for extensions
+like .ll.8 and .ll_LL.8
+
+If dh_installman seems to install a man page into the wrong section or with
+the wrong extension, this is because the man page has the wrong section
+listed in its .TH line. Edit the man page and correct the section, and
+dh_installman will follow suit.  See to L<man(7)> for details about the .TH
+section.  If dh_installman seems to install a man page into a directory
+like /usr/share/man/pl/man1/, that is because your program has a
+name like "foo.pl", and dh_installman assumes that means it is translated
+into Polish. There is currently no support for resolving this ambiguity;
+programs in debian should proably not have extensions like that anyway.
 
 Any man page filenames specified as parameters will be installed into the
 first package dh_installman is told to act on. By default, this is the
@@ -78,7 +89,7 @@ foreach my $package (@{$dh{DOPACKAGES}}) {
 	}
 
 	foreach my $page (@manpages) {
-		my $basename=Debian::Debhelper::Dh_Lib::basename($page);
+		my $basename=basename($page);
 
 		# Support compressed pages.
 		my $gz='';
@@ -115,19 +126,21 @@ foreach my $package (@{$dh{DOPACKAGES}}) {
 			error("Could not determine section for $page");
 		}
 		
+		# Get the man page's name -- everything up to the last dot.
+		my ($instname)=$basename=~m/^(.*)\./;
+	
 		my $destdir="$tmp/usr/share/man/man$realsection/";
 		# Translated man pages are typically specified by adding the
 		# language code to the filename, so detect that and
-		# redirect to appropriate directory.
+		# redirect to appropriate directory, stripping the code.
 		my ($langcode)=$basename=~m/.*\.([a-z][a-z](?:_[A-Z][A-Z])?)\.(?:[1-9]|man)/;
 		if (defined $langcode && $langcode ne '') {
 			$destdir="$tmp/usr/share/man/$langcode/man$section/";
+			# Strip the language code from the instname.
+			$instname=~s/\.$langcode$//;
 		}
 		$destdir=~tr:/:/:s; # just for looks
 
-		# Get the man page's name -- everything up to the last dot.
-		my ($instname)=$basename=~m/^(.*)\./;
-	
 		if (! -e "$destdir/$instname.$section" && 
 		    ! -l "$destdir/$instname.$section") {
 			if (! -d $destdir) {
@@ -157,20 +170,20 @@ foreach my $package (@{$dh{DOPACKAGES}}) {
 sub find_so_man {
 	# The -s test is becuase a .so file tends to be small. We don't want
 	# to open every man page. 1024 is arbitrary.
-	if (! -f $_ || -s $_ > 1024) {
+	if (! -f $_ || -s $_ > 1024 || -s == 0) {
 		return;
 	}
 
 	# Test first line of file for the .so thing.
-	open (SOTEST,$_);
+	open (SOTEST,$_) || die "$_: $!";
 	my $l=<SOTEST>;
 	close SOTEST;
 	if ($l=~m/\.so\s+(.*)/) {
 		my $solink=$1;
 		# This test is here to prevent links like ... man8/../man8/foo.8
-		if (Debian::Debhelper::Dh_Lib::basename($File::Find::dir) eq
-		    Debian::Debhelper::Dh_Lib::dirname($solink)) {
-			$solink=Debian::Debhelper::Dh_Lib::basename($solink);
+		if (basename($File::Find::dir) eq
+		    dirname($solink)) {
+			$solink=basename($solink);
 		}
 		else {
 			$solink="../$solink";