public inbox for crossgcc@sourceware.org
 help / color / mirror / Atom feed
* [PATCH 0 of 1] Use CT_CONFIG_SHELL instead of CT_SHELL
@ 2011-11-30 11:14 Titus von Boxberg
  2011-11-30 11:14 ` [PATCH 1 of 1] scripts/crosstool-NG: " Titus von Boxberg
  0 siblings, 1 reply; 3+ messages in thread
From: Titus von Boxberg @ 2011-11-30 11:14 UTC (permalink / raw)
  To: crossgcc

Yann, all,

crosstool-NG.sh uses CT_SHELL which is undefined.
It seems to be a relic.
I have replaced it by CT_CONFIG_SHELL.

Regards
Titus

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

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

* [PATCH 1 of 1] scripts/crosstool-NG: Use CT_CONFIG_SHELL instead of CT_SHELL
  2011-11-30 11:14 [PATCH 0 of 1] Use CT_CONFIG_SHELL instead of CT_SHELL Titus von Boxberg
@ 2011-11-30 11:14 ` Titus von Boxberg
  2011-11-30 23:05   ` Yann E. MORIN
  0 siblings, 1 reply; 3+ messages in thread
From: Titus von Boxberg @ 2011-11-30 11:14 UTC (permalink / raw)
  To: crossgcc

# HG changeset patch
# User Titus von Boxberg <titus@v9g.de>
# Date 1322651279 -3600
# Node ID 24eba4b323db812398fd364a626fcc3b7c867dfb
# Parent  f6adc331f779970b1fc8f30ccf22e1aa52dc1d9a
scripts/crosstool-NG: Use CT_CONFIG_SHELL instead of CT_SHELL

CT_SHELL is undefined.
Thus, the generated wrapper scripts are not executable by the kernel
because they do not contain a valid interpreter.
Use CT_CONFIG_SHELL instead.

Signed-off-by: "Titus von Boxberg" <titus@v9g.de>

diff -r f6adc331f779 -r 24eba4b323db scripts/crosstool-NG.sh.in
--- a/scripts/crosstool-NG.sh.in	Tue Nov 29 10:02:33 2011 +0100
+++ b/scripts/crosstool-NG.sh.in	Wed Nov 30 12:07:59 2011 +0100
@@ -80,7 +80,7 @@
     tool="${line%%=*}"
     path="${line#*=}"
     CT_DoLog DEBUG "Creating script-override for '${tool}' -> '${path}'"
-    printf "#${BANG}${CT_SHELL}\nexec '${path}' \"\${@}\"\n" >"${CT_TOOLS_OVERIDE_DIR}/bin/${tool}"
+    printf "#${BANG}${CT_CONFIG_SHELL}\nexec '${path}' \"\${@}\"\n" >"${CT_TOOLS_OVERIDE_DIR}/bin/${tool}"
     CT_DoExecLog ALL chmod 700 "${CT_TOOLS_OVERIDE_DIR}/bin/${tool}"
 done
 export PATH="${CT_TOOLS_OVERIDE_DIR}/bin:${PATH}"
@@ -414,7 +414,7 @@
             # bally needed
             if [ -n "${where}" ]; then
                 CT_DoLog DEBUG "  '${!v}-${tool}' -> '${where}'"
-                printf "#${BANG}${CT_SHELL}\nexec '${where}' \"\${@}\"\n" >"${CT_BUILDTOOLS_PREFIX_DIR}/bin/${!v}-${tool}"
+                printf "#${BANG}${CT_CONFIG_SHELL}\nexec '${where}' \"\${@}\"\n" >"${CT_BUILDTOOLS_PREFIX_DIR}/bin/${!v}-${tool}"
                 CT_DoExecLog ALL chmod 700 "${CT_BUILDTOOLS_PREFIX_DIR}/bin/${!v}-${tool}"
             else
                 case "${tool}" in
@@ -448,7 +448,7 @@
     # Go ahead with those, by creating a wrapper that keeps partial files, and that
     # never fails:
     CT_DoLog DEBUG "  'makeinfo' -> '$(CT_Which makeinfo)'"
-    printf "#${BANG}${CT_SHELL}\n$(CT_Which makeinfo) --force \"\${@}\"\ntrue\n" >"${CT_BUILDTOOLS_PREFIX_DIR}/bin/makeinfo"
+    printf "#${BANG}${CT_CONFIG_SHELL}\n$(CT_Which makeinfo) --force \"\${@}\"\ntrue\n" >"${CT_BUILDTOOLS_PREFIX_DIR}/bin/makeinfo"
     CT_DoExecLog ALL chmod 700 "${CT_BUILDTOOLS_PREFIX_DIR}/bin/makeinfo"
 
     # Carefully add paths in the order we want them:

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

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

* Re: [PATCH 1 of 1] scripts/crosstool-NG: Use CT_CONFIG_SHELL instead of CT_SHELL
  2011-11-30 11:14 ` [PATCH 1 of 1] scripts/crosstool-NG: " Titus von Boxberg
@ 2011-11-30 23:05   ` Yann E. MORIN
  0 siblings, 0 replies; 3+ messages in thread
From: Yann E. MORIN @ 2011-11-30 23:05 UTC (permalink / raw)
  To: crossgcc; +Cc: Titus von Boxberg

Titus, All,

On Wednesday 30 November 2011 12:13:20 Titus von Boxberg wrote:
> # HG changeset patch
> # User Titus von Boxberg <titus@v9g.de>
> # Date 1322651279 -3600
> # Node ID 24eba4b323db812398fd364a626fcc3b7c867dfb
> # Parent  f6adc331f779970b1fc8f30ccf22e1aa52dc1d9a
> scripts/crosstool-NG: Use CT_CONFIG_SHELL instead of CT_SHELL

Wooa...

That one has been there for almost two months now, and only shows up now...
Nice catch!

Applied as #cf509170838f, thank you!

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  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] 3+ messages in thread

end of thread, other threads:[~2011-11-30 23:05 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-11-30 11:14 [PATCH 0 of 1] Use CT_CONFIG_SHELL instead of CT_SHELL Titus von Boxberg
2011-11-30 11:14 ` [PATCH 1 of 1] scripts/crosstool-NG: " Titus von Boxberg
2011-11-30 23:05   ` 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).