- use bb_xchdir instead of ignoring eventual errors of chdir
  Thanks to Erik Hovland

- fix bug #887, in bb_get_chomped_line_from_file(), the last char was removed
  unconditionally, even if it was not a newline.
  This was apparently broken by r14254
- whitespace while at it.

add ordering warning on enum Location

- make sure that we see string_insert only if we need it.

remove leading "Enable" from the config option text.  not only is it
redundant, but it made the menuconfig shortcut mechanism mostly
useless, since so many of the entries started with 'E'.

made "test" an ash built-in.

moved the contents of libbb/bb_echo.c back into coreutils/echo.c,
which is a more reasonable place for them than libbb.  this
forces anyone who wants echo and test to be builtin to ash to
also have them available as applets.  their cost is very small,
and the number of people who wouldn't want them as applets is
also very small.

added warning about shell builtins vs. CONFIG_FEATURE_SH_STANDALONE_SHELL,
which conflicts with their use.

thanks to nathanael copa for debugging help.

some string size optimization in test.c may have been lost with
this commit, but this is a good new baseline.

Patch from Erik Hovland, via Tito.

Some day, maybe gcc will be able to optimize out static functions that are
never used so we don't have to #ifdef them.  Wouldn't that be nice?

need prototype for new vfork_daemon()

initial support for no-mmu systems

sometimes daemonizing and vfork()/exit() is ok

bb_xdaemon() isnt available on no-mmu systems

comment all fields

hint at where the Magic comes from

- do away with the comment that noted that the current implementation
  unly supports linux.

Fix from Tito to read from stdin only when it's not a tty.

- spelling

add reference to proc(5)

- reuse strings and messages. Saves about 600B

Consolidate devfs garbage and mark it as obsolete.

use bb_xbind/bb_xlisten

whitespace

whitespace

add a --noclobber flag

add support for cleaning out previous busybox install

dont install emtpy libdirs

dont mix tabs and spaces, kthx

remove cvs $Id tag

- add applet taskset to set/retrieve the CPU affinity of a process
   text    data     bss     dec     hex filename
    584       0       0     584     248 taskset.o.gcc-2.95
    509       0       0     509     1fd taskset.o.gcc-3.3
    505       0       0     505     1f9 taskset.o.gcc-3.4
    506       0       0     506     1fa taskset.o.gcc-4.0
    498       0       0     498     1f2 taskset.o.gcc-4.1
    495       0       0     495     1ef taskset.o.gcc-4.2-HEAD

Tito writes: this patch fixes two potential bugs with the -i -I flags that could be reset during option parsing

- fix the build if we have ftpput without ftpget (speeling reeoe)
  Thanks to Stephane Billiart

Callers to identify() converted the endianness of the buffer.  So did
identify().  This meant big endian systems had a NUXI problem.  Removed
the redundant conversion from the callers, and made some in-passing cleanups
while I was there.

Remove a config entry that leaked in from my tree before I decided to
implement cat -v as a separate applet.

Whack the one last warning in make allbareconfig...

import support for microblaze relocations from uClinux-dist

make the default prompt suck less when using fancy prompts

fix from uClinux-dist for proper exit status if reading /proc/modules failed

merge blackfin/microblaze from uClinux-dist

use vfork() instead of vork() since the only thing we do is execve() an app after fork (allows time to work on no-mmu)

only check __GNU_LIBRARY__ if it is actually defined

