Re: multi-platform libunwind and frysk-import/configure.ac

[Mark Wielaard <mark at klomp dot org>]

On Tue, 2007-03-27 at 12:51 +0200, Mark Wielaard wrote:
> On Thu, 2007-03-22 at 20:08 +0100, Stepan Kasal wrote:
> > On Thu, Mar 22, 2007 at 02:42:28PM -0400, Nurdin Premji wrote:
> > > >>$confdir/libunwind/configure --target=i386-unknown-linux-gnu
> > 
> > > Well the reason why this has to be done is that we need to compile 
> > > libunwind for each architecture we support in order to get 
> > > multi-platform unwinding working.
> > 
> > At first glance, I'd guess that implementing this with one configure
> > only might be easier.  I'll give it a try and report here.
> 
> Did you have any luck with this? The way it is currently done just
> discards any configure flags normally used, which breaks unless you only
> use all the defaults (no options to autogen.sh). I am a bit stuck how to
> handle this and need to get frysk installed again to try to replicate a
> bug. See also: http://sourceware.org/bugzilla/show_bug.cgi?id=4279

Saw your suggestion earlier in the thread:

>> $confdir/libunwind/configure --target=i386-unknown-linux-gnu
>>     
> But this is just a rough cut, because this does not pass the options
> given to the outer configure.
>
> Well, you can add "--build=${build} --host=${host}" to pass the arch
> names (in case they were given on the cmd-line).
> We could try "--prefix=$prefix" to pass another bit of info.

And I added that workaround for now:

2007-03-27  Mark Wielaard  <mark@klomp.org>

        * configure.ac: Pass on host, build and prefix to libunwind-i386
        configure (and add FIXME).

Thanks,

Mark

diff -u -r1.66 configure.ac
--- configure.ac        22 Mar 2007 20:08:55 -0000      1.66
+++ configure.ac        27 Mar 2007 11:04:09 -0000
@@ -65,8 +65,11 @@
   cd libunwind-i386
   # Make sure the inplace build does not break with "already configured"
   # message; the file is going to regenerated, anyway.
+  # FIXME - This is wrong since it discards any configure flags given
+  # except those explicitly passed on...
   rm -f $confdir/libunwind/config.status
-  $confdir/libunwind/configure --target=i386-unknown-linux-gnu
+  $confdir/libunwind/configure --target=i386-unknown-linux-gnu \
+       --build=${build} --host=${host} --prefix=${prefix}
 )
 
 AC_CONFIG_SUBDIRS([elfutils libunwind])