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

[PATCH 1/3] git-remote-mediawiki: honor DESTDIR in "make install"

From
Jonathan Nieder <jrnieder@gmail.com>
Date
Nov 11, 2013, 20:45 UTC
Message-ID
<20131111204529.GN10302@google.com>
In-Reply-To
<20131111204504.GM10302@google.com>
So now you can run
	DESTDIR=$(pwd)/tmp make -Ccontrib/mw-to-git install

to install the mediawiki remote helper, git-mw tool, and Git::Mediawiki perl module under tmp/ as preparation for zipping it up and extracting on another machine.

While at it, make sure the directory that should contain Git::Mediawiki exists before putting a file there. Without this patch, the makefile uses DESTDIR when installing git-mw and git-remote-mediawiki but not the perl module, resulting in errors from "make install" if the $(INSTLIBDIR)/Git directory does not exist:

 install: cannot create regular file \
 '/usr/share/perl/5.18.1/Git/Mediawiki.pm': No such file or directory
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
---
 contrib/mw-to-git/Makefile | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/contrib/mw-to-git/Makefile b/contrib/mw-to-git/Makefile
index c5547f9..ee78fda 100644
--- a/contrib/mw-to-git/Makefile
+++ b/contrib/mw-to-git/Makefile
@@ -30,7 +30,9 @@ test: all
 check: perlcritic test
 
 install_pm:
-	install $(GIT_MEDIAWIKI_PM) $(INSTLIBDIR)/$(GIT_MEDIAWIKI_PM)
+	install -d -m 755 $(DESTDIR)$(INSTLIBDIR)/Git
+	install -m 644 $(GIT_MEDIAWIKI_PM) \
+		$(DESTDIR)$(INSTLIBDIR)/$(GIT_MEDIAWIKI_PM)
 
 build:
 	$(MAKE) -C $(GIT_ROOT_DIR) SCRIPT_PERL="$(SCRIPT_PERL_FULL)" \
-- 
1.8.4.1
Previous: Jonathan NiederNext: Jonathan Nieder
Message 4 of 7 in “git-remote-mediawiki: do not remove installed files in "clean" target”
  1. git-remote-mediawiki: do not remove installed files in "clean" targetJonathan Nieder, Nov 9, 2013
  2. Matthieu MoyNov 9, 2013
  3. Jonathan NiederNov 11, 2013
  4. 1/3 git-remote-mediawiki: honor DESTDIR in "make install"Jonathan Nieder, Nov 11, 2013
  5. 2/3 git-remote-mediawiki build: make 'install' command configurableJonathan Nieder, Nov 11, 2013
  6. 3/3 git-remote-mediawiki build: handle DESTDIR/INSTLIBDIR with whitespaceJonathan Nieder, Nov 11, 2013
  7. Matthieu MoyNov 11, 2013

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.