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

Re: [PATCH] Makefile: check that tcl/tk is installed

From
Todd Zullinger <tmz@pobox.com>
Date
Nov 27, 2017, 05:22 UTC
Message-ID
<20171127052254.GA3693@zaya.teonanacatl.net>
In-Reply-To
<20171127043502.GA5946@sigill>
Jeff King wrote:
> Yeah, this side-steps the "other half" of the issue that Christian's 
> patch addresses, which seems like the more controversial part (I don't 
> have a strong opinion myself, though).

I don't either. The general motivation there, as far as I understand, is that it's undesirable to have 'make install' install tools that cannot run.

Perhaps it's worth noting that there are other commands installed by default which won't work out of the box. For example, 'git svn' requires subversion at run time but not at build time.

If there aren't many such commands, then maybe checking for them in the Makefile is reasonable to make installing git from source easier for new users. Without looking closely, I can't do more than take a wild guess.

As a package maintainer and an aging sysadmin, I'd bet that there are more dependencies than one might initially guess. I also tend to think that by the time most users want something newer than their OS provides that they're a bit more able to deal with these sort of issues. But I could easily be wrong on all counts.

-- 
Todd
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Going to trial with a lawyer who considers your whole life-style a
Crime in Progress is not a happy prospect.
    -- Hunter S. Thompson
Previous: Jeff KingNext: Christian Couder
Message 11 of 25 in “Makefile: check that tcl/tk is installed”
  1. Makefile: check that tcl/tk is installedChristian Couder, Nov 20, 2017
  2. Christian CouderNov 20, 2017
  3. Jonathan NiederNov 20, 2017
  4. Christian CouderNov 20, 2017
  5. Jeff KingNov 26, 2017
  6. Christian CouderNov 26, 2017
  7. Jeff KingNov 27, 2017
  8. Junio C HamanoNov 27, 2017
  9. Junio C HamanoNov 27, 2017
  10. Jeff KingNov 27, 2017
  11. Todd ZullingerNov 27, 2017
  12. Christian CouderNov 27, 2017
  13. Jeff KingNov 27, 2017
  14. Junio C HamanoNov 27, 2017
  15. Junio C HamanoNov 28, 2017
  16. travis-ci: avoid new tcl/tk build requirementTodd Zullinger, Nov 28, 2017
  17. Christian CouderNov 28, 2017
  18. Todd ZullingerNov 28, 2017
  19. Junio C HamanoNov 28, 2017
  20. Junio C HamanoNov 27, 2017
  21. Christian CouderNov 25, 2017
  22. Junio C HamanoNov 26, 2017
  23. Christian CouderNov 26, 2017
  24. Ramsay JonesNov 26, 2017
  25. Christian CouderNov 26, 2017

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.