threads / patch / 17703

patchfix the installation path for html documentation

Subject: [PATCH] fix the installation path for html documentation

## tl;dr

3 messages between Feb 10, 2009 and Feb 11, 2009. Diffs are folded; open one to read it.

replies: 2people: 3as markdown or json

Michael J Gruber· Feb 10, 2009, 15:14 UTC · lore

Since 026fa0d5ad9538ca76838070861531c037d7b9ba (Move computation of absolute paths from Makefile to runtime (in preparation for RUNTIME_PREFIX) the installation of html doc was broken: Documentation/Makefile does not know how to resolve relative dirs, which results in html doc being installed in $(git-build-root)/Documentation/share/doc/git-doc.

Fix this by not exporting htmldir from Makefile since this allows Documentation/Makefile to compute htmldir from prefix.

Signed-off-by: Michael J Gruber <git@drmicha.warpmail.net>
---
An alternative would be to teach Documentation/Makefile to take the
relative htmldir and make it into an absolute path. But the approach I'm
taking with this patch as analogous to how mandir is handled, with the
same draw back. But all is well as long as the build monkey messes with
prefix only.
 Makefile |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
Show changes to Makefile +1 −1
diff --git a/Makefile b/Makefile
index 27b9569..862b96c 100644
--- a/Makefile
+++ b/Makefile
@@ -228,7 +228,7 @@ GITWEB_FAVICON = git-favicon.png
 GITWEB_SITE_HEADER =
 GITWEB_SITE_FOOTER =
 
-export prefix bindir sharedir htmldir sysconfdir
+export prefix bindir sharedir sysconfdir
 
 CC = gcc
 AR = ar

-- 
1.6.1.2.253.ga34a
Johannes Sixt· Feb 10, 2009, 20:25 UTC · re: Michael J Gruber · lore

Re: [PATCH] fix the installation path for html documentation

On Dienstag, 10. Februar 2009, Michael J Gruber wrote:
Show 17 quoted lines
> Since 026fa0d5ad9538ca76838070861531c037d7b9ba (Move computation of
> absolute paths from Makefile to runtime (in preparation for
> RUNTIME_PREFIX) the installation of html doc was broken:
> Documentation/Makefile does not know how to resolve relative dirs, which
> results in html doc being installed in
> $(git-build-root)/Documentation/share/doc/git-doc.
>
> Fix this by not exporting htmldir from Makefile since this allows
> Documentation/Makefile to compute htmldir from prefix.
>
> Signed-off-by: Michael J Gruber <git@drmicha.warpmail.net>
> ---
> An alternative would be to teach Documentation/Makefile to take the
> relative htmldir and make it into an absolute path. But the approach I'm
> taking with this patch as analogous to how mandir is handled, with the
> same draw back. But all is well as long as the build monkey messes with
> prefix only.
Makes sense. There is no point handling mandir and htmldir differently.
-- Hannes

← back to recent threads