[PATCH v2] builtin: Fix handling of trailing IFS white spaces

2016-06-06 Thread Herbert Xu
On Fri, Dec 04, 2015 at 08:51:19PM +0100, Harald van Dijk wrote: > > Here it is. Attached is an updated patch that ignores the complete > terminator if only a single field remains, otherwise ignores only > trailing IFS whitespace. There is a little bug in the patch if you happened to start with

[PATCH] eval: Return status in eval functions

2016-06-06 Thread Herbert Xu
On Sat, Dec 05, 2015 at 04:40:59PM +0100, Harald van Dijk wrote: > > used to print Fail, and needed the same modification in the evalstring > function to make that print OK (included in the attached patch). There > may be other similar bugs lurking. Not exactly similar but there are a bunch of

Re: builtin: Reject malformed printf specifications with digits after '*'

2016-06-06 Thread Herbert Xu
On Sun, Dec 06, 2015 at 03:09:42PM +0100, Gioele Barabucci wrote: > Hello, > > I am forwarding a patch related to the bug described at > . > > You can find the same patch at > . > > I

Re: trap: Implement POSIX.1-2008 trap reset behaviour (#2)

2016-06-06 Thread Herbert Xu
On Mon, Dec 07, 2015 at 05:50:47PM +, Jonathan Perkin wrote: > Clarifies a couple of issues with the previous patch, and expands on > the rationale in the commit message. Sorry for the noise -- jperkin Thanks for the patch. I've decided to do something similar to your final patch: ---8<---

Re: [SHELL] Fix 64-bit Solaris build

2016-06-06 Thread Herbert Xu
On Fri, Dec 11, 2015 at 01:07:14PM +, Jonathan Perkin wrote: > In a 64-bit Solaris environment there is no stat64() function, only > stat(). This conflicts with the stat64 #define used to support > dietlibc/klibc when stat64() is not found and results in: > > ./../config.h:194:16: error:

Re: man: Document ulimit -v

2016-06-06 Thread Herbert Xu
On Mon, Jan 11, 2016 at 08:12:35PM +0100, Gioele Barabucci wrote: > Document that `ulimit` can set the `RLIMIT_AS` limit (virtual memory) with > the `-v` flag. > > Fixes: https://bugs.debian.org/78556 > Reported-by: Vincent Lefevre > Signed-off-by: Gioele Barabucci

Re: shell: Fix build on Solaris 9

2016-06-06 Thread Herbert Xu
On Fri, Dec 11, 2015 at 11:56:43AM +, Jonathan Perkin wrote: > Ensure dash can build in a default Solaris 9 or older environment: > > - Execute scripts with $SHELL rather than /bin/sh, the latter does not >support e.g. "if ! .." used by mkbuiltins. > > - /bin/awk does not support ?:

Re: [SHELL] Fix 64-bit Solaris build

2016-06-06 Thread Jonathan Perkin
* On 2016-06-06 at 15:27 BST, Herbert Xu wrote: > On Fri, Dec 11, 2015 at 01:07:14PM +, Jonathan Perkin wrote: > > In a 64-bit Solaris environment there is no stat64() function, only > > stat(). This conflicts with the stat64 #define used to support > > dietlibc/klibc when stat64() is not