public inbox for crossgcc@sourceware.org
 help / color / mirror / Atom feed
* [PATCH] scripts: unset LIBRARY_PATH and LPATH
@ 2012-05-13  0:21 Erik Inge Bolsø
  2012-09-25 17:54 ` Erik Inge Bolsø
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Erik Inge Bolsø @ 2012-05-13  0:21 UTC (permalink / raw)
  To: crossgcc

[-- Attachment #1: Type: TEXT/PLAIN, Size: 641 bytes --]

Fooling around with crosstool-ng hg version, I happened to get gcc 
configure to yell at me right off. Promising.

The breakage is in build-cc-core-pass-2, saying
[ERROR]    configure: error: 
[CFG  ]    *** LIBRARY_PATH shouldn't contain the current directory when
[CFG  ]    *** building gcc. Please change the environment variable
[CFG  ]    *** and run configure again.

I figure this could bite others, and I see no good reason for LIBRARY_PATH 
to be set at this point.

The reason I had them set in the first place was due to some nvidia OpenCL 
libraries.

Patch attached to unset these two.

--
-erik	http://useofwords.blogspot.com/

[-- Attachment #2: Type: TEXT/PLAIN, Size: 1165 bytes --]

# HG changeset patch
# User Erik Inge Bolsø <knan-ct@anduin.net>
# Date 1336864160 -7200
# Node ID 4cc441b115867faaf37b89629209ebde734e3b06
# Parent  c800b7d6a50c267c3e0a64829da4b082e260deff
scripts: unset LIBRARY_PATH and LPATH

These environment variables set search path for gcc at link time, which can break the build.

Signed-off-by: Erik Inge Bolsø <knan-ct@anduin.net>

diff --git a/scripts/crosstool-NG.sh.in b/scripts/crosstool-NG.sh.in
--- a/scripts/crosstool-NG.sh.in
+++ b/scripts/crosstool-NG.sh.in
@@ -39,6 +39,10 @@
 CT_TestAndAbort "Don't set LD_LIBRARY_PATH. It screws up the build." -n "${LD_LIBRARY_PATH}"
 CT_TestAndAbort "Don't set CFLAGS. It screws up the build." -n "${CFLAGS}"
 CT_TestAndAbort "Don't set CXXFLAGS. It screws up the build." -n "${CXXFLAGS}"
+CT_Test "LIBRARY_PATH screws up the build. Resetting." -n "${LIBRARY_PATH}"
+export LIBRARY_PATH=
+CT_Test "LPATH screws up the build. Resetting." -n "$LPATH"
+export LPATH=
 CT_Test "GREP_OPTIONS screws up the build. Resetting." -n "${GREP_OPTIONS}"
 export GREP_OPTIONS=
 # Workaround against openSUSE 12.1 that breaks ./configure for cross-compilation:

[-- Attachment #3: Type: text/plain, Size: 71 bytes --]

--
For unsubscribe information see http://sourceware.org/lists.html#faq

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

* [PATCH] scripts: unset LIBRARY_PATH and LPATH
  2012-05-13  0:21 [PATCH] scripts: unset LIBRARY_PATH and LPATH Erik Inge Bolsø
@ 2012-09-25 17:54 ` Erik Inge Bolsø
  2012-09-25 20:46 ` Yann E. MORIN
  2012-09-25 21:04 ` Yann E. MORIN
  2 siblings, 0 replies; 4+ messages in thread
From: Erik Inge Bolsø @ 2012-09-25 17:54 UTC (permalink / raw)
  To: crossgcc

