trunk broken since die_jmp is configured out

[Denis Vlasenko]

On Wednesday 16 May 2007 08:44, Bernhard Fischer wrote:
> On Wed, May 16, 2007 at 02:08:29AM +0200, Denis Vlasenko wrote:
> >Hi Bernhard,
> >
> >On Tuesday 15 May 2007 11:41, Bernhard Fischer wrote:
> >> $ egrep "(PREFER_APPLETS|HUSH)" build_i386/busybox/.config
> >> # CONFIG_FEATURE_PREFER_APPLETS is not set
> >> # CONFIG_FEATURE_SH_IS_HUSH is not set
> >> # CONFIG_HUSH is not set
> >> # CONFIG_HUSH_INTERACTIVE is not set
> >> # CONFIG_HUSH_JOB is not set
> >
> >Unable to reproduce:
> >
> >$ egrep "(PREFER_APPLETS|HUSH)" .config
> ># CONFIG_FEATURE_PREFER_APPLETS is not set
> ># CONFIG_FEATURE_SH_IS_HUSH is not set
> ># CONFIG_HUSH is not set
> ># CONFIG_HUSH_INTERACTIVE is not set
> ># CONFIG_HUSH_JOB is not set
> >
> >Please send your full .config and give gcc and ld versions.
> 
> gcc-4.2 off the branch as well as
> gcc version 4.1.2 20061115 (prerelease) (Debian 4.1.1-21)
> 
> ld 2.17.50.0.16 and 2.17
> 
> $ make clean allnoconfig busybox
> $ make clean
> $ make EXTRA_CFLAGS="-Os -Wfatal-errors" -f scripts/Makefile.IMA
> 
> See?

Yes.

Attached patch works around the problem by #ifdef'ing out
a bunch of functions which are never called in this .config
anyway.

The real problem is that linker tries to pull in unneeded stuff,
which in turn requires die_jmp.

Will test it with randomconfig now...
--
vda
-------------- next part --------------
A non-text attachment was scrubbed...
Name: z.diff
Type: text/x-diff
Size: 477 bytes
Desc: not available
Url : http://lists.busybox.net/pipermail/busybox/attachments/20070517/3bf5bd59/attachment-0002.bin