Re: gmake check runs just 13 tests instead of 77

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Mr OCP <mr_ocp(at)hotmail(dot)com>, pgsql-admin(at)postgresql(dot)org, andrew(at)libertyrms(dot)info
Subject: Re: gmake check runs just 13 tests instead of 77
Date: 2002-11-13 00:27:26
Message-ID: 200211130027.gAD0RQf26386@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-admin

Tom Lane wrote:
> "Mr OCP" <mr_ocp(at)hotmail(dot)com> writes:
> > I am testing the beta 5 version of 7.3, by using make check under solaris 7
> > and 8 and it runs just 13 tests as against 77 tests as indicated in the
> > README.regression,
>
> Hmm, maybe a portability problem in the pg_regress script ... but I
> thought we'd already checked 7.3 on Solaris. Andrew, can you reproduce
> this problem?

It does report:

All 13 tests passed.

I wonder if Solaris folks are thinking that is a success. I see you
found that my awk 'BEGIN ...' was the problem. I didn't realize awk
would do that. On BSD/OS using gawk, it is OK:

$ echo "1\n2" | while read FILE; do echo $FILE;awk 'BEGIN {print "a"}';
done
1
a
2
a

Anyway, pg_regress.sh should just be fixed. I added this code so I
didn't have to do gymnatics on echoing a backslash:

echo "SET autocommit TO 'on';"; awk 'BEGIN {printf "\\set ECHO all\n"}'; cat "$inputdir/sql/$name.sql") |

I didn't use this method in any of my other autocommit fixes. I did it
this way only so I could turn autocommit on without having it show up in
the regression output.

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073

In response to

Browse pgsql-admin by date

  From Date Subject
Next Message Andrew Sullivan 2002-11-13 03:15:00 Re: gmake check runs just 13 tests instead of 77
Previous Message Tom Lane 2002-11-12 20:55:55 Re: gmake check runs just 13 tests instead of 77