(Something I should have done in the previous checkin...) Also broke out
substitution command execution from do_sed_command() and put it in it's own
do_subst_command() function.

Minor code reorg: Changed the interface to index_of_next_unescaped_slash to an
interface that seems a little more sensible to me. Also broke out s///
expression parsing into it's own subroutine.

Fix uninitialized variable.
 -Erik

Patch from Marc Nijdam <marc_nijdam@hp.com>
> First (of many more) patch of cp_mv to getopt use. I'm using the most
> simplistic approach, just get getopt used,  then worry about a cleaner
> option parsing style using getopt later.
>
> Marc

Added a smallish TODO comment.

From Matt Kraai <kraai@alumni.carnegiemellon.edu>
> Here is a patch so that tr can handle NULL characters as well.  Give it
> a shot and let me know what you think.  It is against the latest CVS
> version.
>
> In my tests, the following now works as expected.
>
> tr '\0' '\n'
>
> Later,
> Matt

Correct argument parsing and other minor cleanups.

Added some smallish comments to help folks understand why we have two tables
of builtins and the reasoning behind it.

Changed bb_regcomp to xregcomp and #if 0'ed out destroy_cmd_strs in sed.c
(maybe I'll remove it later).

Patch from Jon McClintock <jonm@bluemug.com>
>The attached patch adds the ability to parse the actual mode specifier in
>fbset. So now
>
>    fbset -n 640x480-72
>
>Will actually work. I don't know if I've reinvented the wheel though...

 -Erik

Always report the applet name when doing error reporting.

Use global applet_name instead of local versions.

Use global applet_name instead of local versions.

The applet name isn't constant.

Changed verbiage.
 -Erik

Replaced projects with products/projects.
 -Erik

Added the Kerbango Internet Radio to projects/products using BusyBox.
 -Erik

Applied patch from Matt Kraai to call destroy_cmd_strs in atexit(), rather
than peppering it throughout the code.

Added applet_name to contain the applet name.

Final updates for release.
 -Erik

Final update for the release.

Forgot to commit these cleanups it seems.
 -Erik

More changelog updates
 -Erik

Added changelog entries for grep/sed/regex changes.

Some changelog updates.
 -Erik

Re-enable ln -n in the help.
 -Erik

Another patch from Matt Kraai <kraai@alumni.carnegiemellon.edu>:
>
> The following patch allows ln -n to function like GNU.  It also fixes a
> typo with my previous patch to add support for ln FILE DIRECTORY.  And
> it removes some code that checks the maximum length of the filenames.  I
> can't figure out why that code is necessary.  Anyone know?
>
> Matt

Added 'dumpkmap' to allow people to dump a binary keymap, which can then be
loaded in by 'loadkmap' -- submitted by Arne Bernin <arne@matrix.loopback.org>
 -Erik

Applied patch from Matt Kraai which does the following:
 - adds case-insensitive matching in sed s/// epxressions
 - consolodates common regcomp code in grep & sed into bb_regcomp and put in
   utility.c
 - cleans up a bunch of cruft

Fix dead link.
 -Matt

Fixed NFS so it supports 2.4.x kernels and NFSv3.  Should close bug #1009.
 -Erik

Removed all trace of hand-tooled regexp routines. Bye bye, baby.

Added a rule to make tags files for vi(m).

Comment on kernel stuff
 -Erik

Brand, new sed that uses libc regex routines.
There is some common code used by both sed & grep that should be put into
utility.c as per Mat Kraai's suggestions/patch on the mailing list.
Specifically, a common regex_compile() and a regex_subst() function need to be
made.

More linux kernel header file removal.
 -Erik

Remove yet more kernel header dependancies.
 -Erik

Remove unused variable.
 -Erik

Remove the problematic CTRLCHAR macro.
 -Erik

Put the GROWBY variable inside the get_line_from_file function, as that is the
only function where it's used and it's always good to keep the namespace
clean. :-)

Removed unnecessary #include "regexp.h" line from find.c as per Matt Kraai's
suggestion on the mailing list.

Function name cleanup.
 -Erik

From Matt Kraai <kraai@alumni.carnegiemellon.edu>:

Howdy,

Bug #1006 reports that

