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

Re: bug: git-sh-setup should not be in $PATH

From
Jakub Narebski <jnareb@gmail.com>
Date
Dec 6, 2006, 16:52 UTC
Message-ID
<200612061752.39443.jnareb@gmail.com>
In-Reply-To
<4576F29B.6010105@xs4all.nl>
Han-Wen Nienhuys wrote:
> Jakub Narebski escreveu:
Show 29 quoted lines
>>> there should be Makefiles in said directories. Otherwise, every rule
>>> in the toplevel Makefile saying
>>>
>>>
>>>   $(MAKE) -C subdir/
>>>
>>> will fail because $builddir/ does not have a Makefile in
>>> $builddir/subdir.
>> 
>> Wouldn't it be better just to modify toplevel Makefile to say:
>> 
>>    $(MAKE) -C $(srcdir)/subdir/
>> 
>> without messing with ./configure script, and creating "redirect"
>> Makefiles?
>> 
> 
> that doesn't work.
> 
>   make -C foo bla
> 
> just does 
> 
>   (cd foo && make bla)
> 
> If you did this, part of the object files would end up in the source dir; 
> The idea of separate srcdir builds,is to keep the srcdir completely clean 
> of generated files.  That can't really be done without aid of a script,
> to setup the builddir. In this case that is the configure script.

Thanks for an explanation. At least some of it should be in commit message, to explain what you want the patch to do.

I don't know autotools enough to check if autotools (autoconf) doesn't have it's own way to do this, some autoconf macro, to not need to write this script creating redirect Makefiles.

I'm still not sure if this has place in git. Is it really common to support building outside source directory? If git didn't support this, so what?

-- 
Jakub Narebski
Previous: Han-Wen NienhuysNext: Han-Wen Nienhuys
Message 14 of 17 in “bug: git-sh-setup should not be in $PATH”
  1. Han-Wen NienhuysDec 6, 2006
  2. Johannes SchindelinDec 6, 2006
  3. Han-Wen NienhuysDec 6, 2006
  4. Jakub NarebskiDec 6, 2006
  5. Han-Wen NienhuysDec 6, 2006
  6. Jakub NarebskiDec 6, 2006
  7. Han-Wen NienhuysDec 6, 2006
  8. Jakub NarebskiDec 6, 2006
  9. Han-Wen NienhuysDec 6, 2006
  10. Jakub NarebskiDec 6, 2006
  11. Han-Wen NienhuysDec 6, 2006
  12. Jakub NarebskiDec 6, 2006
  13. Han-Wen NienhuysDec 6, 2006
  14. Jakub NarebskiDec 6, 2006
  15. Han-Wen NienhuysDec 6, 2006
  16. Jakub NarebskiDec 6, 2006
  17. Andreas EricssonDec 7, 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.