git@vger.kernel.org mailing list mirror (one of many)
 help / color / mirror / code / Atom feed
From: Johannes Sixt <j.sixt@viscovery.net>
To: Jonathan Nieder <jrnieder@gmail.com>
Cc: git@vger.kernel.org, Jeff King <peff@peff.net>
Subject: [PATCH] t0005: work around strange $? in ksh when program terminated by a signal
Date: Fri, 09 Jul 2010 09:05:16 +0200	[thread overview]
Message-ID: <4C36CA2C.5050305@viscovery.net> (raw)
In-Reply-To: <20100709030812.GA16877@dert.cs.uchicago.edu>

From: Johannes Sixt <j6t@kdbg.org>

ksh is known to report $? of programs that terminated by a signal as
256 + signal number instead of 128 + signal number like other POSIX
compliant shells. (ksh's behavior is still POSIX compliant in this regard.)

Signed-off-by: Johannes Sixt <j6t@kdbg.org>
---
Am 7/9/2010 5:08, schrieb Jonathan Nieder:
> I can’t seem to get test-sigchain to run with ksh93 on linux:
> 
>  $ ksh
>  
>  ksh>$ ../bin-wrappers/test-sigchain 
>  three
>  two
>  one
>  Terminated
>  ksh>$ echo $?
>  271

ksh is not wrong, just strange. See http://www.opengroup.org/onlinepubs/9699919799/utilities/V3_chap02.html#tag_18_08_02 (last sentence).

 -- Hannes

 t/t0005-signals.sh |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/t/t0005-signals.sh b/t/t0005-signals.sh
index 09f855a..93e58c0 100755
--- a/t/t0005-signals.sh
+++ b/t/t0005-signals.sh
@@ -13,6 +13,7 @@ test_expect_success 'sigchain works' '
 	test-sigchain >actual
 	case "$?" in
 	143) true ;; # POSIX w/ SIGTERM=15
+	271) true ;; # ksh w/ SIGTERM=15
 	  3) true ;; # Windows
 	  *) false ;;
 	esac &&
-- 
1.7.1.585.gf3448

  reply	other threads:[~2010-07-09  7:05 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-07-09  3:08 t0005 failing with ksh93 on linux Jonathan Nieder
2010-07-09  7:05 ` Johannes Sixt [this message]
2010-07-09 11:53   ` [PATCH] t0005: work around strange $? in ksh when program terminated by a signal Jeff King
2010-07-09 14:39   ` Jonathan Nieder
2010-07-09 15:45   ` Brandon Casey
2010-07-09 16:03     ` Ævar Arnfjörð Bjarmason
2010-07-09 16:07       ` Brandon Casey
2010-07-12  6:41     ` [PATCH v2] t0005: work around strange $? in ksh93 " Johannes Sixt
2010-07-12  6:47       ` Junio C Hamano

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: http://vger.kernel.org/majordomo-info.html

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4C36CA2C.5050305@viscovery.net \
    --to=j.sixt@viscovery.net \
    --cc=git@vger.kernel.org \
    --cc=jrnieder@gmail.com \
    --cc=peff@peff.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).