[Buildroot] [PATCH] Remove readline for which

[Bernhard Fischer]

Hi Wade, *,

On Tue, Aug 05, 2008 at 01:44:50PM -0600, Wade Berrier wrote:
>Hi,
>
>I've patched 'which' to not require readline on the source level
>instead of selecting it on the buildroot dependency level.

I suggest you try to get that fixed upstream so they do allow for
./configure --disable-readline-and-yes-i-really-do-not-want-it

Short of that, why don't you use busybox in the first place?

I'm really curious how big your "fixed" which binary is?

Mine looks like this, perhaps your's is smaller?
$ size debianutils/which.o
   text	   data	    bss	    dec	    hex	filename
    211	      0	      0	    211	     d3	debianutils/which.o

Just to give you some background: buildroot is ment to serve as an
environment for uClibc. That's it's primary goal AFAIU. Second, it's
ment to serve as an easy way to get you a _small_ filesystem (thus using
busybox targeted at uClibc).

There is really no reason why .._HIDE_OTHER is visible to a casual user
since it's not ment to be actually used, but just to test uClibc, in my
POV. Please, just set BR2_PACKAGE_BUSYBOX_HIDE_OTHERS=y in your toplevel
.config.
>
>Patch attached.

As said. Re-read the patch. Send it to the maintainers of "debianutils"
and understand why they will most likely reject it in it's current form.
HTH,