remove infinity loop ./busybox --help busybox

The previous commit left confdata writing out:

CONFIG_NUMERIC_CONSTANT=

And on reading it back in, it would complain that '' was an invalid value for
that field.  I.E. "make allnoconfig && make" worked fine, but
"make allnoconfig && make menuconfig" barfed reading in the config file.

So now I have it write out "0" as the blank value.  (It's initialized to the
default value when the menu becomes visible anyway; I checked.)  That seems
to work.

Now that "make sizes" is in, the faq should mention it...

Fix the warnings, and fix the following two obvious segfaults:
./busybox getopt -n one -n two woot
./busybox getopt -o one -o two woot

This entire applet is still an enormous pile of garbage, which I can't clean
up because I really have no idea what it's for.  (Both "man getopt" and trying
it out on the command line a bit fail to enlighten me.  Reading the code, the
fact half of it seems to be special cases for bash vs tcsh does not fill me
with confidence.)

Actually returning ret might be good.

Fixes so "make allnoconfig" works again.

The configure system's save function edited out sub-menus that wouldn't be
displayed in the current configuration, meaning config.h wouldn't have #udef
entries for those symbols, meaning bb_config.h would have the relevant
ENABLE_ missing instead of defined to 0.  This broke the build.

So I fixed it, and then reorganized the applets.c and busybox.c to take
away the warnings this revealed (code that would be optimized out was making
calls to functions that hadn't been prototyped.  So I added an #else case
to those #ifdefs to #define the relevant functions to empty macros to
placate the warnings.

I also reorganized the applets.c code to make adding such an #else case less
of a pain (and make the need for prototyping go away by moving the functions
up before they were used, and generally wind up with fewer #ifdefs in
the code by putting all the logic in one place).  This resulted in a huge
seeming patch, when most if it just moves code from one place to another
without touching it...

Upside: make allyesconfig and make allnoconfig should both work now.

Remove trailing crud from busybox.tests.  (Sleep deprivation strikes again...)

General cleanup of command line parsing to allow "busybox" to work as a prefix.
(I.E. any argv[0] that starts with "busybox" winds up in busybox_main().)

Added testing/busybox.tests which tests the following permutations:

./busybox
./busybox-suffix
./busybox cat
./busybox-suffix cat
./busybox --help
./busybox-suffix --help
./busybox --help cat
./busybox-suffix --help cat
./busybox --help unknown
./busybox-suffix --help unknown
./unknown

Also repair the test suite so ./runtest calls the ".tests" scripts properly.

Note: you can now go "busybox busybox busbox ls -l" and it'll take it.  The
new code is pretty generic.  I can block that if anybody can come up with a
good reason to...

touchup some grammar, formatting, and remove some outdated notes

update the verbage describing what is needed for developer access

This goes along with the new testsuite code...

Working on a new test harness.  Moved the sort tests into it.

Missing break was screwing up 'y//' command.  Bug #248.

Fix for bug 383: attempting to "tar c /" would error out because stripping
trailing / turns that into an empty string.

According to bug #63, crond is unhappy with crontab lines that don't end in a
newline, or lines that have trailing spaces.

Tracking system but #4: php needs the environment variable SCRIPT_FILENAME.

Bernhard Fischer trimmed down dos2unix a bit.

Patch -i support from Berhnard Fischer.

Bernhard Fischer suggested that  get_terminal_width_height() should return
the result of the ioctl so callers can tell if we have a tty.  (0 means
we have a tty, nonzero means the ioctl couldn't find size info and we
fake 80x24.  Really we should fake 80x25, but oh well...)

Off by one error in max_leases sanity check.  Bug 349, apparently.

Bernhard Fischer sent a patch to make "make sizes" work when building in
another output directory.

Busybox should only multiplex based on argv[1] when it's called as "busybox".
Otherwise if you build busybox without a given applet you get the wrong error
message when you call it via a symlink to that applet.

(You also get the wrong behavior; it tries to use argv[1] as the command
name just like busybox does for _any_ unknown, and although I doubt
"echo rm -rf *" is common usage there's no upside and enough downside to
make me nervous.)

This fixes it.

Anand Avati hit an integer overflow problem in our unzip code.

Don't comment warnings, _FIX_ warnings.  (And putting in #warnings about
other warnings is just gross.)

On a side note, while I was there, I made the code slightly smaller.

Why have a separate CONFIG_INIT_SWAPON when we already have CONFIG_SWAPONOFF?

Bug system entry #1 says that when busybox doesn't have swapoff,
init shouldn't call swapoff.

