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

just punt all the f_frsize crap since not all linux headers support it Bug 346

ifdef all of loop.c with CONFIG_FEATURE_MOUNT_LOOP.  won't
compile due to CONFIG_FEATURE_MOUNT_LOOP_MAX otherwise.  reported
by Stephane Billiart.

applying fix for:
     0000093: Patch for dpkg - can't handle scripts
      Attached patch is needed to fix dpkg's support for preinst,
       postinst etc script files.

remove duplicate check against chaddr.

revert 10881, and refix by changing "if (vallen)" to "if (val)".  this
is per the upstream fix for dash, in dash_0.5.2-6.diff.  thanks vodz, for
catching this.

applying fixes from:
    0000142: unzip enhancements

applying fix for:
    0000026: poor man's "scriptable" telnet

applying fix for:
    0000271: [PATCH] tftp -g fails if a TFTP_ACK is lost

applying fix for:
    0000265: tail -f should keep following files even if they
                    were truncated

applying fix for:
    0000263: nc cannot use -e when initiating a tcp connection
     to something else

applying fix for:
     0000262: tar -x doesn't believe it has reached the end of archive

applying fix for:
    0000261: Unsafe empty env var export in ash

applying fix for:
     0000260: udhcpc doesn't validate client hardware address

applying patch from:
    0000227: sort use wrong type for getopt return variable

applying fix for:
    0000203: 'ip route flush cache' not implemented

applying fix for:
    0000185: httpd infinite loop when piping to CGI script

applyinf fix for:
    0000155: variable expansion with braces in backticks in msh

applying fix from:
    0000152: ash: quoting rules for local variables different to globals

applying fix for:
     0000143: sysklogd remote logging adds a space between facility and tag

applying fix for:
    0000118: vi join command does not mark file as modified for certain lines.

much more concise fix for bug #45.  just align the packet...

If /tmp and /home were different partitions, then "mv /tmp/file /home/file"
would delete /home/file even if /tmp/file didn't exist.

This fixes that, although the logic of both mv and cp is a bit tangled and
should probably be untangled.

applying fix for:
     0000117: Remove linefeed after overwrite prompt using cp -i
  User input not on the same line as the prompt when about to
  overwrite a file.

applying fix from:
    0000108: busyboxy/networking/ftpgetput.c not conforming to
                    RFC 959.  ftpget and ftpput send <LF> as EOL.

applying fix from;
    0000092: looks like the initializer for .need_suid was missing.

applying fix from:
    0000088: inetd chargen stream does not generate the
    characters as recommended in RFC 864
    Chragen service is generating garbage characters.

applying fix from:
     0000068: mount limited to max 8 loop devices (patch provided)

(made minor wording change for config help message)

applying fix from:
 0000067: cp -p produces misleading error message

applying:
0000054: Tab completing filenames in ash causes SEGV
Simple tab completion operations cause busybox (ash) to
access illegal addresses.

applying fix for:
 0000045: traceroute causes an alignment trap due to unaligned buffer on arm

Fixup makedevs to handle regular files, and also fix
it to properly update file permissions as specified.

fix for "0000027: patch: nc will spin if stdin closed"

