[PATCH] dev-install: create the 'install' link in the builddir not i...

[Norbert Thiebaud (via Code Review)]

Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/3887

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/87/3887/1

dev-install: create the 'install' link in the builddir not in the srcdir

Change-Id: I9d1e7ac746928c2a793ac05d932af60d0c8edc06
---
M Makefile.in
1 file changed, 5 insertions(+), 4 deletions(-)



diff --git a/Makefile.in b/Makefile.in
index 68ac28c..26a00ad 100644
--- a/Makefile.in
+++ b/Makefile.in
@@ -316,7 +316,7 @@
 	@ooinstall -l $(DEVINSTALLDIR)/opt
 endif
 endif
-	@rm -f $(SRCDIR)/install && ln -s $(DEVINSTALLDIR)/opt/ $(SRCDIR)/install
+	@rm -f $(BUILDDIR)/install && ln -s $(DEVINSTALLDIR)/opt/ $(BUILDDIR)/install
 	@echo
 	@echo "If you want to edit the .ui files with glade first execute:"
 	@echo
@@ -325,12 +325,12 @@
 	@echo
 	@echo "Developer installation finished, you can now execute:"
 	@echo
-	@echo "$(SRCDIR)/install/program/soffice"
+	@echo "$(BUILDDIR)/install/program/soffice"
 else ifeq ($(OS),MACOSX)
 	@echo
 	@echo "Developer installation finished, you can now run:"
 	@echo
-	@echo "  open $(SRCDIR)/install/LibreOffice.app"
+	@echo "  open $(BUILDDIR)/install/LibreOffice.app"
 	@echo
 	@echo "To debug: gdb install/LibreOffice.app/Contents/MacOS/soffice"
 endif
@@ -338,6 +338,7 @@
 dev-install-nocheck: build-nocheck
 	$(MAKE) dev-install -o build
 
+# FIXME: shouldn't linkoo be BUILDDIR vs SRCDIR aware ?
 dev-update:
 ifeq ($(DISABLE_LINKOO),TRUE)
 	@linkoo --copy $(DEVINSTALLDIR)/opt $(SRCDIR)
@@ -354,7 +355,7 @@
 	@ooinstall $(DEVINSTALLDIR)/opt
 	@install-gdb-printers
 endif
-	@rm -f $(SRCDIR)/install && ln -s $(DEVINSTALLDIR)/opt/ $(SRCDIR)/install
+	@rm -f $(BUILDDIR)/install && ln -s $(DEVINSTALLDIR)/opt/ $(BUILDDIR)/install
 
 distro-pack-install: install
 	$(SRCDIR)/bin/distro-install-clean-up

-- 
To view, visit https://gerrit.libreoffice.org/3887
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I9d1e7ac746928c2a793ac05d932af60d0c8edc06
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Norbert Thiebaud <nthiebaud at gmail.com>