It is now possible to select in busybox.def.h whether minixfs version 2
is to be supported.

The result of getopt() is "int" and should be treated as such

Extract usage information into a separate file.

Don't use strings directly in calls to usage().  This is in preparation
for their extraction to a separate file.

Fixed a couple of buglets:
 - add_cmd_str: segv's were being generated if there was a '# comment' line
   (and probably other kinds of lines, too) that was not followed by a
   semi-colon or whitespace
 - parse_edit_cmd: was returning a wrong number (too low) for the index; it
   was not accounting for backslashes eaten, for the fact that we start at the
   3rd index in the string, or for the fact that we add an extra newline.
 - parse_cmd_str: was returning a wrong number (again, too low) for the index
   in the case of single-letter commands (p,d). There was some
   over-compensation for this in the 'return' stmt at the end which also
   needed some help.
 - load_cmd_file: was not eating trailing newlines off the line read from the
   command file. This had the deleterious effect of printing an extra newlines
   after text displayed from edit (i,a,c) commands.

 - Added support for semicolon delimited command lines. (woo-hoo!)
 - Obsoleted the trim_str function (#if 0'ed out -- maybedelete later) in
   favor of strrspn.
 - Obsoleted the strrspn function (#if 0'ed out as well) as soon as I
   discovered that it wasn't needed either.
 - Fixed a subtle bug in parse_subst_cmd where it would choke with an error if
   there was any trailing space after the s/match/replace/ expression.

Getopt'ed by Marc Nijdam <marc_nijdam@hp.com>
 -Erik

Getopt'rd by Marc Nijdam <marc_nijdam@hp.com>
 -Erik

More fixes for "signed vs. unsigned" warnings.

Reduced the number of "signed vs. unsigned" warnings.
Sometimes such warnings matter (esp. on PPC with char default to unsigned)

More fixes for "missing initializers"

Fixed a warning about missing initializer

The result of getopt() should not be converted to "char" - it breaks on PPC

More cleanups.
 -Erik

Fix argument parsing.

Be more pedantic aboutthe terminating null applet.
 -Erik

uid and gid were unsigned, but were compared vs signed values (-1)
 -Erik

A couple of minor warning cleanups.
 -Erik

Use errorMsg rather than fprintf.

Fix some stupid memory bugs.
 -Erik

Add in redimentary backtick suport (doesn't work properly yet, but is
close).
 -Erik

It dawned on me that I would need to grow a char buffer one extra char bigger
to accomodate a trailing '\n'ewline that I append to it later one. This is
only necessary for the case of one inserted, appended, or changed line, but
it's still necessary.

Added error checking for edit commands; only a beginning address can be
specified.

Added support for (a)ppend, (i)nsert, and (c)hange commands to sed.

Fixed bug where sed '/foo/p' was printing "invalid command".

(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