allow both ^H and DEL to backspace in insert mode (bug #23)

Fix vi so that error messages, insert mode messages, etc are
all actually displayed in the status line as expected

a bit more polish

a bit of polish on makedevs

the makedevs config option was pretty much totally broken

Fixup device table based makedevs so it actually works

move var decls around a little to help gcc make smaller code

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

        * init/init.c: Do not include sys/mount.h.
        (message): Use O_NONBLOCK instead of O_NDELAY.
        (console_init): Ditto.

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

        * init/Config.in (CONFIG_FEATURE_INIT_SWAPON): New option.
        * init/init.c (check_memory): Disable the swapon feature unless
        CONFIG_FEATURE_INIT_SWAPON is defined.

rename log var to log_console so we dont override internal gcc/glibc log func

2005-06-30  Shaun Jackman  <sjackman@gmail.com>

        * loginutils/getty.c: (open_tty): Use dup2 instead of close/dup.

2005-06-30  Shaun Jackman  <sjackman@gmail.com>

        * loginutils/getty.c: Include utmp.h only if
        CONFIG_FEATURE_U_W_TMP is defined.
        (getty_main): Use ISSUE only if it is defined.

Patch by jonlar in Bug 312 to split the U_W_TMP feature into sep UTMP and WTMP options

patch by Shaun Jackman to combine dup/close funcs into dup2

dont use f_frsize unless linux-2.6.0 or better

rip out all the non-linux code and ugly workarounds

tweak signed/unsigned char usage to avoid mismatches

- remove extra/unneeded function call. testing svn

Enabling runtime SUID/SGID configuration via /etc/busybox.conf
is not a very good default.  Better to default to having it off
and let people get the default behavior.  If they want to enable
/etc/busybox.conf they should explicitly ask for it.

applets specified as _BB_SUID_ALWAYS in applets.h should also select
CONFIG_FEATURE_SUID to ensure proper behavior when installed.

characters encoded as html should have a trailing semicolon
to be interpreted properly

Rodney Radford submitted ipcs and ipcrm (system V IPC stuff).  They could use
some more work to shrink them down.

Tito says: unify verbose/quiet flags

Tito says: strip unused program_name
Vladimir N. Oleynik says: uname() can be replaced with get_kernel_revision()

use xmalloc instead of malloc

use malloc instead of xmalloc

use xmalloc() instead of malloc()

use xmalloc() and bb_perror_msg_and_die()

Note that memory allocaiton needs to be cleaned up too.

remove com_err.h includes

DOS only crap

import initial fat mke2fs

import tune2fs support

fix signed/unsigned char pointers

oops, we only want to affect local CFLAGS

initial fat tune2fs/findfs/e2label source

replace simple is_null func with a memcmp define

force including of e2fsbb.h and move the HAVE_* defines to it

move config.h requirement to the actual .depend target rather than the depend alias

make sure clean removes objects in subdirs

need strings.h for ffs()

only define some variables if legacy EXT2FS_ENABLE_SWAPFS is enabled

replace simple functions with defines

whitespace updates

replace functions with defines

whitespace updates

whitespace updates

recode functions to shrink size

setup the HAVE_* defines

About time to just apply this and kill off the patches

Tito posted a devfsd error message fix.  It's highly deprecated and will
presumably be removed eventually (use udev), but as long as it's in there.

Tito says:

The sense of this patch is to call:
read_config_file_err:
#ifdef CONFIG_DEVFSD_VERBOSE
    msg_logger(((optional ==  0 ) && (errno == ENOENT))? DIE : NO_DIE, LOG_ERR, "read config file: %s: %m\n", path);
#else
    if(optional ==  0  && errno == ENOENT)
        exit(EXIT_FAILURE);
#endif

just after the failure of the  call that set errno ( stat and fopen)
to avoid false error messages.

Clean up strings.c to use busybox's option processing.  Bug 006, apparently.

Patch from Dmitry Zakharov:

Charlie Brady wrote:
> Here's another awk parsing problem - unary post increment - pre is fine:
>
>bash-2.05a$ echo 2,3 | gawk -F , '{ $2++ }'
>bash-2.05a$ echo 2,3 | /tmp/busybox/busybox awk -F , '{ $2++ }'
>awk: cmd. line:1: Unexpected token
>
Here's a fix for this. There is another problem with constructions like
"print (A+B) ++C", I don't
know whether somebody uses such constructions (fixing both these
problems would require very
serious change in awk code).

Thus spake Brenda J. Butler:

We were seeing some timeouts when getting files with the busybox tftp
client.

With tcpdump, we saw that the tftp client was receiving blocks and
ack'ing them, but the server was failing to receive the occasional
ack.

When that happened, the server would send the last block over again,
but the tftp client was expecting the next block.

This patch allows the client to recover from this situation
(it sends an ack for the repeat block but does not write it
to the local file).

I hope it meets your approval, please don't hesitate to send
me comments for improvement.

The patch is against "head" in svn, I tested it on an older version
of busybox in our environment.  It applied cleanly to the older
version.

Credit for this goes to my co-worker John McCarthy for finding
it and me for fixing it (assuming it works for everyone else too).

cheerio,
bjb

Shaun Jackman submitted a patch converting an allocation to use
CONFIG_RESERVE_BUFFER.  (Rob Landley removed an #ifdef, per discussion on
the list.)

Patch from Shaun Jackman to save a few bytes.

Tobias Krawutschke found a bug where the DHCP client would accept packets
with the wrong ARP address, meaning we could easily get somebody else's IP.
That is a bad thing, and this is the minimal two-line fix.