Amir Shalem found some bugs in the new mount code; unknown options didn't get
added to the list, and my assumption that nfsmount() actually called
mount() was incorrect (and I coded it wrong anyway; I hate having to touch
codepaths I can't personally test).

Dirk Clemens pointed out how easy it is to support bzip2 compression, since we
shell out to an external program to handle gzip anyway...

point to busybox.net

Dumb little hack, "make sizes".  Try it.

make sure __NF_sysfs exists before trying to use it

some portability fixes by rmh in Bug 395

Bernhard Fischer submitted a couple of Makefile patches:
- Fix building out-of-tree
- remove duplicate rule in toplevel Makefile
- peruse make's builtin notion of `dirname $@'

Remove something we did, not that I'm doing something, and add something we
haven't done.  Woo.

When compiling with FEATURE_MTAB_SUPPORT disabled, the call to erase_mtab()
can never be made because useMtab is initialized to 0, and all the other
assignments of that variable assign 0 to it.  Any compiler that can perform
simple constant propogation on local variables will optimize away if statements
testing against that variable, thus the call to erase_mtab() will never be
made.

When compiling for arm using gcc 3.3.3 with FEATURE_MTAB_SUPPORT disabled,
the linker complains that it can't find erase_mtab().  The arm optimizer isn't
exactly the brightest member of the family, and apparently needs to be hit over
the head with a hammer to get its' attention...

Two FEATURE_CLEAN_UP patches from Shaun Jackman, closing filehandles.

Bernhard Fischer provided a mountpoint(1) applet.  This is apparently something
sysvinit provides, and which is used by the debian init scripts.

Announce busybox 1.01.

Tito (among others) writes: previous patch was wrong, please stop screwing things up

When doing 'make release' nuke the .svn directories, not the
CVS dirs (since they are no longer present)

the () around isspace actually matters

Bernhard Fischer says: use xmalloc() instead of malloc()

Bernhard Fischer says: fix comment typo

Bernhard Fischer writes: fix devfsd usage

If we goto singlemount, do _not_ try to continue through the loop we jumped
into.  (That means "mount -t ext2 /dev/thingy thingy" would segfault if
it failed instead of giving us an error message.)

Patch from Stephane Billiart to improve ash error message.

Stephane Billiart noticed that the return code of the new mount was horked.
(Cleaned up the return code path to consistently use numbers so I don't keep
confusing EXIT_SUCCESS with TRUE and EXIT_FAILURE with FALSE...)

Very minor tweak to tail.

make sure distclean punts include/bbconfigopts.h

Major rewrite of mount, umount, losetup.  Untangled lots of code, shrunk
things down a bit, fixed a number of funky corner cases, added support for
several new features (things like mount --move, mount --bind, lazy unounts,
automatic detection of loop mounts, and so on).  Probably broke several
other things, but it's fixable.  (Bang on it, tell me what doesn't work for
you...)

Note: you no longer need to say "-o loop".  It does that for you when
necessary.

Still need to add "user mount" support, which involves making mount suid.  Not
too hard to do under the new infrastructure, just haven't done it yet...

The previous code had the following notes, that belong in the version
control comments:

- * 3/21/1999   Charles P. Wright <cpwright@cpwright.com>
- *             searches through fstab when -a is passed
- *             will try mounting stuff with all fses when passed -t auto
- *
- * 1999-04-17  Dave Cinege...Rewrote -t auto. Fixed ro mtab.
- *
- * 1999-10-07  Erik Andersen <andersen@codepoet.org>.
- *              Rewrite of a lot of code. Removed mtab usage (I plan on
- *              putting it back as a compile-time option some time),
- *              major adjustments to option parsing, and some serious
- *              dieting all around.
- *
- * 1999-11-06  mtab support is back - andersee
- *
- * 2000-01-12   Ben Collins <bcollins@debian.org>, Borrowed utils-linux's
- *              mount to add loop support.
- *
- * 2000-04-30  Dave Cinege <dcinege@psychosis.com>
- *             Rewrote fstab while loop and lower mount section. Can now do
- *             single mounts from fstab. Can override fstab options for single
- *             mount. Common mount_one call for single mounts and 'all'. Fixed
- *             mtab updating and stale entries. Removed 'remount' default.
- *

implemented a builtin echo command in ash.  moved the guts of the
echo applet into libbb, and now call bb_echo() from both echo.c
and ash.c

commiting:
    0000025: vi-editing mode for ash

applying jim bauer's patch to eliminate modprobe's dependency
on /bin/sh.  bug #8.
     0000008: modprobe applet is dependent on having a shell

In bb_config.h, replace AUTOCONF_INCLUDED guard with the more reasonable and
obvious (and less side-effect prone in strange build environments) BB_CONFIG_H.

Yeah, I know Erik ripped it out of our copy of menuconfig (which is a good
thing), but that doesn't fix people whose headers have it inherited from
linux-kernel headers or old versions of uclibc, and Erik's fix could easily
get forgotten and reverted the next time we update menuconfig anyway...

