The if() is cheaper than the extra function parameter.  According to make
bloatcheck, this saves 112 bytes on x86...

Remove a link that leaked in from the pending llist_t changes.

re-enable udhcp in a few more places

add back in udhcp support

Shrink the code about 50 bytes, allocate less run-time memory, and add a
comment that null terminating the string we sscanf() shouldn't be required
since the kernel adds \n to the end of it and sscanf will stop there.

More busting on legacy mtab support in the help text.

Cleanup patch from tito.

The udhcp was deleted from busybox in 14634.  Bolting an external project to
the side of the tree doesn't _COUNT_, and I will not ship it.

Udhcp was deleted shortly after I posted my philosophy for what should and
shouldn't go into busybox:

http://www.busybox.net/lists/busybox/2006-March/019484.html

I complained about the change t the time.  I've complained repeatedly since.
But nobody felt like fixing it.  External dependencies are something to be
minimized.  I don't care about the ability for packages to build outside
busybox: something is either part of busybox, or it isn't.  If I convert any
part of the external udhcp repository to use libbb, I've broken the external
package.  Any random cleanups that touch that directory suddenly have to worry
about external dependencies that are NOT OUR PROBLEM.  Therefore, that
directory is not and cannot be part of busybox.  Wishful thinking isn't going
to change that.  I will not ship something I can't maintain.

I'll try to get a new dhcp client and server in before the ship window closes,
but I have a half-dozen other projects pending.  I'm sorry this happened, but
I'm not the one who removed it, and I'm not the one who ignored the project
maintainer's repeated complaints about the situation for the next month and a
half.

