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

Re: [PATCH] t0090: mark add-interactive test with PERL prerequisite

From
Jeff King <peff@peff.net>
Date
Nov 18, 2014, 18:49 UTC
Message-ID
<20141118184924.GB13479@peff.net>
In-Reply-To
<20141118184347.GE6527@google.com>
On Tue, Nov 18, 2014 at 10:43:47AM -0800, Jonathan Nieder wrote:
Show 8 quoted lines
> ... and here's a patch on top to give git-p4 the same treatment.
> 
> -- >8 --
> Subject: Makefile: have python scripts depend on NO_PYTHON setting
> 
> Like the perl scripts, python scripts need a dependency to ensure they
> are rebuilt when switching between the "dummy" versions that run
> without Python and the real thing.

Thanks, I didn't think to look for similar cases. It seems python is the only other thing that gets the "unimplemented" treatment. If you do:

  make
  make NO_TCLTK=Yes

I think you'll still end up with a crufty gitk build, but the fix there is much more involved (it would have to create a "Sorry, gitk wasn't built" script). I don't think it's worth the effort.

-Peff
Previous: Jonathan NiederNext: Pete Wyckoff
Message 8 of 11 in “t0090: mark add-interactive test with PERL prerequisite”
  1. t0090: mark add-interactive test with PERL prerequisiteJeff King, Nov 18, 2014
  2. t960[34]: mark cvsimport tests as requiring perlJeff King, Nov 18, 2014
  3. Jonathan NiederNov 18, 2014
  4. Jeff KingNov 18, 2014
  5. Jeff KingNov 18, 2014
  6. Jonathan NiederNov 18, 2014
  7. Jonathan NiederNov 18, 2014
  8. Jeff KingNov 18, 2014
  9. Pete WyckoffNov 18, 2014
  10. Jeff KingNov 18, 2014
  11. Jonathan NiederNov 18, 2014

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.