threads / patch / 9789

patch, 3 partsMakefile: Add cache-tree.h to the public headers list

Subject: [PATCH 3/3] Makefile: Add cache-tree.h to the public headers list

## tl;dr

3 messages between Sep 5, 2007 and Sep 6, 2007. Diffs are folded; open one to read it.

replies: 2people: 2as markdown or json

Dmitry V. Levin· Sep 5, 2007, 23:22 UTC · lore
Some external projects (e.g. parsecvs) need cache-tree.h file.
Signed-off-by: Dmitry V. Levin <ldv@altlinux.org>
---
 Makefile |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
Show changes to Makefile +1 −1
diff --git a/Makefile b/Makefile
index d50e30b..2b04829 100644
--- a/Makefile
+++ b/Makefile
@@ -283,7 +283,7 @@ LIB_FILE=libgit.a
 XDIFF_LIB=xdiff/lib.a
 
 LIB_H = \
-	archive.h blob.h cache.h commit.h csum-file.h delta.h grep.h \
+	archive.h blob.h cache.h cache-tree.h commit.h csum-file.h delta.h grep.h \
 	diff.h object.h pack.h pkt-line.h quote.h refs.h list-objects.h sideband.h \
 	run-command.h strbuf.h tag.h tree.h git-compat-util.h revision.h \
 	tree-walk.h log-tree.h dir.h path-list.h unpack-trees.h builtin.h \
-- 
ldv
Junio C Hamano· Sep 6, 2007, 00:16 UTC · re: Dmitry V. Levin · lore

Re: [PATCH 3/3] Makefile: Add cache-tree.h to the public headers list

"Dmitry V. Levin" <ldv@altlinux.org> writes:
> Some external projects (e.g. parsecvs) need cache-tree.h file.

Your patch shows that our Makefile has been lacking a necessary dependency for a long time. LIB_H is not "public headers list", but more like "the headers everybody depends on", and the header file should have been on that list. Thanks.

I am however not convinced it is a good idea to treat libgit.a as if it is a library. It is not a library in the usual sense of the word. Originally we did libgit.a primarily so that we do not have to list all the *.o dependencies in the Makefile out of laziness ;-)

Dmitry V. Levin· Sep 6, 2007, 00:37 UTC · re: Junio C Hamano · lore

Re: [PATCH 3/3] Makefile: Add cache-tree.h to the public headers list

On Wed, Sep 05, 2007 at 05:16:47PM -0700, Junio C Hamano wrote: [...]

> I am however not convinced it is a good idea to treat libgit.a
> as if it is a library.
> It is not a library in the usual sense of the word.

I'd certainly prefer libgit.so with stable interface instead, but parsecvs needs -lgit now, it cannot wait for "good library".

-- 
ldv

← back to recent threads