All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/2] Revert "dependencies: ensure that DESTDIR isn't set when running Buildroot"
@ 2012-07-02 14:21 Thomas Petazzoni
  2012-07-02 14:21 ` [Buildroot] [PATCH 2/2] Ensure DESTDIR will not confuse the build Thomas Petazzoni
  2012-07-02 18:54 ` [Buildroot] [PATCH 1/2] Revert "dependencies: ensure that DESTDIR isn't set when running Buildroot" Peter Korsgaard
  0 siblings, 2 replies; 3+ messages in thread
From: Thomas Petazzoni @ 2012-07-02 14:21 UTC (permalink / raw)
  To: buildroot

We should instead simply unset it at runtime, like we do for
PKG_CONFIG_PATH.

This reverts commit 9910eba33adb2b783b0df5d90a857816e82fbd55.
---
 support/dependencies/dependencies.sh |    6 ------
 1 file changed, 6 deletions(-)

diff --git a/support/dependencies/dependencies.sh b/support/dependencies/dependencies.sh
index 43ec3cc..c47ffcf 100755
--- a/support/dependencies/dependencies.sh
+++ b/support/dependencies/dependencies.sh
@@ -48,12 +48,6 @@ if test -n "$PERL_MM_OPT" ; then
     exit 1
 fi
 
-if test -n "$DESTDIR" ; then
-    /bin/echo -e "\nYou have the DESTDIR environment variable set. Please"
-    /bin/echo -e "unset it so that Buildroot can work properly."
-    exit 1
-fi
-
 # Verify that which is installed
 if ! which which > /dev/null ; then
 	/bin/echo -e "\nYou must install 'which' on your build machine\n";
-- 
1.7.9.5

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [Buildroot] [PATCH 2/2] Ensure DESTDIR will not confuse the build
  2012-07-02 14:21 [Buildroot] [PATCH 1/2] Revert "dependencies: ensure that DESTDIR isn't set when running Buildroot" Thomas Petazzoni
@ 2012-07-02 14:21 ` Thomas Petazzoni
  2012-07-02 18:54 ` [Buildroot] [PATCH 1/2] Revert "dependencies: ensure that DESTDIR isn't set when running Buildroot" Peter Korsgaard
  1 sibling, 0 replies; 3+ messages in thread
From: Thomas Petazzoni @ 2012-07-02 14:21 UTC (permalink / raw)
  To: buildroot

Having DESTDIR defined will confuse the build of certain packages, so
we ensure that it is undefined from the environment when Buildroot
starts.

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 Makefile |    4 ++++
 1 file changed, 4 insertions(+)

diff --git a/Makefile b/Makefile
index bcbe9fc..b80477f 100644
--- a/Makefile
+++ b/Makefile
@@ -168,6 +168,10 @@ export HOSTCC_NOCCACHE HOSTCXX_NOCCACHE
 # Make sure pkg-config doesn't look outside the buildroot tree
 unexport PKG_CONFIG_PATH
 
+# Having DESTDIR set in the environment confuses the installation
+# steps of some packages.
+unexport DESTDIR
+
 # bash prints the name of the directory on 'cd <dir>' if CDPATH is
 # set, so unset it here to not cause problems. Notice that the export
 # line doesn't affect the environment of $(shell ..) calls, so
-- 
1.7.9.5

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [Buildroot] [PATCH 1/2] Revert "dependencies: ensure that DESTDIR isn't set when running Buildroot"
  2012-07-02 14:21 [Buildroot] [PATCH 1/2] Revert "dependencies: ensure that DESTDIR isn't set when running Buildroot" Thomas Petazzoni
  2012-07-02 14:21 ` [Buildroot] [PATCH 2/2] Ensure DESTDIR will not confuse the build Thomas Petazzoni
@ 2012-07-02 18:54 ` Peter Korsgaard
  1 sibling, 0 replies; 3+ messages in thread
From: Peter Korsgaard @ 2012-07-02 18:54 UTC (permalink / raw)
  To: buildroot

>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@free-electrons.com> writes:

 Thomas> We should instead simply unset it at runtime, like we do for
 Thomas> PKG_CONFIG_PATH.

Committed both, thanks.

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2012-07-02 18:54 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-07-02 14:21 [Buildroot] [PATCH 1/2] Revert "dependencies: ensure that DESTDIR isn't set when running Buildroot" Thomas Petazzoni
2012-07-02 14:21 ` [Buildroot] [PATCH 2/2] Ensure DESTDIR will not confuse the build Thomas Petazzoni
2012-07-02 18:54 ` [Buildroot] [PATCH 1/2] Revert "dependencies: ensure that DESTDIR isn't set when running Buildroot" Peter Korsgaard

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.