public inbox for crossgcc@sourceware.org
 help / color / mirror / Atom feed
* [PATCH 1 of 2] Add g++ to required tools
  2009-08-02 15:45 [PATCH 0 of 2] GMP build issues Bart vdr. Meulen
@ 2009-08-02 15:45 ` Bart vdr. Meulen
  2009-08-02 16:58   ` Yann E. MORIN
  2009-08-02 15:53 ` [PATCH 2 of 2] Add config shell override option Bart vdr. Meulen
  1 sibling, 1 reply; 5+ messages in thread
From: Bart vdr. Meulen @ 2009-08-02 15:45 UTC (permalink / raw)
  To: Yann E. MORIN; +Cc: crossgcc

# HG changeset patch
# User Bart vdr. Meulen <bartvdrmeulen@gmail.com>
# Date 1249224333 -7200
# Node ID f04c7879f3c2d38cc4aa317532c24395fc0e760a
# Parent  76f8feef64c455d4cd451ee5f3bf522f6a371f3d
Add g++ to required tools

Because gmp is now always build with --enable-cxx,
g++ has become a required tool

Signed-off-by: Bart vdr Meulen <bartvdrmeulen@gmail.com>

---

diff -r 76f8feef64c4 -r f04c7879f3c2 scripts/crosstool-NG.sh.in
--- a/scripts/crosstool-NG.sh.in	Sat Aug 01 19:18:12 2009 +0200
+++ b/scripts/crosstool-NG.sh.in	Sun Aug 02 16:45:33 2009 +0200
@@ -405,7 +405,7 @@
             else
                 case "${tool}" in
                     # We'll at least need some of them...
-                    ar|as|gcc|ld|nm|objcopy|objdump|ranlib)
+                    ar|as|gcc|g++|ld|nm|objcopy|objdump|ranlib)
                         CT_Abort "Missing: '${t}${tool}${!s}' or '${t}${tool}' or '${tool}' : either needed!"
                         ;;
                     # Some are conditionnally required



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

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

* [PATCH 0 of 2] GMP build issues
@ 2009-08-02 15:45 Bart vdr. Meulen
  2009-08-02 15:45 ` [PATCH 1 of 2] Add g++ to required tools Bart vdr. Meulen
  2009-08-02 15:53 ` [PATCH 2 of 2] Add config shell override option Bart vdr. Meulen
  0 siblings, 2 replies; 5+ messages in thread
From: Bart vdr. Meulen @ 2009-08-02 15:45 UTC (permalink / raw)
  To: Yann E. MORIN; +Cc: crossgcc


Two patches to fix somes issues I had when trying to build
GMP.

First one is to check wheter a C++ compiler is installed on the build system,
this is needed due to the --enable-cxx flag.

Second one it to override the shell for config scripts, usefull on ubuntu systems
where the user does not want to change the default shell script.
Note I posted a draft of this patch already friday in response to question
about build problems with gmp




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

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

* [PATCH 2 of 2] Add config shell override option
  2009-08-02 15:45 [PATCH 0 of 2] GMP build issues Bart vdr. Meulen
  2009-08-02 15:45 ` [PATCH 1 of 2] Add g++ to required tools Bart vdr. Meulen
@ 2009-08-02 15:53 ` Bart vdr. Meulen
  2009-08-02 17:03   ` Yann E. MORIN
  1 sibling, 1 reply; 5+ messages in thread
From: Bart vdr. Meulen @ 2009-08-02 15:53 UTC (permalink / raw)
  To: Yann E. MORIN; +Cc: crossgcc

# HG changeset patch
# User Bart vdr. Meulen <bartvdrmeulen@gmail.com>
# Date 1249224337 -7200
# Node ID 40e69fbd288ec6cd9bfaaee1228212c1a8169e3c
# Parent  f04c7879f3c2d38cc4aa317532c24395fc0e760a
Add config shell override option

Add config option to override the shell used in configuration
script making it possible to build on an Ubuntu host without
overriding the default shell (dash)

Signed-off-by: Bart vdr Meulen <bartvdrmeulen@gmail.com>

---