Patch to fix bug 868, and some related cleanup while I was in the area.
A tab is now taken as the end of filename if it's there, but if it isn't
(because the timestamp isn't there) we continue with the existing untruncated
line as the filename.

Script to easily get an assembly dump of a single function.

Remove bb_strlen() in favor of -fno-builtin-strlen.  Saves as many bytes
as the old optimization did (actually does slightly better under gcc 4.0), and
simplifies the code.

Add "make bloatcheck" to run Matt Mackall's bloat-o-meter.  Copy an old
busybox_unstripped to busybox_old, build a new version, and "make bloatcheck"
to see a detailed breakdown of the size difference.

Dennis Vlasenko pointed out that run_applet_by_name sets bb_applet_name already.

fix udhcp again

make sure script is executable

Add an "optional" line for FEATURE_READLINK_FOLLOW.

New test from Natanael Copa.

didnt mean to commit debug enable

use %p when printing pointers, not %x

fix Bug 659 as reported by Robin Getz

Busybox's udhcp was deleted a while ago.  Bolting an external project onto the
tree doesn't change that.

Another patch from Garrett: remove bad block bitmap checking code.  Everything
produced in the list ten years has some variant of internal error correction
(disks, cdrom, flash), so if it has user-visible bad blocks on it the
hardware has exhausted its remapping reserve and is dying, and you need to get
your data off pronto.  (The one exception I can think of is floppies, and I
don't care.)

Migrate endianness macros.

Patch from Dennis Vlasenko resetting values that get reused.

Patch from Dennis Vlasenko simplifying the APPLET macros.

Patch from Jason Schoon to make mount -a not abort on the first failure.

- ignore missing helptext for options which are not selectable.

- add helptext for CONFIG_UNIX2DOS

Not quite compiler independent, but we've never tried to be gcc independent
anyway.  This is at least less ugly than what was there before, and fixes
building all sources at once.

It was sitting there, in the patches directory, for years.  It was delete it
or apply it.  It's small, simple, evil, part of SUSv3, and we can switch it
off.

There aren't any bzip2 based RPMs.  If people want to experiment with new
packaging formats, there's ipkg and such...

CONFIG->ENABLE migration.

Fix allbareconfig.

- add script to check for missing help entries of config options
  Currently we have these errors:

./modutils/Config.in: No helptext for 'CONFIG_FEATURE_QUERY_MODULE_INTERFACE'
./networking/Config.in: No helptext for 'CONFIG_IPADDR'
./networking/Config.in: No helptext for 'CONFIG_IPLINK'
./networking/Config.in: No helptext for 'CONFIG_IPROUTE'
./networking/Config.in: No helptext for 'CONFIG_IPTUNNEL'
./coreutils/Config.in: No helptext for 'CONFIG_UNIX2DOS'

Big dead code elimination pass from Garrett.

Build break fix from Bernhard fischer, fixing fallout from svn 14974.  (My bad.)

Whitespace and documentation cleanup from Dennis Vlasenko.

Patch from Rich Felker to make ls use libc's qsort.

Rich Felker writes:
mistake in a macro caused statement before declaration.

Patch from Garrett Kajmowicz to move a lot of #defines into the
header file.

Remove HDIO_GETGEO_BIG which is dead (removed from 2.6) and was pointless back
in 2.4 (it fetched the same hd_geometry struct only with uint cylinders instead
of ushort cylinders: nothing that big has ever accurately reported gemoetry,
it's all LBA.

Also stop declaring the hd_geometry struct static: it's two chars, a short, and
a long for a grand total of 8 bytes.  That can go on the stack.

Use ATTRIBUTE_NORETURN as suggested by tito, plus a few in-passing cleanups.

Patch from Jason Schoon to add optional SIGUSR1 support to dd.

Minor tweaks: remove traces of old $COMMAND environment variable.

Patch from Shaun Jackman:

ls has an ugly bug. ls uses an array of pointers, the elements of
which are all in a linked list. To free the elements, instead of
freeing all the elements in the array, array[0..nelements], it frees
by iterating the linked list starting at array[0], which it assumes is
the head of the list. Unfortunately, ls also sorts the array! So,
array[0] is no longer the head, but somewhere in the middle of the
linked list. This patch fixes this bug, and also adds an
ENABLE_FEATURE_CLEAN_UP stanza.

Patch from tito removing obsolete code.

Notes on portability, and on when #include <linux/blah> is appropriate.

Fixup a build break if ENABLE_SHOW_USAGE switched off.  (Fallout from the
compressed usage messages patch).

Patches from Denis Vlasenko to rework "current_name" handling and remove
print_current_name().

Patch from Denis Vlasenko:
* Do not initialize globals to 0, it is done automatically
* unsigned short -> uint16_t,  unsigned int -> uint32_t
  where appropriate (did it ever work on Alphas?)
* triple sync() is silly - removed
* check_zone_nr uses check_zone_nr2 now
* remove trailing periods from messages, uppercase first letter

Patch from Denis Vlasenko:
ix bug (inode_map and zone_map are char* pointers,
sizeof() on them gives 4 or 8)

Remove some unused code pointed out by Tito, plus a slightly more graceful
way of handling endianness.

Style fixes from Denis Vlasenko.

Another patch from Denis Vlasenko:
Make hdparm smaller:
* Make bb_ioctl return the status
* Replace ioctl with bb_ioctl in a few places
* Add bb_ioctl_alt, use where appropriate (four places)
* unsigned char args1[4+512] = {WIN_IDENTIFY,0,0,1,};
  ate 0.5k of rodata, fix that

From Dennis Vlasenko, Make hdparm smaller:
 * Use combined no_scsi_no_xt()
 * Have common unsigned char args[4] = {WIN_SETFEATURES,0,0,0};

Cleanup to hdparm by tito.

termios.h is already included so dont include sys/termios.h as well

detect more obsolete headers

just include fcntl.h not sys/fcntl.h

just include fcntl.h not sys/fcntl.h

Rob Sullivan writes: rewrite the regex handling to improve stability

compile.h is incorrect; create a new do_link.h and use that for bb_mkdep and usage

use pretty output for scripts/bb_mkdep and scripts/usage

- rerun gcse pass after reload, enable whole-program iff we are not building
  the lib (due to bug in gcc; see comment.)
I would have run the testsuite to check if everything is still behaving, but
unfortunately i cannot run the testsuite anymore: it just sits there and does
nothing even with a pristine checkout and an old gcc.
srcdir=/tmp/busybox/testsuite /tmp/busybox/testsuite/runtest
PASS: Applet order
FAIL: Common typos
PASS: Obsolete function usage
PASS: Obsolete function usage
<spins, ^C>
make[1]: *** [check] Interrupt
make: *** [check] Interrupt

I've just corrected that wrong double-printing.

Anyway. with gcc-4.1-HEAD from earlier today, we now have for
defconfig+ COMPILE_AT_ONCE:

880 -rwxr-xr-x  1 433 433 893476 Apr 28 21:41 busybox.oorig-4.0
860 -rwxr-xr-x  1 433 433 874560 Apr 28 21:36 busybox.oorig
844 -rwxr-xr-x  1 433 433 858752 Apr 28 21:49 busybox

   text    data     bss     dec     hex filename
 879920   11568 1039148 1930636  1d758c busybox.oorig-4.0
 862802   10192 1038796 1911790  1d2bee busybox.oorig
 848066    9100 1037536 1894702  1ce92e busybox

values for 4.0.2 just added for reference..

- print "Obsolet headers" instead of "Obsolete function usage" when checking
  for obsolet includes.

More usage.h cleanups, with collateral changes to httpd.  Specifically,
most of our CONFIG entries switch stuff on instead of off when enabled, so fix
the gratuitously reversed INETD thing.

Patch from tito to update output of hdparm.

Patch from Dennis Vlasenko to add the option to compress help text.

Re-wordwrap the new tests, and fix seq.c to pass them.

- add two regression tests for seq.
  "seq two arguments equal, arbitrary negative step" fails but should pass..

- add note about crond

- patch from Denis Vlasenko to make sizes PHONY and to add objsizes which prints
  the sizes of the individual object files.

Convert a chunk of usage.h to USE_ and SKIP_ (more to do there), and fix a
DEFALT typo in du while I was there.

Portability tweak from Shaun Jackman, don't include asm/page.h directly.

From Denis Vlasenko:
* CPU% = process[i].ticks/sum(process[i].ticks) * busy_cpu_ticks/total_cpu_ticks
* got rid of empty line under "Load average" line
  and used last line - +2 visible processes
* do not do float conversion of loadavg, just read it as string from /proc
* fix display on small screens
* dropped unused .stime, .utime fields
* a few variables were renamed
* style fixes

- return ret so we return something meaningful.

Jason Schoon writes: Attached is a patch to display the usage message if fuser is called with no arguments, rather than doing no work and failing silently.

Jason Schoon writes: This makes the case where LFS is not defined, but fdisk is configured with large disk support use the pretty build rules.

2006-04-20  Shaun Jackman  <sjackman@gmail.com>
        * Makefile: Build a bFLT binary.
        * Rules.mak: Ditto.

- passwd doesnt use salt with md5 passwords; bug #604 thanks taviso

kill signed warnings

Patch from Jean Wolter:

it looks like the introduced support for character classes and
equivalence classes is not correct. The attached patch tries to fix
some symptoms and tries to make tr behave like gnu tr for the added
test cases. The patch

 - removes if clauses with side effects
 - fixes handling of buffer pointer (strcat added characters to the
   buffer without increasing the buffer pointer)
 - re-arranges character classes to match ASCII order

regards,
Jean

svn 14824 broke -funsigned-char and possibly building out of tree, because
$(srcdir) can't bind early since it's set to a different value in each subdir.
If it binds early, it's blank, hence an empty -I which eats the next option
as a directory to look for #include files in.

So CFLAGS has to bind late, but the check_gcc stuff should only get run once.
I added a "make V=2" mode to show when check_gcc gets run, and turned WARNINGS
into a CFLAGS line since it was always getting added anyway.

New applet cksum, from Rob Sullivan.

Remove some #if 0 code.

Simplify the GPL boilerplate.

- typo: s/derefernce/dereference/g
  Thanks to P.J. Day.

- commentary typo and superfluous whitespace removal in weird messages.

- improve wording in some help texts a little bit.

- add typo s/deamon/daemon/g

- patch from P.J. Day to rely on the automatic indenting provided by depends.

- enable dumpleases applet again; bad rob

- add grep for typo s/algorithic/algorithmic/g

- BusyBox is under GPL.
  in http://www.busybox.net/lists/busybox/2006-April/020364.html Denis Vlasenko
  said when asked which license to use for these files:
  "Sure. LGPL or GPL is fine with me."
  Adjusting accordingly.

- simplify config options of the ip applet.

From Rob Sullivan: Fix a segfault with searching, plus some cleanups.

Robert Eggers noticed that nfsmount would sometimes incorrectly return success.

Glenn gave permission for GPLv2 "or later".

Patch from Tito to fix printing null and make output look like hdparm 6.3.

Remove a superfluous quote and more non-generic standalone code.