Bug#470835: colordiff: return exit code resulting from diff so can be used as drop in replacement

[Tim Connors]

Package: colordiff
Version: 1.0.7-2
Severity: normal
Tags: patch

diff -u -b -r1.5 -r1.6
--- colordiff   22 Nov 2006 12:13:42 -0000      1.5
+++ colordiff   4 Mar 2008 02:59:25 -0000       1.6
@@ -164,11 +164,14 @@
 
 my @inputstream;
 
+my $exitcode=0;
 if (defined $ARGV[0]) {
     # More reliable way of pulling in arguments
-    open2(\*INPUTSTREAM, undef, "diff", @ARGV);
+    my $pid = open2(\*INPUTSTREAM, undef, "diff", @ARGV);
     @inputstream = <INPUTSTREAM>;
     close INPUTSTREAM;
+    waitpid $pid, 0;
+    $exitcode=$? >> 8;
 }
 else {
     @inputstream = <STDIN>;
@@ -385,3 +388,5 @@
 ###     $type =~ s/-/_/;
 ###     eval "\$$type = \$colour{$color}";
 ### }
+
+exit $exitcode;


-- System Information:
Debian Release: lenny/sid
  APT prefers unstable
  APT policy: (500, 'unstable'), (500, 'testing'), (500, 'stable'), (1, 
'experimental')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.24 (SMP w/2 CPU cores)
Locale: LANG=en_AU, LC_CTYPE=en_AU (charmap=ISO-8859-1)
Shell: /bin/sh linked to /bin/bash

Versions of packages colordiff depends on:
ii  perl                          5.8.8-12   Larry Wall's Practical Extraction 

colordiff recommends no packages.

-- no debconf information



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