* [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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox