In Bug 5, Tito writes:

This is a first attempt to improve the comments of getopt_ulflags.c.
Maybe under some aspects the text could be refined, but so
far it is already usable and should help people who "avoided getopt_ulflags as the pest" to understand how it works.

This patch was created with the help of
Vodz, the author of the code, who explained me patiently
how getopt_ulflags works
and with the help of
Paul Fox, who corrected my broken english.

So thanks and merits should go to them also.

touchups as suggested by Vladimir N. Oleynik

err, use name attr of a tag, not href

import fakeidentd module started by Thomas Lundquist

add a new list of external tiny utilities

Patch from Bernhard Fischer to make a bunch of symbols static
which were otherwise cluttering the global namespace.

jfb2 writes in Bug 119:
The init applet will restart (re-exec) itsself when it
receives a SIGHUP. However, just before it enters its
main loop, it resets SIGHUP to either re-load the inittab
(or ignore it if no inittab is used). Thus preventing
the re-exec option from being triggerable.

This patch adds a signal handler for SIGQUIT for init that
always causes init to re-exec itsself (along with killing
anything else that might be still running).

Fix broken macro

Only define a local environ when not using the prototype from unistd.h

Rework to fix bugs.uclibc.org/view.php?id=107

Add bb_msg_read_error

In Bug 78, shortkey points out:

If you run `grep -h . file1 file2 file3 ...`, the output is not prefixed
with the filename.  If you run `grep -h . file1` however, the filename
will incorrectly prefix the output.

minor style touchups

In Bug 208, bernhardf writes:

On machines with only ANSI compliant compilers, not explitily delcaring
an empty parameter list 'void' causes failure.

In Bug 207, bernhardf writes:
proper escape strings in usage.h

In Bug 209, bernhardf writes:
struct named {
    named_field: init,
};
appear to be depraced by now; rather use C{99} format as in
struct named {
    .named_field = init,
};

bernhardf reports with Bug 210:

For certain non-gcc compilers, alloca_h is defined (included) but there,
no alloca() is declared.  Fallback to malloc if _ALLOCA_H is defined but
still, there is no alloca() in the included _ALLOCA_H.

Use a sane VEOL setting

fake out support for POSIX -H and -L options since busybox cp dereferences everything by default