ln -s /tmp/foo .

does not work correctly.  In fact, it appears that any instantiation of

ln -s FILE... DIRECTORY

does not work.  The following patch adds support for this form, which
then fixes the particular instance noted in the bug report.

In the process, I needed the basename function.  This appears in the
string.h provided by glibc, but not uC-libc.  So I wrote my own to go in
utility.c, called get_last_path_component.  I also modified the basename
utility to use this function.

At some point it might be desirous to use the basename from the library
if it exists, and otherwise compile our own.  But I don't know how to do
this.

Matt

Patch from Matt Kraai <kraai@alumni.carnegiemellon.edu>:

GNU tr complains on the following:

$ tr a ''
tr: when not truncating set1, string2 must be non-empty

BusyBox tr does not complain:

$ tr a ''
a
^D
0

It should result in an error, not in some spurious output.  The attached
patch generates an error.

Matt

Applied patch from Matt Kraai as per his email:

However, the case of

grep foo$ file

didn't work, due to a problem with the flags used in regular expression
compilation.  The attached patch fixes this problem.

---patch-------
Index: grep.c
===================================================================
RCS file: /var/cvs/busybox/grep.c,v
retrieving revision 1.30
diff -u -r1.30 grep.c
--- grep.c      2000/07/04 22:17:01     1.30
+++ grep.c      2000/07/10 08:57:04
@@ -141,8 +141,10 @@
        if (argv[optind] == NULL)
                usage(grep_usage);

-       /* compile the regular expression */
-       reflags = REG_NOSUB; /* we're not going to mess with sub-expressions
        */
