Re: dejagnu handlig fails if {build,host,target}_ailas is not defined

[Alexandre Duret-Lutz]

| Hello,
| dejagnu support assumes build_alias and others are defined, but they're
| empty when nothing was given on the command line. In this case, the
| site.exp file contains `set build_alias' and similar invalid lines.
| This should fix it.
|       Mirek
| 
| --- dejagnu.am.orig   Sat May 11 22:16:17 2002
| +++ dejagnu.am        Sat May 11 22:17:58 2002
| @@ -82,11 +82,11 @@
|       @echo 'set tool $(DEJATOOL)' >>site.tmp
|       @echo 'set srcdir $(srcdir)' >>site.tmp
|       @echo "set objdir `pwd`" >>site.tmp
| -?BUILD?      @echo 'set build_alias $(build_alias)' >>site.tmp
| +?BUILD?      @echo 'set build_alias "$(build_alias)"' >>site.tmp
|  ?BUILD?      @echo 'set build_triplet $(build_triplet)' >>site.tmp
| -?HOST?       @echo 'set host_alias $(host_alias)' >>site.tmp
| +?HOST?       @echo 'set host_alias "$(host_alias)"' >>site.tmp
|  ?HOST?       @echo 'set host_triplet $(host_triplet)' >>site.tmp
| -?TARGET?     @echo 'set target_alias $(target_alias)' >>site.tmp
| +?TARGET?     @echo 'set target_alias "$(target_alias)"' >>site.tmp
|  ?TARGET?     @echo 'set target_triplet $(target_triplet)' >>site.tmp
|       @echo '## All variables above are generated by configure. Do Not Edit 
##' >>site.tmp
|       @test ! -f site.exp || \

Thanks, and sorry for the late answer.  
I'm checking in the following on HEAD and branch-1-6.

2002-06-05  Miloslav Trmac  <address@hidden>

        * lib/am/dejagnu.am (site.exp): Quote build_alias, host_alias, and
        target_alias.

Index: THANKS
===================================================================
RCS file: /cvs/automake/automake/THANKS,v
retrieving revision 1.166.2.6
diff -u -r1.166.2.6 THANKS
--- THANKS      24 May 2002 10:39:31 -0000      1.166.2.6
+++ THANKS      5 Jun 2002 08:56:14 -0000
@@ -120,6 +120,7 @@
 Michael Brantley       address@hidden
 Michel de Ruiter       address@hidden
 Miles Bader            address@hidden
+Miloslav Trmac         address@hidden
 Miodrag Vallat         address@hidden
 Morten Eriksen         address@hidden
 Motoyuki Kasahara      address@hidden
Index: lib/am/dejagnu.am
===================================================================
RCS file: /cvs/automake/automake/lib/am/dejagnu.am,v
retrieving revision 1.16.2.1
diff -u -r1.16.2.1 dejagnu.am
--- lib/am/dejagnu.am   30 Apr 2002 18:38:55 -0000      1.16.2.1
+++ lib/am/dejagnu.am   5 Jun 2002 08:56:14 -0000
@@ -82,11 +82,12 @@
        @echo 'set tool $(DEJATOOL)' >>site.tmp
        @echo 'set srcdir $(srcdir)' >>site.tmp
        @echo "set objdir `pwd`" >>site.tmp
-?BUILD?        @echo 'set build_alias $(build_alias)' >>site.tmp
+## Quote the *_alias variables because they might be empty.
+?BUILD?        @echo 'set build_alias "$(build_alias)"' >>site.tmp
 ?BUILD?        @echo 'set build_triplet $(build_triplet)' >>site.tmp
-?HOST? @echo 'set host_alias $(host_alias)' >>site.tmp
+?HOST? @echo 'set host_alias "$(host_alias)"' >>site.tmp
 ?HOST? @echo 'set host_triplet $(host_triplet)' >>site.tmp
-?TARGET?       @echo 'set target_alias $(target_alias)' >>site.tmp
+?TARGET?       @echo 'set target_alias "$(target_alias)"' >>site.tmp
 ?TARGET?       @echo 'set target_triplet $(target_triplet)' >>site.tmp
        @echo '## All variables above are generated by configure. Do Not Edit 
##' >>site.tmp
        @test ! -f site.exp || \

-- 
Alexandre Duret-Lutz