diff -r f04c7879f3c2 -r 40e69fbd288e config/global/build-behave.in
--- a/config/global/build-behave.in	Sun Aug 02 16:45:33 2009 +0200
+++ b/config/global/build-behave.in	Sun Aug 02 16:45:37 2009 +0200
@@ -56,4 +56,13 @@
       
       Saying 'y' here should be safe enough, but some components may not
       accept using a shell other than bash (this is most probably true for
-      old versions). If you have trouble, say 'n' here and see if it fixes it.
+
+config CONFIG_SHELL_OVERRIDE
+    string
+    prompt "Use alternative as CONFIG_SHELL"
+    help
+      Enter the path for the shell to be used by ./configure scripts,
+      instead of the default /bin/sh
+      This can be usefull on Ubuntu hosts where the default is dash
+      which does not work with some configuration scripts
+
diff -r f04c7879f3c2 -r 40e69fbd288e scripts/crosstool-NG.sh.in
--- a/scripts/crosstool-NG.sh.in	Sun Aug 02 16:45:33 2009 +0200
+++ b/scripts/crosstool-NG.sh.in	Sun Aug 02 16:45:37 2009 +0200
@@ -62,10 +62,10 @@
 
 # What's our shell?
 # Will be plain /bin/sh on most systems, except if we have /bin/ash and we
-# _explictly_ required using it
+# _explictly_ required using it or shell is overriden
 CT_SHELL="/bin/sh"
 [ "${CT_CONFIG_SHELL_ASH}" = "y" -a -x "/bin/ash" ] && CT_SHELL="/bin/ash"
-
+[ "${CT_CONFIG_SHELL_OVERRIDE}" != "" -a "${CT_CONFIG_SHELL_OVERRIDE}" ] && CT_SHELL="${CT_CONFIG_SHELL_OVERRIDE}" 
 # Create the bin-overide early
 # Contains symlinks to the tools found by ./configure
 # Note: CT_DoLog and CT_DoExecLog do not use any of those tool, so



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

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

* Re: [PATCH 1 of 2] Add g++ to required tools
  2009-08-02 15:45 ` [PATCH 1 of 2] Add g++ to required tools Bart vdr. Meulen
@ 2009-08-02 16:58   ` Yann E. MORIN
  0 siblings, 0 replies; 5+ messages in thread
From: Yann E. MORIN @ 2009-08-02 16:58 UTC (permalink / raw)
  To: Bart vdr. Meulen; +Cc: crossgcc

Bart,
All,

On Sunday 02 August 2009 17:34:24 Bart vdr. Meulen wrote:
> # HG changeset patch
> # User Bart vdr. Meulen <bartvdrmeulen@gmail.com>
> # Date 1249224333 -7200
> # Node ID f04c7879f3c2d38cc4aa317532c24395fc0e760a
> # Parent  76f8feef64c455d4cd451ee5f3bf522f6a371f3d
> Add g++ to required tools
> 
> Because gmp is now always build with --enable-cxx,
> g++ has become a required tool

Granted. Will go in.

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +0/33 662376056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| --==< ^_^ >==-- `------------.-------:  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] 5+ messages in thread

* Re: [PATCH 2 of 2] Add config shell override option
  2009-08-02 15:53 ` [PATCH 2 of 2] Add config shell override option Bart vdr. Meulen
@ 2009-08-02 17:03   ` Yann E. MORIN
  0 siblings, 0 replies; 5+ messages in thread
From: Yann E. MORIN @ 2009-08-02 17:03 UTC (permalink / raw)
  To: Bart vdr. Meulen; +Cc: crossgcc

Bart,
All,

On Sunday 02 August 2009 17:34:25 Bart vdr. Meulen wrote:
> Add config shell override option
> 
> Add config option to override the shell used in configuration
> script making it possible to build on an Ubuntu host without
> overriding the default shell (dash)

I'd rather select the shell to be used in a choice menu, like:

"Shell to use as CONFIG_SHELL:"
   "sh (system shell)"
   "ash"
   "bash"
   "Custom shell"
     "Path to custom shell:"

I've already done this in a local branch I'll be pushing later tonight...

OTOH, if we do know that some shells do not work, they why leave such
an option, and why not forcibly use the shell we know works ?

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +0/33 662376056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| --==< ^_^ >==-- `------------.-------:  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] 5+ messages in thread

end of thread, other threads:[~2009-08-02 17:03 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-08-02 15:45 [PATCH 0 of 2] GMP build issues Bart vdr. Meulen
2009-08-02 15:45 ` [PATCH 1 of 2] Add g++ to required tools Bart vdr. Meulen
2009-08-02 16:58   ` Yann E. MORIN
2009-08-02 15:53 ` [PATCH 2 of 2] Add config shell override option Bart vdr. Meulen
2009-08-02 17:03   ` 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).