git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] t0005: add test for trap handling from deeply nested function calls

From
Johannes Sixt <j6t@kdbg.org>
Date
Mar 26, 2010, 20:19 UTC
Message-ID
<201003262119.45110.j6t@kdbg.org>
In-Reply-To
<vm2zHZjrZOha7LnIM_9fAEAvwVZrJtZlAI9f2XI_VUsdqr51ihAMW68a5BT9tvMdRDSqk6GYlSE@cipher.nrlssc.navy.mil>
On Freitag, 26. März 2010, Brandon Casey wrote:
Show 7 quoted lines
> +test_expect_success 'trap triggered in deeply nested function works
> correctly' ' +	(atrap () { exit 0; }
> +	 func3 () { exit 1; }
> +	 func2 () { func3; }
> +	 func1 () { trap atrap EXIT; func2; }
> +	 func1)
> +'

What do you want to achieve with this test? That /usr/xpg4/bin/sh is not used on Solaris?

If git-bisect triggers this bug in /usr/xpg4/bin/sh and if this is our shell of choice on Solaris, wouldn't it be better to work around it in git-bisect?

-- Hannes
Previous: Junio C HamanoNext: Brandon Casey
Message 3 of 4 in “t0005: add test for trap handling from deeply nested function calls”
  1. t0005: add test for trap handling from deeply nested function callsBrandon Casey, Mar 26, 2010
  2. Junio C HamanoMar 26, 2010
  3. Johannes SixtMar 26, 2010
  4. Brandon CaseyMar 27, 2010

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.