- 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.

More size shrinkage.

Size reductions, mostly switching things to use libbb functions.

Add SWAP_LE?? and SWAP_BE?? macros, and make things use them.  Converts values
to/from little endian or big endian, which is a NOP if that's what the current
platform already is.

This is not C++.

A few new todo items from email with Bernhard.

Remove _() and N_() from platform.h.  #define them as NOP macros in the two
files still using them.  I didn't remove them from e2fsck.c to avoid stomping
pending cleanup patches from Garrett, and I didn't bother to remove them from
fdisk.c because that entire file needs to be rewritten from scratch.

Remove _().

Remove _().

I changed last10 to last25, and just bumped it up to 50 so I renamed it
"recent.html".  There's a symlink from each of the old names for the moment,
I'll zap those eventually...

Fix hdparm to use PRIu64 instead of typecasting to long long (which is 128 bits
on 64 bit platforms), and move #include <inttypes.h> to libbb.h.

Put the ending quote on the help entry...

Fix some warnings in allbareconfig.

My first bout of untangling udhcp.  Make lots of gratuitous #defines go
away, substitutie BB_VER for an external VERSION, use busybox CONFIG symbols
rather than checking for them then defining others, etc.  Lots more cleanup
to do...

Move portability stuff to platform.h, and clean up adjtimex.c a bit while
I'm in the area.

Consolidate #include <sys/time.h> so libbb.h does it.

Update FAQ to mention make baseline when talking about make bloatcheck.

Consolidate the .PHONY targets and add "make baseline" for bloatcheck.

- use the defines CURRENT_TTY and VC_1; also shrink xargs a tiny bit:
add/remove: 0/0 grow/shrink: 2/6 up/down: 5/-37 (-32)
function                                     old     new   delta
static.add_interface                         271     274      +3
static.glob3                                  36      38      +2
static.renice_main                           409     408      -1
static.readcmd                              1083    1081      -2
static.changepath                            176     174      -2
static.parse_opts                             56      51      -5
static.rx_main                              1095    1088      -7
static.xargs_main                            759     739     -20
----------------------------------------------------------------
Result                                                :-)    -32

Change llist_add_* to take the address of the list rather than returning the new
head, and change all the callers.

Slight sanity fix: data is void *, not char *.  And it's called data
almost everywhere, so be consistent.

- provide fallback defines for non-linux

- provide fallback defines for non-linux

- add central knob to turn off getopt_long everywhere. EXPERIMENTAL!
  Adds "Enable getopt long" under "General options", default y.
  Send patches to fix getopt_ulflags and run_parts.c if you turn this off..
  See http://busybox.net/lists/busybox/2006-May/021828.html for a start to run-parts

- use strtol instead of strtoll if the latter does not exist
- add and use wrapper for attribute
- add and use replacement for vasprintf if it is unavailable

- introduce and use bb_path_wtmp_file for portability (saves 11 Bytes).
- fix last.c to also look at the double-underscore UT_ defines.

- use portability wrapper define bb_setpgrp.

- work around bug in gcc-3.4.x on ARM

- move llist to top of file so we can potentially use it in all prototypes easily

- make find_mount_point conditional on df and eject; Fixes `make hosttools´ on
  hosts where mntent.h does not exist.

- add workaround for tar being broken since it uses a non-portable constant.

- use the SED given by the user
- use the portable `` instead of $(). There is no bbsh, so this is needed.

- add target hosttools (see make help)
- rename check_gcc to check_cc and pass the CC to use as arg#1; peruse check_cc for HOSTCFLAGS
- add and use check_strip
- add checks for {,no-}whole-archive and {start,end}-group LD flags

- use ATTRIBUTE_ALWAYS_INLINE if requested
- use shorter boilerplate while at it

- move libc checks from busybox.h to platform.h
- add ATTRIBUTE_ALWAYS_INLINE, endian handling for DEC UNIX, some more
  compiler dependent defines to platform.h
- add conditional bb_setpgrp define to platform.h
- remove superfluous specifying args from "#define fdprintf dprintf"

- patch from Tito to pretty print the overall delta

Yann E. MORIN. writes:
hdparm has a uint64_t passed to printf with a %llu format (which is ok, because
it's the same size), but without first being casted(?) to unsigned long long int.
Which is kinda funny because it's done OK on a few lines above...

errno and h_errno values are not the same #876 by rfelker

fix spelling mistakes

check for queueing dependancy dependancies typos

queuing is spelled correctly

Rich Felker suggested removing dprintf() from watch, and one thing led to
another... This adds bb_xspawn() support, which does vfork/exec.  (I don't
know why using a static instead of a local adds ~40 bytes, but using
the local doesn't work...)

And fix the error I just introduced. :)

Remove pointless #define.

Patch from Tito:
Remove -v verbose -V version -q quiet flags
Remove parse_opt_v2();
Rename parse_opt_v3() to parse_xfermode()
Remove if_strcat()
Rearrange code in  dump_identity() to reduce size
Modify some strings so that they could be optimized by the compiler
Minor code clean up
Remove unneeded #define <shm.h>

#if BB_BIG_ENDIAN && !defined(__USE_XOPEN)
# define __USE_XOPEN
#endif    
Must be before #include <unistd.h> and #include "busybox.h"
(noticed by Bernhard Fischer <rep.nop@aon.at>)

Remove duplicate code in do_time()

Remove a pointless #define.

make the testsuite a little less brittle:
- 'function fn_name\n{' breaks on older FreeBSD default shells, so use the more
  widely supported 'fn_name () {'. This needs more fixing..
- test for integers ought to use the proper operators
- test for strings ought to use quoting of the strings to be fair to strange
  implementations of test(1)
- make sure not to ignore return-codes != 0 from commands; Some shells exit
  immediately on this (much like explicitely requesting set -e in e.g. bash)

TODO:
*) Some older shells do not allow a space after the test-condition in an "if"
statement. This doesn't work:
if [ $status -ne 0 ] ; then
as opposed to this:
if [ $status -ne 0 ]; then
or this
if [ $status -ne 0 ]
then

*) strict spacing between commands. In some shells you have to say:
foo ; bar ; baz
The affected shells barf on stuff like ommitting the space, so this doesn't
work:
foo; bar   ;baz

*) $() vs. ``
The former isn't really portable as opposed to the latter.

*) fix frong assumption that the testsuite is run from the source-dir.
This is a complete misconception and renders the testsuite completely useless.

That said, i note that IMO a test-harness ought to do it's best to work in
a wide variety of environments, everything else defeats it's purpose.

Two more issues pointed out by Bernhard.

Yann Morin spotted some leftover flotsam from halfway through the recent
llist.c interface change that accidentally got checked in.  My bad.