[-- Attachment #1: Type: TEXT/PLAIN, Size: 641 bytes --]

Fooling around with crosstool-ng hg version, I happened to get gcc 
configure to yell at me right off. Promising.

The breakage is in build-cc-core-pass-2, saying
[ERROR]    configure: error: 
[CFG  ]    *** LIBRARY_PATH shouldn't contain the current directory when
[CFG  ]    *** building gcc. Please change the environment variable
[CFG  ]    *** and run configure again.

I figure this could bite others, and I see no good reason for LIBRARY_PATH 
to be set at this point.

The reason I had them set in the first place was due to some nvidia OpenCL 
libraries.

Patch attached to unset these two.

--
-erik	http://useofwords.blogspot.com/

[-- Attachment #2: Type: TEXT/PLAIN, Size: 1165 bytes --]

# HG changeset patch
# User Erik Inge Bolsø <knan-ct@anduin.net>
# Date 1336864160 -7200
# Node ID 4cc441b115867faaf37b89629209ebde734e3b06
# Parent  c800b7d6a50c267c3e0a64829da4b082e260deff
scripts: unset LIBRARY_PATH and LPATH

These environment variables set search path for gcc at link time, which can break the build.

Signed-off-by: Erik Inge Bolsø <knan-ct@anduin.net>

diff --git a/scripts/crosstool-NG.sh.in b/scripts/crosstool-NG.sh.in
--- a/scripts/crosstool-NG.sh.in
+++ b/scripts/crosstool-NG.sh.in
@@ -39,6 +39,10 @@
 CT_TestAndAbort "Don't set LD_LIBRARY_PATH. It screws up the build." -n "${LD_LIBRARY_PATH}"
 CT_TestAndAbort "Don't set CFLAGS. It screws up the build." -n "${CFLAGS}"
 CT_TestAndAbort "Don't set CXXFLAGS. It screws up the build." -n "${CXXFLAGS}"
+CT_Test "LIBRARY_PATH screws up the build. Resetting." -n "${LIBRARY_PATH}"
+export LIBRARY_PATH=
+CT_Test "LPATH screws up the build. Resetting." -n "$LPATH"
+export LPATH=
 CT_Test "GREP_OPTIONS screws up the build. Resetting." -n "${GREP_OPTIONS}"
 export GREP_OPTIONS=
 # Workaround against openSUSE 12.1 that breaks ./configure for cross-compilation:

[-- Attachment #3: Type: text/plain, Size: 71 bytes --]

--
For unsubscribe information see http://sourceware.org/lists.html#faq

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

* scripts: unset LIBRARY_PATH and LPATH
  2012-05-13  0:21 [PATCH] scripts: unset LIBRARY_PATH and LPATH Erik Inge Bolsø
  2012-09-25 17:54 ` Erik Inge Bolsø
@ 2012-09-25 20:46 ` Yann E. MORIN
  2012-09-25 21:04 ` Yann E. MORIN
  2 siblings, 0 replies; 4+ messages in thread
From: Yann E. MORIN @ 2012-09-25 20:46 UTC (permalink / raw)
  To: Erik Inge Bolsø; +Cc: crossgcc

Erik, All,

Your patch:
    scripts: unset LIBRARY_PATH and LPATH

has been applied as #e43643e41fb9. Thank you!

Regards,
Yann E. MORIN.



--
For unsubscribe information see http://sourceware.org/lists.html#faq

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

* Re: scripts: unset LIBRARY_PATH and LPATH
  2012-05-13  0:21 [PATCH] scripts: unset LIBRARY_PATH and LPATH Erik Inge Bolsø
  2012-09-25 17:54 ` Erik Inge Bolsø
  2012-09-25 20:46 ` Yann E. MORIN
@ 2012-09-25 21:04 ` Yann E. MORIN
  2 siblings, 0 replies; 4+ messages in thread
From: Yann E. MORIN @ 2012-09-25 21:04 UTC (permalink / raw)
  To: crossgcc; +Cc: Erik Inge Bolsø

Erik, All,

On Tuesday 25 September 2012 22:45:47 Yann E. MORIN wrote:
> Your patch:
>     scripts: unset LIBRARY_PATH and LPATH
> 
> has been applied as #e43643e41fb9. Thank you!

Sorry it took so long to apply your patch.

Next time, would you please use 'hg email' to send your patches, that will
send the patches in-line so it's easier to review and catch my eye).
There a tuto on how to do that in: docs/C - Misc. tutorials.txt.

Thank you!

Regards,
Yann E. MORIN.

PS. It seems my send-an-email-for-applied-patches script is now working! :-)
YEM.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

--
For unsubscribe information see http://sourceware.org/lists.html#faq

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

end of thread, other threads:[~2012-09-25 21:04 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-05-13  0:21 [PATCH] scripts: unset LIBRARY_PATH and LPATH Erik Inge Bolsø
2012-09-25 17:54 ` Erik Inge Bolsø
2012-09-25 20:46 ` Yann E. MORIN
2012-09-25 21:04 ` Yann E. MORIN

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).