Package: dh-exec
Severity: wishlist

Dear Maintainer,

Please consider adding and equivalent to "install-rename" for
renaming files to the dh_installman / .manpages

I tried hacking src/dh-exec-install.c to allow it to be used on .manpages
(in addition to .install), but unfortunately the output it generates
are not compatible with dh_installman because of destdir....

This patch should make the output compatible with dh_installman but that
would then kill the moving support for dh_install....

(I'm not suggesting you actually use this. Please consider if
it would be possible to send additional information to determine
in dh-exec-install-rename if we should generate dh_install or dh_installman
compatible output -- or if a dedicated manpages helper should be added
basically duplicating most of lib/dh-exec-install-rename.)


diff --git a/lib/dh-exec-install-rename b/lib/dh-exec-install-rename
index 6e8d9a7..6aeb3dd 100755
--- a/lib/dh-exec-install-rename
+++ b/lib/dh-exec-install-rename
@@ -29,5 +29,5 @@ if (/([^\s]*)\s+=>\s+([^\s]*)/) {
                 move (File::Spec->catfile ("debian/tmp", $src),
                       File::Spec->catfile ($debpath, $dstfile)) or
                                 die "Copy failed: $!";
-        $_ = File::Spec->catfile ($debpath, $dstfile) . " " . $dstpath . "\n";
+        $_ = File::Spec->catfile ($debpath, $dstfile) . "\n";
 }
diff --git a/src/dh-exec-install.c b/src/dh-exec-install.c
index 25e5845..53dcf5b 100644
--- a/src/dh-exec-install.c
+++ b/src/dh-exec-install.c
@@ -40,7 +40,9 @@ preamble(int argc, char *argv[])
 
   /* Handle cases where the source is not an .install file */
   if (strcmp (src, "install") != 0 &&
-      fnmatch ("*[./]install", src, 0) != 0)
+      fnmatch ("*[./]install", src, 0) != 0 &&
+      strcmp (src, "manpages") != 0 &&
+      fnmatch ("*[./]manpages", src, 0) != 0)
     {
       /* Source is stdin, we're piped, ignore it. */
       if (argc < 2)
@@ -50,7 +52,7 @@ preamble(int argc, char *argv[])
           /* Source is from the command-line directly, raise an
              error. */
           fprintf (stderr,
-                   "%s: Only .install filename extensions are allowed: %s\n",
+                   "%s: Only .install or .manpages filename extensions are 
allowed: %s\n",
                    argv[0], src);
           return EXIT_FAILURE;
         }



-- System Information:
Debian Release: jessie/sid
  APT prefers unstable
  APT policy: (500, 'unstable'), (1, 'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 3.14-1-amd64 (SMP w/4 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=sv_SE.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash


-- 
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]

Reply via email to