X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?p=dgit.git;a=blobdiff_plain;f=Debian%2FDgit.pm;h=cb4a599b20849ea13b0656937c2fd15fb006a932;hp=fa85374150288cc53a1ef363042262315b76ab52;hb=b5f605a1e7d6a92c07636b378deb74e895b0967e;hpb=9acb31a971d4f653836bbcf07410f51d3f80dbdd

diff --git a/Debian/Dgit.pm b/Debian/Dgit.pm
index fa853741..cb4a599b 100644
--- a/Debian/Dgit.pm
+++ b/Debian/Dgit.pm
@@ -5,9 +5,12 @@ package Debian::Dgit;
 use strict;
 use warnings;
 
+use Carp;
 use POSIX;
 use IO::Handle;
 use Config;
+use Digest::SHA;
+use Data::Dumper;
 
 BEGIN {
     use Exporter   ();
@@ -18,6 +21,7 @@ BEGIN {
     @EXPORT      = qw(setup_sigwarn
                       debiantag server_branch server_ref
                       stat_exists link_ltarget
+		      hashfile
                       fail ensuredir executable_on_path
                       waitstatusmsg failedcmd
                       cmdoutput cmdoutput_errok
@@ -184,8 +188,19 @@ sub waitstatusmsg () {
 }
 
 sub failedcmd {
+    # Expects $!,$? as set by close - see below.
+    # To use with system(), set $?=-1 first.
+    #
+    # Actual behaviour of perl operations:
+    #   success              $!==0       $?==0       close of piped open
+    #   program failed       $!==0       $? >0       close of piped open
+    #   syscall failure      $! >0       $?=-1       close of piped open
+    #   failure              $! >0       unchanged   close of something else
+    #   success              trashed     $?==0       system
+    #   program failed       trashed     $? >0       system
+    #   syscall failure      $! >0       unchanged   system
     { local ($!); printcmd \*STDERR, _us().": failed command:", @_ or die $!; };
-    if ($!) {
+    if ($? < 0) {
 	fail "failed to fork/exec: $!";
     } elsif ($?) {
 	fail "subprocess ".waitstatusmsg();
@@ -195,7 +210,7 @@ sub failedcmd {
 }
 
 sub cmdoutput_errok {
-    die Dumper(\@_)." ?" if grep { !defined } @_;
+    confess Dumper(\@_)." ?" if grep { !defined } @_;
     debugcmd "|",@_;
     open P, "-|", @_ or die "$_[0] $!";
     my $d;
@@ -229,6 +244,13 @@ sub link_ltarget ($$) {
     link $old, $new or die "link $old $new: $!";
 }
 
+sub hashfile ($) {
+    my ($fn) = @_;
+    my $h = Digest::SHA->new(256);
+    $h->addfile($fn);
+    return $h->hexdigest();
+}
+
 sub git_rev_parse ($) {
     return cmdoutput qw(git rev-parse), "$_[0]~0";
 }