Release 0.5.3.

[BUILTIN] Fix \c spillage across echo commands

On Thu, Nov 03, 2005 at 07:16:53PM +0100, Mike Hommey wrote:
> Here is a simple test case:
>
> #!/bin/dash
> echo test\\ test
> echo '\c'
> echo test\\ test
>
> it outputs:
> test\ test
> test\
>
> This is due to mis-usage of a global variable. See attached patch for a
> fix.

Instead of setting rval when \c is detected, this is now set in the
return value of conv_escape_str.  This prevents the spillage reported
in http://bugs.debian.org/337294.

[BUILTIN] Always call conv_escape_str in echocmd

Instead of calling conv_escape_str when we detect a backslash we will call it
unconditionally.  This helps get rid of some unnecessary code in echocmd.

[MEMALLOC] Remove unnecessary truncation in _STPUTC

Using char in the function argument causes unnecessary truncation to occur.
This patch changes it to an int.

[HELPER] Use mktemp if tempfile is not available

tempfile(1) is Debian-specific so let's use mktemp(1) as a fallback.

[JOBS] Fixed support for disabling job control

Since nobody has compiled with JOBS turned off for quite a while, it has
bit-rotted.  This patch makes it build again.

[SYSTEM] Fixed fallback stpcpy implementation

The implementation has an off-by-one bug.  This results in random memory
corruption.  In one particular case, it caused certain bits of a function
body to go missing.

[SYSTEM] Added out-of-line ctypes functions for klibc

Unfortunately klibc doesn't provide out-of-line versions of ctypes
functions such as isalpha.  This is a nasty hack to create them.

[SYSTEM] Include system.h for stpcpy in nodes.c

Since we have a fallback implementation for stpcpy we can now use it
unconditionally in nodes.c.pat.  This also fixes a link error with
klibc.

[SYSTEM] Added dummy sysconf implementation

Add a dummy sysconf implementation that always fails for klibc.

[BUILTIN] Removed standalone/csh support from test

Let's remove the support for standalone support from test for the same
reason as printf.

[SYSTEM] Added dummy strtod implementation

klibc doesn't have strtod or atof.  So add an implementation that
always fails by setting the end pointer to the input string.

[BUILTIN] Removed standalone/csh support from printf

The support for standalone/csh printf only serves to complicate
maintainence.

[SYSTEM] Added default definition for SSIZE_MAX

klibc doesn't define the macro SSIZE_MAX.

[BUILTIN] Disable histcmd completely when SMALL is defined

Now that mkbuiltins correctly disabled histcmd we don't need the dummy
histcmd implementation anymore.

[BUILTIN] Disable ulimit if there is no getrlimit

For systems without getrlimit (e.g., klibc) we will disable ulimit.

In order to achieve this, builtins.def is now produced by cpp which
allows us to use macros such as HAVE_GETRLIMIT in it.

Thie also means that we can get rid of the cflags parsing code in
mkbuiltins.

[SIGNAL] Added default implementation of killpg

klibc doesn't have killpg.  Since we only call it for valid values of
pid, we can call kill instead.

[SIGNAL] Added default implementation of strsignal

klibc doesn't have strsignal but it does have sys_siglist.

Added missing system.h inclusion for mempcpy

All users of mempcpy must include system.h.

Fixed gcc 4.0 compilation problems

Removed obsolete extern declaration on funcnest.  This conflits with the
correct static definition.

Changed memtodest prototype to use char * instead of unsigned char *.
Perform the unsigned char cast inside memtodest instead.

[EXPAND] Added getpwhome as a wrapper for getpwnam

klibc doesn't have and doesn't need getpwnam.  This change creates
getpwhome which always returns NULL if getpwnam doesn't exist.

[SYSTEM] Added default implementation of bsearch

Added impelmentation of bsearch since klibc doesn't have it yet.

Use stat if stat64 does not exist

64-bit file calls such as stat64 don't exist on all architectures for
dietlibc, and they don't exist at all for klibc.  In those cases the
normal calls such as stat are already 64-bit.

So simply define stat64 as stat if it doesn't exist.  Do the same for
all other 64-bit calls as well.

[BUILTIN] Stop using sysexits.h in commandcmd

This gets rid of the only reference of sysexits.h in dash which is
from commandcmd.  This is needed for klibc support since it doesn't
have sysexits.h.

The only uses of sysexits.h in commandcmd is superfluous anyway.  In
fact, it is overly sensitive about usages such as 'command -vV ls'.
By making its behaviour close to that of bash/ksh, we end up saving
a bit of space too.