+       /* compile the regular expression
+        * we're not going to mess with sub-expressions, and we need to
+        * treat newlines right. */
+       reflags = REG_NOSUB | REG_NEWLINE;
        if (ignore_case)
                reflags |= REG_ICASE;
        if ((ret = regcomp(&regex, argv[optind], reflags)) != 0) {
---patch-------

Thanks, Matt, it works great.

More portability updates.  Now compiles cleanly vs glibc, libc5, and uclibc
(except for mkfs_minix and fsck_minix -- and it doesn't yet link vs uclibc due
to missing stuff in the library).
 -Erik

include getopt.h
 -Erik

Fix a bug in get_line_from_file.  If the length of the line is (GROWBY * n) +
GROWBY - 1, then it writes the null character just after the buffer.  Yipe.
Fix thanks to Matt Kraai <kraai@alumni.carnegiemellon.edu> Thanks Matt!
 -Erik

The build depends were too strict.  I depended on on debhelper (>= 2.0.80)
while debhelper 1.1.24 is available for slink.  Just forget the dependancy --
it isn't needed.
 -Erik

Argh.  More cross libc cleanup.  Should be ok now...
 -Erik

More cleanup on umount
 -Erik

Update files to reduce dependance on kernel version...
 -Erik

Add in tinynet
 -Erik

Doc updates to remove sfdisk.
 -Erik

Better description of the BB_FEATURE_USE_DEVPS_PATCH option.
 -Erik

Add in my fix for 2.4.x kernels so /proc/mounts will not have useless
deg entries which break mount, df, and friends.  Renamed WillThisGoIntoTheKernel
to avoid any confision.
 -Erik

Removed sfdisk from BusyBox.  It was buggy, fat, and we really couldn't
maintain it very well, so including it was not very appropriate.  Those wanting
an fdisk are invited to grab a copy from util-linux.
 -Erik

Added in Matt Kraai.
 -Erik

Fix a dependancy bug for the docs.
 -Erik

Naming fix
 -Erik

Recognize Matt's poweroff fix.
 -Erik

Fix for bug #1003 -- BusyBox should now poweroff when asked to
do so...  Fix thanks to Matt Kraai <kraai@alumni.carnegiemellon.edu>
 -Erik

Web page update from Matt Kraai <kraai@alumni.carnegiemellon.edu>
 -Erik

This patch finishes the cleanup of all the commands.  It also cleans up
the remaining chapters -- Matt Kraai <kraai@alumni.carnegiemellon.edu>
 -Erik

Yet another busybox documentation update from Matt Kraai <kraai@alumni.carnegiemellon.edu>
 -Erik

Update 'make release' to remove all the .#filename files
the CVS leaves lying about...
 -Erik

Turned some stuff into features that really are features, not apps.
 -Erik

Fix doc building so the dependancies work properly.
 -Erik

* Fixed tar creation support when reading from stdin ('tar -cf - . ')
    thanks to Daniel Quinlan <quinlan@transmeta.com>
 -Erik

Fixed backspace and delete so they work properly again -- broken
since 0.44.
 -Erik

More doc patches from Matt Kraai <kraai@alumni.carnegiemellon.edu>.
Thanks Matt!
 -Erik

Remove leftovers of block_device, which disapperared around
busybox 0.28.
 -Erik

Change order.
 -Erik

Added (fixed) the '-n' option to fbset.  Renumbered the command parameters to
put a gap between the normal commands and the fancy commands. This makes it
cleaner to add normal commands.  Patch from Jon McClintock <jonm@bluemug.com>.
 -Erik

setkeycodes needs get_console_fd(), so if you disable chvt and deallocvt, but
leave setkeycodes active, busybox will not link.  Also fix a trivial
use-before-initialize warning.  Both fixes from Jon McClintock
<jonm@bluemug.com>.

 -Erik

Remove some warning with glibc.
 -Erik

Comment on kill.c change.
 -Erik

Reorganized signal names for better architecture support -- patch
thanks to simon wood <simon@mungewell.uklinux.net>
 -Erik

DocBook documentation update from Matt Kraai <kraai@alumni.carnegiemellon.edu>
 -Erik

Had an extra tcsetpgrp.
 -Erik

a few minor cleanups.
 -Erik

Remove MAXNAMLEN and use BUFSIZ instead.
 -Erik

Remove misguided klude around for 2.4.x-test* brokenness.  Al Viro
will be removing this stuff from /proc/mounts real soon now I am assured.
 -Erik

* Fix to tr so it recognizes standard escape sequences.  Merged common
    escape seq. code from tr and echo into utility.c.  Fix thanks to
    Matt Kraai <kraai@alumni.carnegiemellon.edu>.
* This should close Bug #1015.  Please test.
 -Erik

Patch from Matt Kraai <kraai@alumni.carnegiemellon.edu> to
simplify ar.c and make it more readable.
 -Erik

Some cosmetic updates.  Changed "OPTIONS" to "Options".
 -Erik

More doc updates
 -Erik

A first pass at integrating the SGML docs into the Makefile.
A first pass a cleaning up the current SGML (lots more cleanup
is needed though).
 -Erik

* Fix to init.c from Stuart Menefy <Stuart.Menefy@st.com> so that
    it always sets the controlling terminal before running any programs
 -Erik

Fixed a potential infinite loop.  First delete any chars after the
cursor, then delete everything before it.  Before this, we would just
delete everything before the cursor and freak out if there were still
chars left (i.e. stuff after the cursor).
 -Erik

sh fixes from Marius Groeger <mag@sysgo.de>
 -Erik

Added support for ignoring '-g' per GNU ls, thanks to David Vrabel
<dvrabel@arcom.co.uk>.  Seems some ftp clients expect the '-g' option
to exist.
 -Erik

Added support for being a login shell, so things like '-su' or '-sh' (stuff
where argv[0][0]=='-') will now always invoke the shell.  Now you can use
BusyBox as a login shell.
 -Erik

Fixed comment.

Added support for reading from stdin with '-' as file name arg.

Yanked out the cstring_alloc() and cstring_lineFromFile() functions from
utility.c and replaced them with get_line_from_file() from the new grep.c.
Also changed declaration in internal.h and replaced instances of
cstring_lineFromFile() in dc.c and sort.c with get_line_from_file(). Tested
them and they worked fine.

Brand, spankin', new grep that uses libc regex routines instead of the
hand-rolled ones. Sed still needs to be replaced and then the regexp stuff can
be axed.

Moved some function decls, a struct, and a #define from regexp.h into here.
Also static-ified said functions so they do not have namespace conflicts with
the libc regex functions.

Removed some function decls (regex, regcomp, etc.) and put them in regexp.c

Better cross libc support.
 -Erik