On Fri, 30 Jun 2006 15:12 +0200, Abe Timmerman <[EMAIL PROTECTED]> wrote:

> Automated smoke report for 5.9.4 patch 28456
> ayla: i86pc (350MHz) (i386/1 cpu)
>     on        Solaris - 2.10
>     using     ? unknown cc version 
>     smoketime 3 minutes 5 seconds (average 23.125 seconds)
> 
> Summary: FAIL(c)
> 
> O = OK  F = Failure(s), extended report at the bottom
> X = Failure(s) under TEST but not under harness
> ? = still running or test results not (yet) available
> Build failures during:       - = unknown or N/A
> c = Configure, m = make, M = make (after miniperl), t = make test-prep

If all these 'c's are because of a warning Nicholas noted:
./Configure: syntax error at line 55: `fi' unexpected

then these should now be fixed.
The patch I applied from Andy created an  empty if, like

if expression ; then
    # comment
    # old statement
fi

Obviously, Solaris doesn't like it. I've now commented out the whole if

Change 28458 by [EMAIL PROTECTED] on 2006/06/30 13:17:59

        Solaris' sh doesn't like empty if's

Affected files ...

... //depot/perl/hints/solaris_2.sh#100 edit

Differences ...

==== //depot/perl/hints/solaris_2.sh#100 (text) ====

559,564c559,565
<               if test $processor = sparc -a X`getconf XBS5_LP64_OFF64_CFLAGS 
2>/dev/null` != X; then
<                   # This adds in -Wa,-xarch=v9.  I suspect that's superfluous,
<                   # since the -m64 above should do that already.  Someone
<                   # with gcc-3.x.x, please test with gcc -v.   A.D. 
20-Nov-2003
<                   # ccflags="$ccflags -Wa,`getconf XBS5_LP64_OFF64_CFLAGS 
2>/dev/null`"
<               fi
---
>
>               # This adds in -Wa,-xarch=v9.  I suspect that's superfluous,
>               # since the -m64 above should do that already.  Someone
>               # with gcc-3.x.x, please test with gcc -v.   A.D. 20-Nov-2003
> #             if test $processor = sparc -a X`getconf XBS5_LP64_OFF64_CFLAGS 
> 2>/dev/null` != X; then
> #                 ccflags="$ccflags -Wa,`getconf XBS5_LP64_OFF64_CFLAGS 
> 2>/dev/null`"
> #             fi

>    28456     Configuration (common) -Dcc=gcc
> ----------- ---------------------------------------------------------
> c - - c - - 
> c - - c - - -Dusemorebits
> c - - c - - -Duseithreads
> c - - c - - -Duseithreads -Dusemorebits
> | | | +----- PERLIO = perlio -DDEBUGGING
> | | +------- PERLIO = stdio  -DDEBUGGING
> | +--------- PERLIO = perlio
> +----------- PERLIO = stdio
> 
> Locally applied patches:
>     SMOKE28456
> 


-- 
H.Merijn Brand        Amsterdam Perl Mongers (http://amsterdam.pm.org/)
using & porting perl 5.6.2, 5.8.x, 5.9.x  on HP-UX 10.20, 11.00, 11.11,
& 11.23, SuSE 10.0, AIX 4.3 & 5.2, and Cygwin.       http://qa.perl.org
http://mirrors.develooper.com/hpux/           http://www.test-smoke.org
                       http://www.goldmark.org/jeff/stupid-disclaimers/

Reply via email to