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

[PATCH 2/2] Makefile: add missing dependency on http.h

From
Jonathan Nieder <jrnieder@gmail.com>
Date
Aug 8, 2010, 21:48 UTC
Message-ID
<20100808214859.GC6962@burratino>
In-Reply-To
<20100808211910.GA7210@burratino>

v1.7.1-rc0~65^2~2 (http: init and cleanup separately from http-walker, 2010-03-02) introduced a direct dependency from http-fetch on the HTTP request library. Declare it.

Detected with "make CHECK_HEADER_DEPENDENCIES=1".
Cc: Tay Ray Chuan <rctay89@gmail.com>
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
---
After this series, “make COMPUTE_HEADER_DEPENDENCIES=YesPlease”
succeeds again.
Ideas for future work:
 - speed up the dependency checker, run it regularly;
 - port COMPUTE_HEADER_DEPENDENCIES to non-gcc compilers so it can
   be used by default;
 - speed up the redundant-dependency detector[1] enough to make it
   usable;
 - autoconf test;
 - “make test-makefile” for cron jobs.
Alas, this series did none of those things.

[1] <20100131231103.GA5287@progeny.tock> which doesn’t seem to have made it out into the world. Will resend as reply.

 Makefile |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/Makefile b/Makefile
index 9a233b0..e151516 100644
--- a/Makefile
+++ b/Makefile
@@ -1856,7 +1856,7 @@ builtin/tar-tree.o archive-tar.o: tar.h
 builtin/pack-objects.o: thread-utils.h
 connect.o transport.o http-backend.o: url.h
 http-fetch.o http-walker.o remote-curl.o transport.o walker.o: walker.h
-http.o http-walker.o http-push.o remote-curl.o: http.h
+http.o http-walker.o http-push.o http-fetch.o remote-curl.o: http.h
 
 xdiff-interface.o $(XDIFF_OBJS): \
 	xdiff/xinclude.h xdiff/xmacros.h xdiff/xdiff.h xdiff/xtypes.h \
-- 
1.7.2.1.544.ga752d.dirty
Previous: Jeff KingNext: Jonathan Nieder
Message 4 of 6 in “add missing dependencies on headers”
  1. 0/2 add missing dependencies on headersJonathan Nieder, Aug 8, 2010
  2. 1/2 Makefile: add missing dependencies on url.hJonathan Nieder, Aug 8, 2010
  3. Jeff KingAug 8, 2010
  4. 2/2 Makefile: add missing dependency on http.hJonathan Nieder, Aug 8, 2010
  5. 15/12 Makefile: check for unnecessary dependencies with CHECK...Jonathan Nieder, Aug 8, 2010
  6. Tay Ray ChuanAug 9, 2010

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.