[SIGNAL] Use bsd_signal if it exists and signal does not

klibc has bsd_signal instead of signal.  So we will define signal as
bsd_signal if

1) signal does not exist.
2) bsd_signal exists.

[SIGNAL] Removed use of __P from error.h

This is the only instance of __P in the entire source so it makes sense
to get rid of it rather than making it work with klibc.

Copyright/licence updates and remove all traces of sys/cdefs.h

This change updates the BSD licence to the three-clause version since
NetBSD has already done so.  This makes dash GPL-compatible.

It also adds Christos Zoulas (NetBSD ash maintainer) to the COPYING file.
I've added "copyright by Herbert Xu" to most files.

Finally all CVS IDs and inclusion of sys/cdefs.h have been removed.
The latter is needed for support of klibc.

[INPUT] Size optimisations in preadbuffer()

Added unlikely markers.
Reduced switch coverage.
Removed p since we now erase NULs as soon as we see them.
Use more to store parselleft.

[INPUT] Fix NUL skipping in preadbuffer

On Sun, Sep 25, 2005 at 07:50:54PM +0000, Gerrit Pape wrote:
> On Sat, Sep 03, 2005 at 02:56:00PM +0200, Martin Dickopp wrote:
> > When analysing the bug I came to the following conclusion: The loop
> > beginning at input.c:302 overwrites a single input line with itself,
> > skipping NUL characters. Therefore, after the loop the line buffer
> > has less characters than originally read if and only if the input line
> > contains NUL characters.
>
> yes.  Thanks a lot for the details and patch.
>
> > The pointer that is used to read from the buffer, 'parsenextc' (cf.
> > input.h:66), is also used as the beginning of the next line in
> > input.c:296. This fails if the buffer contains less characters
> > than originally read into it due to NUL characters.
> >
> > The proposed patch (attached) keeps track of the number of skipped
> > characters and advances 'parsenextc' accordingly before processing
> > the next input line.
>
> Hi Herbert, please see
>  http://bugs.debian.org/317516

Instead of moving the characters in the loop, we will do memmove every
time we see a NUL character.  This hurts if there are a lot of NUL
characters, but should be a win in normal situations.

Added eflag fixes for trap and minusc.

Let evaltree handle traps from cmdloop.
Reset evalskip after minusc is executed.
Stop executing traps once SKIPEVAL is seen.

Removed unnecessary inclusion of eval.h from parser.c.

Let evaltree handle traps from cmdloop.

Handle SKIPEVAL in read_profile by exiting.

Removed unnecessary inclusion of main.h from eval.c.

Generalise setinputfile for use in read_profile/readcmdfile.

Removed some unnecessary inclusions of input.h.

Turn evalskip into a bit field.

This allows SKIPEVAL and SKIPFUNC to coexist which is needed for eval return 1.

Eliminate first null termination in setvar.

Add trailing equal sign in setvar for variables set to null.

Get rid of duplicate -g -O2 in CFLAGS.

Cleaned up src/Makefile.am.

Fixed support for cross-compilation.

Do not clobber exit status in dotcmd.

Removed redundant setstackmark from dotcmd.

Removed qflag.

Invert return value of test_eaccess and rename it to test_st_mode.

Eliminate duplicate stat in test_eaccess.

Optimised FILGZ test.

Enclose abort insode ifdef DEBUG.

Renamed error to sh_error.

Normalise input in likely/unlikely macros.

Removed expcmd built-in.

Only set skipcount for break and continue.

Update funcnest atomically.

Replaced EXEVAL with SKIPEVAL.

Corrected that/than typo in manual page.

Optimise away status and case statement in main().

Eliminated global exerrno.

Only reread exitstatus on EXEXIT in exitshell.

Do not clobber exit status on EXEVAL.

Size optimisations with state/s in main().

Corrected grammar in manual page.

Catch set -e exits within built-in commands.

Changed boolean rootshell into shlvl counter.

Release 0.5.2.
TAG: v0.5.2

Corrected manual entry about ENV and non-interactive shells.

Removed debian files.

Spell behaviour consistently in manual page.

Fixed spelling errors in manual page.

Fixed signed char promotion in src/expand.c.

Size optimisations around varvalue() in src/expand.c.

Fixed  expansion when leading argument is null in src/expand.c.

Allow negative pid argument to kill(1) in src/jobs.c.

Fixed cd - when OLDPWD is unset.

Include system.h.

Initial import.