about summary refs log tree commit diff
diff options
context:
space:
mode:
authorLars Hjemli <hjemli@gmail.com>2008-08-01 01:25:51 +0200
committerLars Hjemli <hjemli@gmail.com>2008-08-01 02:04:41 +0200
commita1266edfe6ec1fd678c8f6b60ffbd21d88ff5a93 (patch)
treeda90c4ae217fd171a28b4ce85dd38cc2db09c7ea
parentModify default value for a few cgitrc options (diff)
downloadcgit-pink-a1266edfe6ec1fd678c8f6b60ffbd21d88ff5a93.tar.gz
cgit-pink-a1266edfe6ec1fd678c8f6b60ffbd21d88ff5a93.zip
Makefile: another take on git dependency rules
When building cgit we depend on xdiff/lib.a and libgit.a in the git
directory, but the previous attempt on describing this dependency
failed since the build instructions for the libs was placed under the
phony `git` target.

This patch fixes the issue by moving the build instructions to their
real targets. It also makes it clear that only the `cgit` target
depends on the git binaries (since they're only used during linking).

And while at it, the patch also cleans up the list of phony targets.

Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Diffstat (limited to '')
-rw-r--r--Makefile12
1 files changed, 5 insertions, 7 deletions
diff --git a/Makefile b/Makefile
index d83a46a..e1436a3 100644
--- a/Makefile
+++ b/Makefile
@@ -73,7 +73,7 @@ ifdef NEEDS_LIBICONV
 endif
 
 
-.PHONY: all git test install clean distclean emptycache force-version get-git
+.PHONY: all git test install uninstall clean force-version get-git
 
 all: cgit
 
@@ -90,20 +90,18 @@ CFLAGS += -DCGIT_SCRIPT_NAME='"$(CGIT_SCRIPT_NAME)"'
 CFLAGS += -DCGIT_CACHE_ROOT='"$(CACHE_ROOT)"'
 
 
-cgit: $(OBJECTS)
+cgit: $(OBJECTS) git/libgit.a git/xdiff/lib.a
 	$(QUIET_CC)$(CC) $(CFLAGS) -o cgit $(OBJECTS) $(EXTLIBS)
 
-$(OBJECTS): | git/xdiff/lib.a git/libgit.a
-
 cgit.o: VERSION
 
 -include $(OBJECTS:.o=.d)
 
-git/xdiff/lib.a, git/libgit.a: git
+git/libgit.a: git
+	$(QUIET_SUBDIR0)git $(QUIET_SUBDIR1) libgit.a
 
-git:
+git/xdiff/lib.a: git
 	$(QUIET_SUBDIR0)git $(QUIET_SUBDIR1) xdiff/lib.a
-	$(QUIET_SUBDIR0)git $(QUIET_SUBDIR1) libgit.a
 
 test: all
 	$(QUIET_SUBDIR0)tests $(QUIET_SUBDIR1) all