This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: Accept blanks as language separators


On Aug  1, 2000, Manfred Hollstein <manfredh@redhat.com> wrote:

> On , 1 August 2000, 13:26:19 -0300, aoliva@redhat.com wrote:
>> On Aug  1, 2000, Manfred Hollstein <manfredh@redhat.com> wrote:
>> 
>> > You also need to reflect this in gcc/configure.in then.
>> 
>> gcc/configure.in `tr's blanks to commas.  I hadn't seen that before,
>> so I thought only commas were acceptable.

> Well, they are. Blanks will only be converted into commas, when
> ${LANGUAGES} is used because no --enable-languages=whatever was
> specified, but LANGUAGES was; otherwise, ${enable_languages} is
> used literally.

Indeed.  Here's a patch that presumably handles LANGUAGES correctly.
Ok to install, if it works?  (I haven't tested it, because I've got to
leave in a couple of minutes; I'd appreciate if someone else could
give it a try while I'm away)

Index: ChangeLog
from  Alexandre Oliva  <aoliva@redhat.com>
	* configure.in (CHILL_FOR_TARGET, CXX_FOR_TARGET): Convert blanks to
	commas in $LANGUAGES.

Index: configure.in
===================================================================
RCS file: /cvs/gcc/egcs/configure.in,v
retrieving revision 1.57
diff -u -r1.57 configure.in
--- configure.in	2000/08/01 04:10:47	1.57
+++ configure.in	2000/08/01 16:53:08
@@ -1286,8 +1286,8 @@
 if test "x${CHILL_FOR_TARGET+set}" = xset; then
   :
 elif test -d ${topsrcdir}/gcc &&
-   echo ",${enable_languages-${LANGUAGES-CHILL}}," |
-   grep ,CHILL, > /dev/null ; then
+   echo ",${enable_languages-`echo ${LANGUAGES-CHILL} | tr ' ' ','`}," |
+   grep ',CHILL,' > /dev/null ; then
   CHILL_FOR_TARGET='$$r/gcc/xgcc -L$$r/gcc/ch/runtime/'
 elif test "$host" = "$target"; then
   CHILL_FOR_TARGET='$(CC)'
@@ -1302,7 +1302,7 @@
 if test "x${CXX_FOR_TARGET+set}" = xset; then
   :
 elif test -d ${topsrcdir}/gcc &&
-   echo ",${enable_languages-${LANGUAGES-c++}}," |
+   echo ",${enable_languages-`echo ${LANGUAGES-c++} | tr ' ' ','`}," |
    grep ',c[+][+],' > /dev/null ; then
   CXX_FOR_TARGET='$$r/gcc/g++ -nostdinc++ '$libstdcxx_flags
 elif test "$host" = "$target"; then

-- 
Alexandre Oliva   Enjoy Guarana', see http://www.ic.unicamp.br/~oliva/
Red Hat GCC Developer                  aoliva@{cygnus.com, redhat.com}
CS PhD student at IC-Unicamp        oliva@{lsd.ic.unicamp.br, gnu.org}
Free Software Evangelist    *Please* write to mailing lists, not to me

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]