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

Re: [RFC+PATCH 1/1] Move SCM interoperability tools into scm/

From
THTimo Hirvonen <tihirvon@gmail.com>
Date
Jul 10, 2006, 09:45 UTC
Message-ID
<20060710124525.54ecaeab.tihirvon@gmail.com>
In-Reply-To
<44B19D42.6030701@michonline.com>
Ryan Anderson <ryan@michonline.com> wrote:
Show 6 quoted lines
> By "don't split the Makefile", do mean, "Don't use recursive make"?
> 
> I'm fine with that (and if you look at my patch, I "include
> scm/Makefile" to do just that), but if you mean "keep only a top-level
> Makefile", well, I think that continues the problem of "there is too
> much stuff going on here", but I can be persuaded otherwise.

Recursive make is a horrible hack. Splitting Makefile to smaller chunks which would be included in the top level Makefile is almost as bad. You can't cd to subdirectory and run make there and you have to prefix all files with "subdir/".

If you really want to make Makefile shorter just move all the platform specific stuff to a configure script.

-- 
http://onion.dynserv.net/~timo/
Previous: Ryan Anderson
Message 11 of 11 in “Move SCM interoperability tools into scm/”
  1. 1/1 Move SCM interoperability tools into scm/Ryan Anderson, Jul 9, 2006
  2. Johannes SchindelinJul 9, 2006
  3. Martin LanghoffJul 9, 2006
  4. Petr BaudisJul 9, 2006
  5. Johannes SchindelinJul 9, 2006
  6. Ryan AndersonJul 9, 2006
  7. Junio C HamanoJul 10, 2006
  8. Ryan AndersonJul 10, 2006
  9. Timo HirvonenJul 9, 2006
  10. Ryan AndersonJul 10, 2006
  11. Timo HirvonenJul 10, 2006

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.