Move {safe_,}rename() from install{man,perl} into install_lib.pl
authorNicholas Clark <nick@ccl4.org>
Sat, 13 Jul 2013 16:33:53 +0000 (18:33 +0200)
committerNicholas Clark <nick@ccl4.org>
Tue, 23 Jul 2013 11:39:51 +0000 (13:39 +0200)
installman's rename() was identical to installperl's safe_rename() in all
but name (and whitespace), so de-duplicate by moving the code to
install_lib.pl

install_lib.pl
installman
installperl

index 308af70..1fe2579 100644 (file)
@@ -127,4 +127,18 @@ sub samepath {
     }
 }
 
+sub safe_rename {
+    my($from,$to) = @_;
+    if (-f $to and not unlink($to)) {
+        my($i);
+        for ($i = 1; $i < 50; $i++) {
+            last if rename($to, "$to.$i");
+        }
+        warn("Cannot rename to '$to.$i': $!"), return 0
+           if $i >= 50; # Give up!
+    }
+    link($from,$to) || return 0;
+    unlink($from);
+}
+
 1;
index 4f9d469..adb76f8 100755 (executable)
@@ -159,7 +159,7 @@ sub pod2man {
         print "  $xmanpage\n" unless $opts{silent};
         if (!$opts{notify} && $parser->parse_from_file($mod, $tmp)) {
             if (-s $tmp) {
-                if (rename($tmp, $manpage)) {
+                if (safe_rename($tmp, $manpage)) {
                     $packlist->{$xmanpage} = { type => 'file' };
                     next;
                 }
@@ -172,20 +172,6 @@ sub pod2man {
 $packlist->write() unless $opts{notify};
 print "  Installation complete\n" if $opts{verbose};
 
-sub rename {
-    my($from,$to) = @_;
-    if (-f $to and not unlink($to)) {
-       my($i);
-       for ($i = 1; $i < 50; $i++) {
-           last if CORE::rename($to, "$to.$i");
-       }
-       warn("Cannot rename to '$to.$i': $!"), return 0
-           if $i >= 50;        # Give up!
-    }
-    link($from,$to) || return 0;
-    unlink($from);
-}
-
 # Local variables:
 # cperl-indent-level: 4
 # indent-tabs-mode: nil
index 9cf7d7b..f71243f 100755 (executable)
@@ -625,20 +625,6 @@ sub safe_unlink {
     }
 }
 
-sub safe_rename {
-    my($from,$to) = @_;
-    if (-f $to and not unlink($to)) {
-       my($i);
-       for ($i = 1; $i < 50; $i++) {
-           last if rename($to, "$to.$i");
-       }
-       warn("Cannot rename to '$to.$i': $!"), return 0
-          if $i >= 50; # Give up!
-    }
-    link($from,$to) || return 0;
-    unlink($from);
-}
-
 sub copy {
     my($from,$to) = @_;