add aliases == and [[ for = and [ to support more bash scripts

Removed the incorrect and confusing facility argument specified with
openlog() as the correct facility will be specified with syslog()
anyway.

Point to osuosl

Corrected the list of section names in add_ksymoops_symbols() so
that the bss and sbss sections can be correctly identified.

amd64 is rela, not rel

Shaun Jackman writes: Fix typo. Use __NR_umount2 instead of __NR_pivot_root.

Shaun Jackman writes: Newlib uses both __getopt_initialized and optind to track initialization of getopt()

add some more comments

newlib fixe from Shaun Jackman

newlib fixe from Shaun Jackman

Bandaid to make "gzip file1 file2 ..." set the decompression lengths correctly
in the 2nd and later headers.  But this and gunzip really need to be rewritten.

When filling the bit buffer, gzip decompression apparently never checked for end of file, causing it to hang on corrupted input.

Update buildroot pointers

remove whitespace

remove whitespace

Remove whitespace

Add 'nice' and replace 'renice' with a new implementation.

Takeharu KATO writes:

Hi,

I found that gcc in cvs (HEAD in 2005/02/11) reject the gzip source
in the busybox.

This is caused by changing gcc's error handling behavior(
The gcc check the function prototype more strictly).

I show the compilation log as follow:
-- compilation log

-- compilation log
To fix the problem, apply the patch which is attached with this
mail.

Please take a look the patch and apply the patch into svn repository.

- no need to check if JOBS is defined. Config.in ensures it.

- add ash read -t timeout support. initial code provided by Tim Yamin on Oct/21/2004 on the busybox mailing list. Edited his code a little to keep syntax highlighers happy and make it optional when CONFIG_ASH_TIMEOUT is defined

fix typo

update

Cut-n-paste strikes again

Remove mention of CVS and instead point to Subversion

Update the bug submission stuff to point to bugs.busybox.net

Add me as sort maintainer.

Sort rewrite to be SUSv3 compliant.  New config option, updated help, and
a couple of infrastructure bits.

Much bigger to-do list.

fix spelling

minor doc cleanup

Mention the new bug tracking system

cp: make -P a synonym for -d

Dear andersen:

Is the change on libbb/loop.c which you commited in 2005/1/3 effective
really?

The __GLIBC__ macro and __UCLIBC__ macro are defined in
feature.h in glibc source, so the change may not be effective.
If you want to check this with __GLIBC__, feature.h header is needed.

Some architectures(e.g. PPC series) need to include linux/posix_types.h
in stead of asm/posix_types.h, so the patch which is attached with
this mail include <linux/posix_types.h>.

perhaps a better fix

alpha/parisc support

fix typo

Minor in-passing crapectomy.

Workaround for uClibc-specific header problem described here:

http://www.busybox.net/lists/busybox/2004-December/013276.html

Rob

- CONFIG_FEATURE_READLINK_FOLLOW readlink -f patch from Colin Watson <cjwatson@debian.org> on busybox mailing list 08/11/04

merge from udhcp module

Hiroshi found another bug.  Currently sed's $ triggers at end of every file,
and with multiple files SuSv3 says it should only trigger at the end of the
LAST file.

The trivial fix I tried first broke if the last file is empty.  Fixing this
properly required restructuring things to create a file list (actually a
FILE * list), and then processing it all in one go.  (There's probably a
smaller way to do this, merging with append_list perhaps.  But let's get
the behavior correct first.)

Note that editing files in place (-i) needs the _old_ behavior, with $
triggering at the end of each file.

Here's a test of all the things this patch fixed.  gnu and busybox seds produce
the same results with this patch, and different without it.

echo -n -e "1one\n1two\n1three" > ../test1
echo -n > ../test2
echo -e "3one\n3two\n3three" > ../test3
sed -n "$ p" ../test1 ../test2 ../test3
sed -n "$ p" ../test1 ../test2
sed -i -n "$ p" ../test1 ../test2 ../test3

Don't document compiler warnings.  _FIX_ compiler warnings.

Correct the install-hardlinks target the same way as was already done
for the install target.

Alright, I guess I should be in this too...

Hiroshi Ito found some bugs.  The 'c' command (cut and paste) was hardwired
to not put a newline at the end (which was backwards, it should have been
hardwired _to_ put a newline at the end, whether or not the input line
ended with a newline).  Test case for that:

echo | sed -e '$ctest'

And then this would segfault:

echo | sed -e 'g'

Because pattern_space got freed but the dead pointer was only overwritten
in an if statement that didn't trigger if the hold space was empty.  Oops.

While debugging it, I found out that the hold space is persistent between
multiple input files, so I promoted it to a global and added it to the
memory cleanup.  The relevant test case (to compare with That Other Sed) is:

echo -n woo > woo
sed -e h -e g woo
echo "fish" | sed -e '/woo/h' -e "izap" -e 's/woo/thingy/' -e '/fish/g' woo -

And somebody gratuitously stuck in a c99 int8_t type for something that's just
a flag, so I grouped the darn ints.

1.00 is stable

mention scratchbox and openembedded

egor duda writes:

egor duda wrote:
>Ok, here's an updated patch.
>'make check' should work now, and one make creates Makefile in build
>directory, so one can run 'make' in build directory after that.

ahem. It looks like i'm slightly late with it but... Here's a little
addition to make 'make O=/some/where PREFIX=/some/where/else install'
work. Sorry for delay :(

egor

prepare for release

return failure when nslookup fails

Simon Poole writes:

Erik,

Attached is a patch for the udhcpc sample scripts, to correct the order in
which routers are applied if the DHCP server provides more than one (as per
section 3.5 of RFC2132).

Apologies for not being on the mailing list and thanks for your continued
efforts.

Simon.

Make certain clients of bb_make_directory default to honoring
the user's umask

Patch from David Daney:

It seems that date  -s MMDDHHMMYYYY.ss

will ignore the .ss part.  This patch tries to fix the problem.

David Daney.

oops

unmerged fix

Bump version

Fix the supported architectures section

Add an initial FAQ

Fix CONFIG_ASH_MATH_SUPPORT_64 so it actually works

Patch from Claus Klein to increase, and make more apparent
the hard coded limit on the number of mounts

Wade Berrier writes:

Hello,

Here's a patch for a first attempt at static leases for udhcpd.
Included in the tarball are 2 files (static_leases.c, static_leases.h)
and a patch against the latest cvs.

In the config file you can configure static leases with the following
format:

static_lease 00:60:08:11:CE:4E 192.168.0.54
static_lease 00:60:08:11:CE:3E 192.168.0.44

Comments/suggestions/improvements are welcome.

Wade

Patch from Denis Vlasenko to fix a problem where
wget http://1.2.3.4/abc/ loses last '/'

Hiroshi Ito writes:

Hello, all.

Busybox init does not handle removed inittab entry correctly.

# I'm sorry about my poor english, but you can find
# what I would like to say from patch, isn't it?

even if you apply this path,
when yoy try to change a command line option in inittab,
you have to do following steps.
1. remove old line from initrd
2. send HUP signal to init
3. kill old proces which is invoked from init.
4. append new line to inittab
5. send HUP signal to init, again

patch is against current CVS + last patch witch I send it last.

Hiroshi Ito writes:

"kill -HUP 1" reloads inittab, and when I append one line to inittab
and send HUP signal two times, It will starts 2 process.

patch against current CVS is attached.

Hiroshi Ito writes:

ash
   "unset OLDPWD; cd -"  causes segmentation fault.
    ( OLDPWD is not set when sh is invoked from getty. )

patch against current CVS is attached.

Hiroshi Ito writes:

    Hello

    I'm using busy box on mipsel machine.

    "grep -f file" will cause segmentation fault.

Vladimir N. Oleynik writes:

Hiroshi,

Thank for bug report, but your patch is full broken.
Worked patch attached.
(really changes is zero initialize, and indent correcting).

--w
vodz

Tito writes:

Hi to all,
This patch contains just some fixes for some misleading
comments in my_getpwuid.c and my_getug.c.
The code is untouched so this patch will not
cause troubles.

Please apply.

Thanks in advance and Ciao,
Tito

last_patch139.gz from Vladimir N. Oleynik:

>I also don't mean to disagree about leaving 30x status codes until after
>1.0.  In fact, although redirecting http://host/dir to http://host/dir/
>with a 301 is common practice (e.g. Apache, IIS), AFAIK it isn't
>actually required (or mentioned) by the HTTP specs.

Ok.
Attached patch have 302 and 408 implemented features.

--w
vodz

As noticed by egor duda, current_menu is declared as 'extern struct menu
*current_menu;' in scripts/config/lkc.h line 63, and this conflicts with
static definition in mconf.c.

egor duda writes:

Hi!

I've created a patch to busybox' build system to allow building it in
separate tree in a manner similar to kbuild from kernel version 2.6.

That is, one runs command like
'make O=/build/some/where/for/specific/target/and/options'
and everything is built in this exact directory, provided that it exists.

I understand that applyingc such invasive changes during 'release
candidates' stage of development is at best unwise. So, i'm currently
asking for comments about this patch, starting from whether such thing
is needed at all to whether it coded properly.

'make check' should work now, and one make creates Makefile in build
directory, so one can run 'make' in build directory after that.

One possible caveat is that if we build in some directory other than
source one, the source directory should be 'distclean'ed first.

egor

Patch from Michael Tokarev:

Scenario:

  touch x -- creates plain file name `x'
  mkdir x -- exits successefully

libbb/make_directory.c, bb_make_directory(), contains
the following code:

        if (mkdir(path, 0777) < 0) {
            /* If we failed for any other reason than the directory
             * already exists, output a diagnostic and return -1.*/
            if (errno != EEXIST) {
                fail_msg = "create";
                umask(mask);
                break;
            }
            /* Since the directory exists, don't attempt to change
             * permissions if it was the full target.  Note that
             * this is not an error conditon. */
            if (!c) {
                umask(mask);
                return 0;
            }
        }

The assumption that EEXIST error is due to that the *directory*
already exists is wrong: any file type with that name will cause
this error to be returned.  Proper way IMHO will be is to stat()
the path and check whenever this is really a directory.  Below
(attached) is a patch to fix this issue.

Make it more apparent that archive creation is not supported

Patch from William Barsse to fix a segfault when multiple files are specified.

Patch from Dmitry Zakharov to fix a bug triggered by freeswan's scripts.

Patch from Egor Duda
Attached patch prevents modprobe from trying to call 'insmod (null)'
whenever nonexistent module is either passed to modprobe via command
line or mentioned in modules.dep

this replaces cryptic error
sh: Syntax error: word unexpected (expecting ")")
with
modprobe: module some-module not found.

egor.

Fix a typo

Remove this error message at Vodz request, it was misleading.

Patch from David Daney to make the -i option work with -l.

A bit of extra explanation regarding STANDALONE

Add some notes on how to make telnetd actually work

Add a missing brace, patch by Hideki IWAMOTO