* Subject: [PATCH] Fix git-commit-script to output on stderr when -v fails
@ 2005-08-20 16:07 Marco Costalba
0 siblings, 0 replies; only message in thread
From: Marco Costalba @ 2005-08-20 16:07 UTC (permalink / raw
To: junkio; +Cc: git
From: Marco Costalba <mcostalba@yahoo.it>
Date: 1124553736 +0200
When git-commit-script is called with -v option and
verify test fails result is print on stdout
instead of stderr.
Signed-off-by: Marco Costalba <mcostalba@yahoo.it>
---
git-commit-script | 10 +++++-----
1 files changed, 5 insertions(+), 5 deletions(-)
676a1dcf50a5abc3bc9751c9dbb0146ae31ebfc9
diff --git a/git-commit-script b/git-commit-script
--- a/git-commit-script
+++ b/git-commit-script
@@ -113,16 +113,16 @@ t)
sub bad_line {
my ($why, $line) = @_;
if (!$found_bad) {
- print "*\n";
- print "* You have some suspicious patch lines:\n";
- print "*\n";
+ print STDERR "*\n";
+ print STDERR "* You have some suspicious patch lines:\n";
+ print STDERR "*\n";
$found_bad = 1;
}
if ($reported_filename ne $filename) {
- print "* In $filename\n";
+ print STDERR "* In $filename\n";
$reported_filename = $filename;
}
- print "* $why (line $lineno)\n$line\n";
+ print STDERR "* $why (line $lineno)\n$line\n";
}
open $fh, "-|", qw(git-diff-cache -p -M --cached HEAD);
while (<$fh>) {
__________________________________________________
Do You Yahoo!?
Tired of spam? Yahoo! Mail has the best spam protection around
http://mail.yahoo.com
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2005-08-20 16:07 UTC | newest]
Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2005-08-20 16:07 Subject: [PATCH] Fix git-commit-script to output on stderr when -v fails Marco Costalba
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).