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

Re: t7405.17 breakage vanishes with GETTEXT_POISON=1

From
Duy Nguyen <pclouds@gmail.com>
Date
Oct 29, 2018, 16:19 UTC
Message-ID
<CACsJy8CWkxfkOEzTBwuD0hyATuc+i3DWYZ2k+kA0CbxFhcgs6g@mail.gmail.com>
In-Reply-To
<20181028121517.GO30222@szeder.dev>
On Sun, Oct 28, 2018 at 1:15 PM SZEDER Gábor <szeder.dev@gmail.com> wrote:
Show 25 quoted lines
>
> On Sun, Oct 28, 2018 at 06:41:06AM +0100, Duy Nguyen wrote:
> > Something fishy is going on but I don't think I'll spend time hunting
> > it down so I post here in case somebody else is interested. It might
> > also indicate a problem with poison gettext, not the test case too.
>
> I haven't actually run the test under GETTEXT_POISON, but I stongly
> suspect it's the test, or more accurately the helper function
> 'test_i18ngrep'.
>
> The test in question runs
>
>   test_i18ngrep ! "refusing to lose untracked file at" err
>
> which fails in normal test runs, because 'grep' does find the
> undesired string; that's the known breakage.  Under GETTEXT_POISION,
> however, 'test_i18ngrep' always succeeds because of this condition:
>
>   if test -n "$GETTEXT_POISON"
>   then
>           # pretend success
>           return 0
>   fi
>
> and then in turn the whole test succeeds.
And this is something your poison-with-scrambling code does help ;-) ;-)
-- 
Duy
Previous: Junio C Hamano
Message 4 of 4 in “t7405.17 breakage vanishes with GETTEXT_POISON=1”
  1. Duy NguyenOct 28, 2018
  2. SZEDER GáborOct 28, 2018
  3. Junio C HamanoOct 29, 2018
  4. Duy NguyenOct 29, 2018

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.