Bernhard Fischer:
[busybox][patch][janitorial] correct typo in news.html and make site html compliant

BsAtHome writes in Bug 433:
Ping packets sent by busybox have wrong endian on f.x. mips32 (openwrt). Attatched is a patch that
uses htons() and ntohs() to be platform independent.

split libbb: moved xregcomp separatelly for speed up recompile

removed strange extern void xregcomp(... from some applets, but declared from libbb.h

FEATURE->ENABLE_FEATURE loses one for 'top' applet

Patch from Berhnard Fischer to keep duplicate objects out of the library.

remove unrequired dependences

Cleanup patch from tito: migrate option parsing to the bb_ functions and fix
types.

Cleanup patch from Tito.

bb_mkdep version 2.0. speed up *2, remove problem of find e2fsprogs/uu*.h, spelling corrections by Bernhard Fischer

Our config should write out each symbol once and only once, but still write
out all symbols in all sub-menus.  I think this finally does it right.

Test full susv3 spec for uniq (except internationalization).

I think this covers it.  We fail two corner cases, both of which are explicit
violations of the spec, and both of which gnu passes.

committing bug #7:
     0000007: which and wd-located files
     which doesn't search $PATH when there's a file in the WD with
     the same name of the 'filename' parameter...

initial ppc64 support

allow either backspace or DEL, in addition to the user's erase
char, to be used in get_input_line()

bb_mkdep release. speed up *3, clearing, more comments

whitespace/syntax cleanups, dont mind me :p

touch up usage for swap utils

Cleanup patch from Tito.

sysctl was seriously broken.  And since in some cases 'CONFIG_SYSCTL'
could be included into application space via the kernel config option
of the same name, nasty compilation problems could result.

Kunihiko IMAI writes:

We're using busybox and thank the authors.

I found a short buffer allocation at busybox/sysklod/syslod.c.  It
mis-declares length of filenames, seems to forget for '\0' or two
column numbered suffix of backup files (ex. messages.10).

Here is a patch for the problem.

Thanks.

bb_mkdep speed up * 10!

new my scripts/mm_mkdep, dependences work now

Clean up memory allocation.

This combines a patch from Bernhard Fischer (moving the clean of
scripts/config from distclean to clean) with a sed consolidation that's
been in my tree for a bit, and switching the GPL boilerplate to just point
at LICENSE.

Cleanup patch by Bernhard Fischer, removing unnecessary includes of
getopt.h, whitespace changes, typos, etc.

use CONFIG_BUFFER macros like cow suggested and shrink code size some more by using less variables and more gotos ;)

Patch from Dirk Clemens so df doesn't run numbers together on drives >100gig.

some tweaks by cow to shrink a little

Put the menu_is_visible test back because it avoids printing spurious
(albeit commented out) garbage.

Rob, I don't know why you feel some unexplainable compulsion to get rid of
a meaningless warning.  But I do expect you preserve the coding style
and variable names when all you're doing is tweaking some of my code.

I repeat... do NOT change whitespace, variable names, or coding style in
any of my code simply to conform to your coding style.

A bit overdue...

Tito sent more size tweaks.

Fix the warning by rewriting the function to be smaller and simpler.

I'd appreciate somebody on a __BIG_ENDIAN platform testing this out; I haven't
got the hardware...

Tito sent in a few patches converting instances of CONFIG_FEATURE_CLEAN_UP to
ENABLE_FEATURE_CLEAN_UP.

committing:
       Summary      0000242: ash: read -t broken
this also implements -n and -s options to read.  (they're configured
together because most of their code is in common, and separating them
seemed silly.

Yeah, Linus is the original author of three of our apps, but A) he doesn't
work at transmeta anymore so that email's bogus, B) hands up everybody who
thinks emailing him about a problem in the busybox version of those apps
would be productive?

Fix warnings.

This code exists to compensate for ancient broken build environments which,
as far as I can tell, are no longer relevant.  Modern busybox refuses to
build under libc5 (there's a specific test and #error for that), and
I'm not sure building against 2.1 kernel headers on Alpha was ever relevant.

I'm happy to put any of this back if anybody can point to a real need for it,
but if so we need to specifically document what environment is being
compensated for.  (And we should quarrantine the build environment code
into one place, anyway.  Maybe "quirks.h" for known compiler and
libc quirks?)

Don't comment warnings.  Don't put #warnings about warnings.  _FIX_ warnings.

This fixes the warning, and makes the binary smaller out of sheer pique.
(Yes, since Manuel did this one it's nice tight code that took several
attempts to shrink, but I was ticked.)

Add the start of a test for uniq; this is about the first 1/3 of the
tests we need for full susv3 coverage of uniq.

Amir Shalem writes:
this patch fix syscall names mix-up in libb/syscalls.c

- sync traceroute with Slackware-10.1
  (support -FIl -g gw -i if -z pt now).
- libbb/getopt_ulflags.c support bb_opt_complementally="x-x" as trigger now

typo, thanks, Bernhard Fischer

Whitespace-level changes.  Replace s0,s1,e0,e1 with real variable names and
clean up whitespace and curly brackets a bit.  Resulting binary should be
identical.

get_terminal_width_height.c was in there twice.

Bernhard Fischer pointed out that CONFIG_FEATURE_MOUNT_FORCE no longer exists.

Regarding 11330: when you fix a bug that the testsuite doesn't catch, add
a test case to the test suite.

destroy bug [0000381]

destroy bug [0000404]

- usage fresh bb_getopt_ulflags for ls -- very size reduce.
- small indent corrections
- remove unecessary malloc and free

1) bb_opt_complementaly -> bb_opt_complementally
2) better support long options
3) new flag '!' for bb_opt_complementally: produce bb_show_usage() if BB_GETOPT_ERROR internally

1) sync with dash_0.5.2-7
2) but expand PS# as config option
3) correct kill error message again
4) remove show "line number" for interactive run (patch pending for dash)

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.