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

Re: 195b7ca6 breaks t9010 at current master

From
Øyvind A. Holm <sunny@sunbase.org>
Date
Mar 29, 2011, 13:37 UTC
Message-ID
<AANLkTi=pDM2AXwLWtQQmp77dAzhooFSMXHtKAoN_S8_a@mail.gmail.com>
In-Reply-To
<20110329042446.GB25693@elie>
On 29 March 2011 06:24, Jonathan Nieder <jrnieder@gmail.com> wrote:
Show 10 quoted lines
> Øyvind A. Holm wrote:
> > When running "make test" at current master (v1.7.4.2-406-gbe919d5),
> > t9010-svn-fe.sh fails. Bisecting shows that 195b7ca6 ("vcs-svn:
> > handle log message with embedded NUL") breaks the test.
>
> Could you try with the following patch applied?  It comes from
> squashing the last two patches from the svn-fe branch:
>
> * tests: make sure input to sed is newline terminated
> * vcs-svn: add missing cast to printf argument

Thanks a lot. Yes, that works. It seems as it's "vcs-svn: add missing cast to printf argument" that does the trick, at least t9010 doesn't fail after that's applied. ... And it has reached the official master now (as 8cc299d), seems it got pushed after I went to bed. I'm running a test on current master (07514c8) now, and things look fine so far.

Regards, Øyvind

Previous: Jonathan NiederNext: Brandon Casey
Message 5 of 7 in “195b7ca6 breaks t9010 at current master”
  1. Øyvind A. HolmMar 29, 2011
  2. Jonathan NiederMar 29, 2011
  3. Brian GernhardtMar 29, 2011
  4. [PULL svn-fe] Re: 195b7ca6 breaks t9010 at current masterJonathan Nieder, Mar 29, 2011
  5. Øyvind A. HolmMar 29, 2011
  6. Brandon CaseyMar 31, 2011
  7. Jonathan NiederMar 31, 2011

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.