Re: CVS commit: src/tests/util/sh

2011-11-16 Thread Julio Merino
On 11/14/11 3:23 PM, Christos Zoulas wrote: Module Name:src Committed By: christos Date: Mon Nov 14 20:23:29 UTC 2011 Modified Files: src/tests/util/sh: Makefile Added Files: src/tests/util/sh: t_evaltested.sh Log Message: Add a test for PR/45613 (eval failing in

Re: CVS commit: src/tests/util/sh

2010-06-07 Thread David Holland
On Mon, Jun 07, 2010 at 07:30:27AM -0400, Christos Zoulas wrote: > | Ok, right now ksh diverges from sh on 32 of 75 cases, including > | several of the ones you changed. bash diverges on 3. > | > | Anyway, the test script now at least documents what the deal is, so > | I'm ok with leaving it

Re: CVS commit: src/tests/util/sh

2010-06-07 Thread Christos Zoulas
On Jun 6, 7:18pm, dholland-sourcechan...@netbsd.org (David Holland) wrote: -- Subject: Re: CVS commit: src/tests/util/sh | On Sun, Jun 06, 2010 at 07:11:05PM +, David Holland wrote: | > On Sun, Jun 06, 2010 at 08:08:57AM -0400, Christos Zoulas wrote: | > > | If it's more

Re: CVS commit: src/tests/util/sh

2010-06-06 Thread David Holland
On Sun, Jun 06, 2010 at 07:11:05PM +, David Holland wrote: > On Sun, Jun 06, 2010 at 08:08:57AM -0400, Christos Zoulas wrote: > > | If it's more important that the nightly test runs come up clean than > > | that it actually check for correct behavior so sh gets fixed, the test > > | scri

Re: CVS commit: src/tests/util/sh

2010-06-06 Thread David Holland
On Sun, Jun 06, 2010 at 08:08:57AM -0400, Christos Zoulas wrote: > | If it's more important that the nightly test runs come up clean than > | that it actually check for correct behavior so sh gets fixed, the test > | script should at least document what the correct behavior is. I have > | done

Re: CVS commit: src/tests/util/sh

2010-06-06 Thread Christos Zoulas
On Jun 6, 4:51am, dholland-sourcechan...@netbsd.org (David Holland) wrote: -- Subject: Re: CVS commit: src/tests/util/sh | If it's more important that the nightly test runs come up clean than | that it actually check for correct behavior so sh gets fixed, the test | script should at

Re: CVS commit: src/tests/util/sh

2010-06-05 Thread David Holland
On Thu, Jun 03, 2010 at 06:53:36PM +, Christos Zoulas wrote: > >> # according to the standard, only failing *simple* commands > >> # cause an exit under -e. () is not a simple command. > >> - dcheck '(set -e; (set +e; false; echo OK; false); echo OK)' 'OK OK' > >> - echeck '(set -e; (se

Re: CVS commit: src/tests/util/sh

2010-06-03 Thread Christos Zoulas
In article <20100603164416.gd6...@apb-laptoy.apb.alt.za>, Alan Barrett wrote: >On Thu, 03 Jun 2010, Christos Zoulas wrote: >> Log Message: >> Pass all the set -e tests. >> >> To generate a diff of this commit: >> cvs rdiff -u -r1.6 -r1.7 src/tests/util/sh/t_set_e.sh > >Now the tests don't match

Re: CVS commit: src/tests/util/sh

2010-06-03 Thread Alan Barrett
On Thu, 03 Jun 2010, Christos Zoulas wrote: > Log Message: > Pass all the set -e tests. > > To generate a diff of this commit: > cvs rdiff -u -r1.6 -r1.7 src/tests/util/sh/t_set_e.sh Now the tests don't match the comments. > # according to the standard, only failing *simple* commands >