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

Re: [PATCH] t/Makefile: remove smoke test targets

From
Thomas Rast <trast@inf.ethz.ch>
Date
Apr 27, 2013, 11:16 UTC
Message-ID
<8738ucgr67.fsf@hexa.v.cablecom.net>
In-Reply-To
<dd20490a6d7911e0e0814fbe2eee0cfea1faa4e5.1366999104.git.john@keeping.me.uk>
John Keeping <john@keeping.me.uk> writes:
Show 12 quoted lines
> Commit d24fbca (Remove Git's support for smoke testing - 2011-12-23)
> removed the smoke test support from the test suite but it was re-added
> by commit 342e9ef (Introduce a performance testing framework -
> 2012-02-17).  This appears to be the result of a mis-merge, since
> re-adding the smoke testing infrastructure does not relate to the
> subject of that commit.
>
> The current 'smoke' target is broken since the 'harness' script it
> uses no longer exists, so just reapply this section of commit d24fbca
> and remove all of the smoke testing section in the makefile.
>
> Signed-off-by: John Keeping <john@keeping.me.uk>
Indeed.  I must have mis-rebased this commit or some such.
Acked-by: Thomas Rast <trast@inf.ethz.ch>
-- 
Thomas Rast
trast@{inf,student}.ethz.ch
Previous: John Keeping
Message 2 of 2 in “t/Makefile: remove smoke test targets”
  1. t/Makefile: remove smoke test targetsJohn Keeping, Apr 26, 2013
  2. Thomas RastApr 27, 2013

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.