Header cleanup on two more networking files (move libbb.h to the top and
remove #includes that libbb.h already does), plus a minor cleanup of
libbb.h to move #includes towards the top of the file where we can see 'em.

Confirming it's not needed.

- add note about need to unify the 4 itoa() implementations.

- add testcase for grep bug (busybox.net/bugs/view.php?id=887)
  The patch for bug #887 seems to work for me..

- add note about ingroup and is_a_group_member

- add note about removing superfluous fmt specifiers

- note to cleanup fprintf(2,...) -> bb_error_msg(...) and
  perror("bar") -> bb_perror_msg("bar");

- use bb_msg_{read,write}_error where appropriate.
   text    data     bss     dec     hex filename
 825015    9100  645216 1479331  1692a3 busybox.old
 824919    9100  645216 1479235  169243 busybox

- use bb functions instead of handrolling stuff.
- add XXX question about supposedly superfluous else clause
   text    data     bss     dec     hex filename
   5918       0       0    5918    171e editors/sed.o.orig
   5901       0       0    5901    170d editors/sed.o

Remove #ifdeffed out code, both KEEP_UNUSED and #if 0.

- remove superfluous call of stack_machine(0) upon exit.

- no need to have timer_duration in the data section.
- cleanup a little bit while at it.

- use bb_msg_write_error in 3 places
- whitespace

- move #include busybox.h to the very top so we pull in the config
  and eventual platform specific includes in early.

- move #include busybox.h to the very top so we pull in the config
  and eventual platform specific includes in early.
- remove two supposedly superfluous newlines from ...error_msg() in modprobe
  and use shorter boilerplate while at it.

- patch from Yann E. Morin: makes modprobe understand shell patterns
  (especially '*') in module aliases, such as:
  "alias usb:v0582p0075d*dc*dsc*dp*ic*isc*ip* snd_usb_audio"

Fixes bug #889

 842162   10244  645924 1498330  16dcda busybox.old-4.1.20060603-1948
 842178   10244  645924 1498346  16dcea busybox.new-4.1.20060603-1948

2006-06-02  Shaun Jackman  <sjackman@gmail.com>

* shell/lash.c (pseudo_exec) [_NEWLIB_VERSION]: Remove the
__getopt_initialized hackery.

- No options should set the current system time.
Shaun Jackman writes:
A bug introduced in svn 11946 broke rdate. It no longer sets the
current system time when no options are specified. The options have
the opposite sense from what one might think, and, oddly enough, -ps
is intentionally a no-op.

Quoth rdate(8) from the BSD System Manager's Manual:

    -p      Do not set, just print the remote time
    -s      Do not print the time.

Patch from Atsushi Nemoto:
This patch make depmod.pl more robust in cross environment.  If native
nm did not recognize target's object, you can override it with NM
environment variable.

- move #include busybox.h to the very top so we pull in the config
  and eventual platform specific includes in early.

Header cleanup: don't #include headers that libbb.h already includes.

#ifdef reduction.

- fix CONFIG_DEBUG_INIT compile failure

use xmalloc()

fix fdflish typo (reported by erik hovland)

- look at bb_applet_name. Should close bugs #892 and #893
  also restructure code a bit so the fix only adds 1 byte..

- shrink logread a bit.
   text    data     bss     dec     hex filename
    862      26     188    1076     434 logread.o.orig
    825      26     188    1039     40f logread.o

make halt and poweroff messages more accurate (i.e., claim "we're about to",
rather than "it's done", and combine with reboot handler to save space.

Shaun Jackman pointed out that KERNEL_VERSION() is used without guards, so
defining it in a guard is silly.

The common case APPLET() macro only needs three arguments.

Bugfix from Shaun Jackman: don't attempt to write 64 bytes and then fail if
the write wasn't 192 bytes long.

Add CONFIG_NITPICK.  (And tweak a couple other help entries while I'm there.)

Nothing's actually using it yet, but we need it.

Patch from Tito to remove long options from hdparm.

Use BB endianness macro.

Header file cleanup.  platform.h apparently needs to be included early on so it
can figure out what header files to include, and override stuff that comes
later.  But applets shouldn't include platform.h directly, they should include
busybox.h or libbb.h.  Since busybox.h already includes libbb.h, move libbb.h
to the top of busybox.h and platform.h near the top of libbb.h (right after
bbconfig.h, which is something platform.h also needs access to).

While we're at it, move some stuff from busybox.h to libbb.h so we have one
big file to audit/clean up/try to make sense of instead of many.

Add catv (separate applet instead of cat -v).  Also cleanup cat.c comments,
the following of which (from cat.c) belongs in svn history instead of the
source code:

/* Mar 16, 2003      Manuel Novoa III   (mjn3@codepoet.org)
 *
 * This is a new implementation of 'cat' which aims to be SUSv3 compliant.
 *
 * Changes from the previous implementation include:
 * 1) Multiple '-' args are accepted as required by SUSv3.  The previous
 *    implementation would close stdin and segfault on a subsequent '-'.
 * 2) The '-u' options is required by SUSv3.  Note that the specified
 *    behavior for '-u' is done by default, so all we need do is accept
 *    the option.
 */

- fix IMA compile error i introduced when adding this untested snippet.

ha! a scratch-buffer! ;)
put the isdigit hack into it to help landley out..

- make getopt_ulflags aware of CONFIG_GETOPT_LONG

- add CONFIG_FEATURE_WGET_LONG_OPTIONS

- convert xcalloc(1,x) to xzalloc

- shrink klogd a bit
   text    data     bss     dec     hex filename
    569       0       0     569     239 sysklogd/klogd.o.orig
    537       0       0     537     219 sysklogd/klogd.o

- s/defined(__uClinux__)/BB_NOMMU/
This needs a second pass to:
+ add bb_daemon(unsigned char no_chdir, unsigned char no_close, const char*flag)
+ eventually globally export argc and argv, so we don't need to pass it to
  bb_daemon().

- conditionally define BB_NOMMU

Since we have less than 100k of usage.h data anyway, using bzip -9 is silly.
That says use 900k chunks when compressing, which needs about 4 megs of data
structures to undo the Burrows-Wheeler transform.  Switching it down to
bzip -1 (100k chunks) should have no impact on the compression (since it
still all fits in one chunk) but should reduce runtime decompression memory
requirements to something like 500k.  Still larger than gunzip, but not
egregiously so.

Test to see if I can rename a file without blanking its history...

Note about BB_NOMMU.

- consolidate "Sending SIG%s to all processes"; untested..
   text    data     bss     dec     hex filename
   5379      32       8    5419    152b init/init.o.oorig
   5358      32       8    5398    1516 init/init.o

- this could use a brother which deals with glibc based systems..

- remove useless variable "status" from init_main.
http://www.opengroup.org/onlinepubs/009695399/functions/waitpid.html suggests
that we need not specify a status if we don't want, and we don't.
"If wait() or waitpid() return because the status of a child process is available, these functions shall return a value equal to the process ID of the child process. In this case, if the value of the argument stat_loc is not a null pointer, information shall be stored in the location pointed to by stat_loc.  "
   text    data     bss     dec     hex filename
   5391      32       8    5431    1537 init/init.o.06
   5379      32       8    5419    152b init/init.o

- use config_buffer for message()
- add second argument to waitfor(*action,pid); if action==NULL then use pid tor
  wait for. If an action was given, we wait for the action to finish just as
  before. In run() remove second and third occurance of the same functionality
  the waitfor() call now provides.
  Adjust the former only caller of waitfor accordingly.

PS: Not using waitfor but creating a second function used a few bytes more than
simply extending and reusing waitfor.
   text    data     bss     dec     hex filename
   5426      32       8    5466    155a init/init.o.orig
   5391      32       8    5431    1537 init/init.o

Put parentheses around some of the SWAP() macros, as pointed out by Peter
Kjellerstedt.

Teach bloatometer about .rodata, and tweak the display into something that
has a better chance of getting merged.

- cleanup conftest temporary files.

- DEBUG_INIT -> config CONFIG_DEBUG_INIT

- landley, remove this?

- set_term() doesn't need an argument. (Like r15220 this is just compile-tested)
   text    data     bss     dec     hex filename
   5432      32       8    5472    1560 init/init.o.r15220
   5426      32       8    5466    155a init/init.o

- move common code into new open_new_terminal() function;
   text    data     bss     dec     hex filename
 864434   10244  645892 1520570  1733ba busybox_old
 864386   10244  645892 1520522  17338a busybox_unstripped

add/remove: 1/0 grow/shrink: 0/2 up/down: 92/-143 (-51)
function                                     old     new   delta
open_new_terminal                              -      92     +92
exec_signal                                  345     276     -69
run                                         1527    1453     -74
Again, 48 != 51 so let's trust the former..

- just whitespace

- ls: remove unused variable
- dpkg.c, diff: use xstat
   text    data     bss     dec     hex filename
 848823    9100  645216 1503139  16efa3 busybox_old
 848679    9100  645216 1502995  16ef13 busybox_unstripped
bloatcheck is completely useless as it sees -79 for this, which is bogus.