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

[RFCv4 6/5] Fix the Makefile-generated path to the git_remote_cvs package in git-remote-cvs

From
Johan Herland <johan@herland.net>
Date
Aug 17, 2009, 16:27 UTC
Message-ID
<200908171827.34370.johan@herland.net>
In-Reply-To
<1250480161-21933-1-git-send-email-johan@herland.net>

Junio discovered that the installed git-remote-cvs executable is unable to find the installed git_remote_cvs Python package. This is due to the 'instlibdir' target of git_remote_cvs/Makefile being invoked with insufficient arguments (prefix and DESTDIR). This patch fixes that.

Signed-off-by: Johan Herland <johan@herland.net>
---
Sorry for the inconvenience.
...Johan
 Makefile                |    2 +-
 git_remote_cvs/Makefile |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/Makefile b/Makefile
index b2af678..b9a7f25 100644
--- a/Makefile
+++ b/Makefile
@@ -1479,7 +1479,7 @@ endif # NO_PERL
 ifndef NO_PYTHON
 $(patsubst %.py,%,$(SCRIPT_PYTHON)): % : %.py
 	$(QUIET_GEN)$(RM) $@ $@+ && \
-	INSTLIBDIR=`MAKEFLAGS= $(MAKE) -C git_remote_cvs -s --no-print-directory instlibdir` && \
+	INSTLIBDIR=`MAKEFLAGS= $(MAKE) -C git_remote_cvs -s --no-print-directory prefix='$(prefix_SQ)' DESTDIR='$(DESTDIR_SQ)' instlibdir` && \
 	sed -e '1{' \
 	    -e '	s|#!.*python|#!$(PYTHON_PATH_SQ)|' \
 	    -e '}' \
diff --git a/git_remote_cvs/Makefile b/git_remote_cvs/Makefile
index 061c247..0d9eb31 100644
--- a/git_remote_cvs/Makefile
+++ b/git_remote_cvs/Makefile
@@ -28,7 +28,7 @@ install: $(pysetupfile)
 	$(PYTHON_PATH) $(pysetupfile) install --prefix $(DESTDIR_SQ)$(prefix)
 
 instlibdir: $(pysetupfile)
-	@echo "$(prefix)/$(PYLIBDIR)"
+	@echo "$(DESTDIR_SQ)$(prefix)/$(PYLIBDIR)"
 
 clean:
 	$(QUIET)$(PYTHON_PATH) $(pysetupfile) $(QUIETSETUP) clean -a
-- 
1.6.4.313.g38b9.dirty


-- 
Johan Herland, <johan@herland.net>
www.herland.net
Previous: Johan HerlandNext: Johan Herland
Message 11 of 12 in “[RFCv4 0/5] CVS remote helper”
  1. Johan HerlandAug 17, 2009
  2. 1/5 Basic build infrastructure for Python scriptsJohan Herland, Aug 17, 2009
  3. 2/5 1/2: Add Python support library for CVS remote helperJohan Herland, Aug 17, 2009
  4. 3/5 2/2: Add Python support library for CVS remote helperJohan Herland, Aug 17, 2009
  5. Sverre RabbelierOct 5, 2009
  6. Johan HerlandOct 5, 2009
  7. Sverre RabbelierOct 5, 2009
  8. Johan HerlandOct 5, 2009
  9. 4/5 git-remote-cvs: Remote helper program for CVS repositoriesJohan Herland, Aug 17, 2009
  10. 5/5 Add simple selftests of git-remote-cvs functionalityJohan Herland, Aug 17, 2009
  11. 6/5 Fix the Makefile-generated path to the git_remote_cvs package in git-remote-cvsJohan Herland, Aug 17, 2009
  12. 7/5 More fixes to the git-remote-cvs installation procedureJohan Herland, Aug 18, 2009

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.