Re: [PATCH] ash: fix error code regression

2017-01-10 Thread Denys Vlasenko
1.26.2 is out. On Sun, Jan 8, 2017 at 6:00 PM, Denys Vlasenko wrote: > There is a bug in ash which requires a new bugfix release (1.26.2) soon > anyway. > > On Fri, Jan 6, 2017 at 12:52 PM, Martijn Dekker wrote: >> Op 02-01-17 om 13:50 schreef Ron

Re: [PATCH] ash: fix error code regression

2017-01-08 Thread Denys Vlasenko
There is a bug in ash which requires a new bugfix release (1.26.2) soon anyway. On Fri, Jan 6, 2017 at 12:52 PM, Martijn Dekker wrote: > Op 02-01-17 om 13:50 schreef Ron Yorston: >> The commit 'ash,hush: set exit code 127 in "sh /does/not/exist" case' >> only partly implemented

Re: [PATCH] ash: fix error code regression

2017-01-06 Thread Martijn Dekker
Op 02-01-17 om 13:50 schreef Ron Yorston: > The commit 'ash,hush: set exit code 127 in "sh /does/not/exist" case' > only partly implemented the dash commit '[ERROR] Allow the originator > of EXERROR to set the exit status'. This resulted in incorrect error > codes for a syntax error: [...]

Re: [PATCH] ash: fix error code regression

2017-01-03 Thread Ron Yorston
Denys Vlasenko wrote: >> --- /dev/null >> +++ b/shell/ash_test/ash-misc/exitcode2.tests >> @@ -0,0 +1,13 @@ >> +# syntax error should return status 2 >> +cat >test.sh <> +) >> +EOF >> +chmod +x test.sh >> +./test.sh > >Will this run test.sh in _the shell being tested_ >(as opposed to system

Re: [PATCH] ash: fix error code regression

2017-01-03 Thread Denys Vlasenko
On Mon, Jan 2, 2017 at 1:50 PM, Ron Yorston wrote: > The commit 'ash,hush: set exit code 127 in "sh /does/not/exist" case' > only partly implemented the dash commit '[ERROR] Allow the originator > of EXERROR to set the exit status'. This resulted in incorrect error > codes for a

[PATCH] ash: fix error code regression

2017-01-02 Thread Ron Yorston
The commit 'ash,hush: set exit code 127 in "sh /does/not/exist" case' only partly implemented the dash commit '[ERROR] Allow the originator of EXERROR to set the exit status'. This resulted in incorrect error codes for a syntax error: $ ) $ echo $? 0 or a redirection error for a