[debhelper-devel] [debhelper] 05/05: dh_installman: Use rename_path instead of doit

Niels Thykier nthykier at moszumanska.debian.org
Mon Jun 26 07:56:08 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 292b514722e6aecdb8fbe90b0496f22298dc8f3b
Author: Niels Thykier <niels at thykier.net>
Date:   Mon Jun 26 07:54:29 2017 +0000

    dh_installman: Use rename_path instead of doit
    
    Signed-off-by: Niels Thykier <niels at thykier.net>
---
 debian/changelog | 1 +
 dh_installman    | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/debian/changelog b/debian/changelog
index bd95f55..4069537 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -18,6 +18,7 @@ debhelper (10.5.1) UNRELEASED; urgency=medium
   * dh_strip: Ditto.
   * Dh_Lib: Avoid 1-2 stat calls in make_symlink by reusing existing
     stat calls.
+  * dh_installman: Avoid a fork+exec for renaming a path.
 
  -- Niels Thykier <niels at thykier.net>  Sun, 25 Jun 2017 18:02:30 +0000
 
diff --git a/dh_installman b/dh_installman
index ec55873..49e58b9 100755
--- a/dh_installman
+++ b/dh_installman
@@ -244,7 +244,7 @@ on_selected_pkgs_in_parallel(\@all_packages, sub {
 						complex_doit "man --recode UTF-8 ./\Q$orig\E > \Q$tmp\E";
 						# recode uncompresses compressed pages
 						rm_files($orig) if s/\.(gz|Z)$//;
-						doit "mv", "-f", $tmp, $_;
+						rename_path($tmp, $_);
 						# Schedule a permission reset
 						push(@files, "${File::Find::dir}/${_}");
 					}, "$tmp/$dir");

-- 
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