{"thread":{"id":"6819","subject":"[PATCH] add install-lib target to install libgit.a library plus headers.","startedAt":"2007-02-14T14:57:25Z","lastAt":"2007-02-14T18:24:09Z","messageCount":3,"participants":["Gerrit Pape","Johannes Schindelin","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"34535","messageId":"20070214145725.2827.qmail@0b3a4e2ad592dc.315fe32.mid.smarden.org","threadId":"6819","inReplyTo":null,"subject":"[PATCH] add install-lib target to install libgit.a library plus headers.","fromName":"Gerrit Pape","fromEmail":"pape@smarden.org","sentAt":"2007-02-14T14:57:25Z","receivedAt":"2007-02-14T14:57:25Z","isPatch":true,"sender":{"key":"pape@smarden.org","avatar":"https://avatars.githubusercontent.com/u/143170252?v=4"},"body":"Patch from Luca Falavigna to install git static library and header files\ninto $(prefix)/lib/, $(prefix)/include/git/, see\n\n http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=407722\n\nSigned-off-by: Gerrit Pape <pape@smarden.org>\n---\n Makefile |   10 +++++++++-\n 1 files changed, 9 insertions(+), 1 deletions(-)\n\ndiff --git a/Makefile b/Makefile\nindex ebecbbd..19fe3d5 100644\n--- a/Makefile\n+++ b/Makefile\n@@ -123,6 +123,8 @@ STRIP ?= strip\n prefix = $(HOME)\n bindir = $(prefix)/bin\n gitexecdir = $(bindir)\n+headerdir = $(prefix)/include/git\n+libdir = $(prefix)/lib\n template_dir = $(prefix)/share/git-core/templates/\n # DESTDIR=\n \n@@ -588,6 +590,8 @@ SHA1_HEADER_SQ = $(subst ','\\'',$(SHA1_HEADER))\n DESTDIR_SQ = $(subst ','\\'',$(DESTDIR))\n bindir_SQ = $(subst ','\\'',$(bindir))\n gitexecdir_SQ = $(subst ','\\'',$(gitexecdir))\n+headerdir_SQ = $(subst ','\\'',$(headerdir))\n+libdir_SQ = $(subst ','\\'',$(libdir))\n template_dir_SQ = $(subst ','\\'',$(template_dir))\n prefix_SQ = $(subst ','\\'',$(prefix))\n \n@@ -863,7 +867,11 @@ install-doc:\n quick-install-doc:\n \t$(MAKE) -C Documentation quick-install\n \n-\n+install-lib: $(LIB_FILE)\n+\t$(INSTALL) -d -m0755 '$(DESTDIR_SQ)$(headerdir_SQ)'\n+\t$(INSTALL) -d -m0755 '$(DESTDIR_SQ)$(libdir_SQ)'\n+\t$(INSTALL) -m0644 $(LIB_H) '$(DESTDIR_SQ)$(headerdir_SQ)'\n+\t$(INSTALL) -m0644 $(LIB_FILE) '$(DESTDIR_SQ)$(libdir_SQ)'\n \n ### Maintainer's dist rules\n \n-- \n1.5.0\n"},{"id":"34540","messageId":"Pine.LNX.4.63.0702141650400.22628@wbgn013.biozentrum.uni-wuerzburg.de","threadId":"6819","inReplyTo":"20070214145725.2827.qmail@0b3a4e2ad592dc.315fe32.mid.smarden.org","subject":"Re: [PATCH] add install-lib target to install libgit.a library plus headers.","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2007-02-14T15:51:36Z","receivedAt":"2007-02-14T15:51:36Z","isPatch":true,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi,\n\nOn Wed, 14 Feb 2007, Gerrit Pape wrote:\n\n> Patch from Luca Falavigna to install git static library and header files\n> into $(prefix)/lib/, $(prefix)/include/git/, see\n\nThis would make sense, _iff_ libgit.a was libified. As it is, too many \nfunctions assume that you can die() anytime something goes awry.\n\nCiao,\nDscho\n"},{"id":"34602","messageId":"7v4ppopph2.fsf@assigned-by-dhcp.cox.net","threadId":"6819","inReplyTo":"20070214145725.2827.qmail@0b3a4e2ad592dc.315fe32.mid.smarden.org","subject":"Re: [PATCH] add install-lib target to install libgit.a library plus headers.","fromName":"Junio C Hamano","fromEmail":"junkio@cox.net","sentAt":"2007-02-14T18:24:09Z","receivedAt":"2007-02-14T18:24:09Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Sorry, but libgit.a is not a publically usable/useful library at\nthis moment, as its internal API is not cast in stone yet.\n\nInstalling it in /usr/lib and /usr/include would give the false\nimpression to the users that linking with user's programs is\nsupported and upgrading git won't break them.  Please do not\ncreate libgit-dev package.\n"}]}