Do not define 'AUTOCONF_INCLUDED'

commiting:
    0000028 03-16-05 patch: new setsid applet

fix bbconfigopts.h depend

screw around with whitespace

commiting:
    0000073: Add option to inetd applet to run in foreground
this option was already there for uclinux -- this just exposes
it in the normal case as well.

commiting patch from bug 71:
    0000071: patch: implement "--color" option for ls coloring control

applying patch for bug 72:
    0000072: Add applet to redirect console output via ioctl(..., TIOCCONS)
applet name changed to setconsole, since suse has a very similar
utility.  better to treat differences as bugs than invent a new command.

applying patch from bug 24:
    0000024 03-16-05 patch: allow init to set controlling tty

new config display applet, from bug 46.  i've changed the name
of the applet from "config" to "bbconfig", and renamed the
source filenames and symbols to match appropriately.

allow suppression of default client-id.  fixes bug 000037.  allows
busybox to match kernel and u-boot behavior with respect to client-id.

patch by pkj in Bug 3 to not export all make variables

patch by apgo in Bug 345 to not use ether_hostton() with uClibc

patch by srowe in Bug 359 to fix fgrep aliasing

typecast to int

use socklen_t instead of size_t with socket related functions

move -s from LDFLAGS to STRIPCMD

use defines instead of const integers

shrink the flag setting by reversing the operation

eat leading whitespace with preprocessor

replace for loops with one function call

Ahem: add new faq entry to list at top of FAQ.html

Fix "nm --size-sort" on the busybox binary and document how to make it work.

only declare do_swap if we need it

touchup syntax

cleanup the error message so we dont use glibcism of %m and cast a pointer to an int (bad on 64bit arches)

cleanup syntax

tell people to ignore the dups/e0 warnings

punt unused variable

tell people to ignore the save_name warning

in order to make sure the INET6 prototypes are exported, we would have to include the libbb.h header file ... but then we have to worry about including too early/etc...
since simply defining the prototype in an internal header file doesnt hurt anyone, lets not worry about the header file crap and just always prototype the inet6 functions

apgo in Bug 322 writes: kill off CONFIG_FEATURE_MTAB_FILENAME

rename the accept/reject names since accept overrides the accept() socket function

apgo in Bug 351 says: punt parse_version_number since it is no longer used

commiting fix from:
     0000353: [PATCH] start-stop-daemon -q is not quiet

further refine check_gcc usage so it is only called the bare min number of times

dont output anything when running in silent mode

From Rich Paul:

I've noticed that when I compile busybox on my laptop, it compiles more
slowly than one would expect, and although it's a (more-or-less)
multiprocessor system and I use -j5, make never seems to run more than
one job at a time.

I believe I have found the culprit:  each time a file is compiled, gcc
runs about 5 times.  This is because the $(check_gcc) macros and the
TARGET_ARCH macros are late binding.

The attached patch cuts the compilation time by 66%, from 1.5 minutes to
30 seconds.  Your mileage may very.  These statements have not been
evaluated by the FDA.

Erik pointed out that in the last try at the #ifdef cleanup
infrastructure, the compiler isn't smart enough to replace const static
int with the constant, and allocates space for each set of them,
bloating the executable something fierce.  Oops.

So now, we #define ENABLE_XXX to 0 or 1 for each CONFIG_XXX (which
is still there so the 1000+ #ifdef/#ifndef tests don't have to be
replaced wholesale).  Changed the test instance in networking/ifconfig.c
to use this.

move the ifdef to after libbb.h include, so it can do some good.

#ifdef reduction infrastructure, based on an argument between Shaun Jackman,
Rob Landley, and others.

Currently CONFIG options are defined or undefined, so we chop out code with
#ifdefs, ala:
#ifdef CONFIG_THING
  stuff();
#endif

This creates a new header file, bb_config.h, which sets the CONFIG entry to 1
or 0, and lets us do:

  if(CONFIG_THING) stuff();

And let the compiler do dead code elimination to get rid of it.  (Note: #ifdef
will still work because for the 1 case it's a static const int, not a #define.)

The change in getty.c in Busybox 1.01 caused the /etc/issue file to not
be displayed unless CONFIG_FEATURE_UTMP is set.  This was not the intended
result.

use toplevel ARFLAGS and update default ARFLAGS to be quiet

2005-04-05  Shaun Jackman  <sjackman@gmail.com>

* libbb/printf.c: Check for __NEWLIB_H__ before __GLIBC__.

2005-03-19  Shaun Jackman  <sjackman@gmail.com>

* networking/ping.c (ping): Change the type of fromlen to socklen_t.

remove unused variable ret as reported by apgo in Bug 350 and touchup syntax along the way

no longer used as reported by apgo in Bug 348