[Fuego] [PATCH 2/2] iperf3: only do static compilation for libiperf

[Li, Xiaoming]

Hi Tim:


> -----Original Message-----
> From: Tim.Bird at sony.com [mailto:Tim.Bird at sony.com]
> Sent: Thursday, August 16, 2018 6:15 AM
> To: daniel.sangorrin at toshiba.co.jp; Li, Xiaoming/李 霄鸣;
> fuego at lists.linuxfoundation.org
> Subject: RE: [Fuego] [PATCH 2/2] iperf3: only do static compilation for libiperf
> 
> 
> 
> > -----Original Message-----
> > From: Daniel Sangorrin
> >
> > > -----Original Message-----
> > > From: Li Xiaoming
> > > Sent: Tuesday, July 3, 2018 4:41 AM
> > > To: fuego at lists.linuxfoundation.org
> > > Subject: [Fuego] [PATCH 2/2] iperf3: only do static compilation for
> > > libiperf
> > >
> > > Not necessary static libraries will be needed, if we do a completely
> > > static compilation. E.g. libc.a.
> > >
> > > Signed-off-by: Li Xiaoming <lixm.fnst at cn.fujitsu.com>
> > > ---
> > >  engine/tests/Benchmark.iperf3/fuego_test.sh | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/engine/tests/Benchmark.iperf3/fuego_test.sh
> > > b/engine/tests/Benchmark.iperf3/fuego_test.sh
> > > index 6876d90..c8c0113 100755
> > > --- a/engine/tests/Benchmark.iperf3/fuego_test.sh
> > > +++ b/engine/tests/Benchmark.iperf3/fuego_test.sh
> > > @@ -2,7 +2,7 @@
> > >  tarball=iperf-3.1.3-source.tar.gz
> > >
> > >  function test_build {
> > > -    LDFLAGS=-static ./configure --host=$HOST --prefix=$(pwd)/build
> > > +    ./configure --host=$HOST --prefix=$(pwd)/build
> > > + --enable-shared=no
> > >      make
> > >  }
> > >
> > > --
> > > 2.7.4
> >
> > Thanks Li.
> > I asked to the upstream developer[1] and he mentioned that the next
> > options were correct. Could you try them and resend?
> >
> > --enable-static --disable-shared
> 
> FWIW, I tried these options, and I still get dynamically-linked binaries.
> 
> root at timdesk:/fuego-rw/buildzone# ls *iperf3*/src/iperf3 | xargs file | grep
> -v default
> Benchmark.iperf3-debian-armhf/src/iperf3:                ELF 32-bit LSB shared
> object, ARM, EABI5 version 1 (SYSV), dynamically linked, interpreter /lib/ld-
> linux-armhf.so.3, for GNU/Linux 3.2.0,
> BuildID[sha1]=ac728b1687158229815b8b3458ff266db488d1b4, not stripped
> Benchmark.iperf3-poky-aarch64/src/iperf3:                ELF 64-bit LSB executable,
> ARM aarch64, version 1 (SYSV), dynamically linked, interpreter /lib/ld-linux-
> aarch64.so.1, for GNU/Linux 3.14.0,
> BuildID[sha1]=c33fde2910981fb08f77b41e9e98631c953d5cd4, not stripped
> Benchmark.iperf3-x86_64/src/iperf3:                      ELF 64-bit LSB executable,
> x86-64, version 1 (SYSV), dynamically linked, interpreter /lib64/ld-linux-x86-
> 64.so.2, for GNU/Linux 2.6.32,
> BuildID[sha1]=f1759fb987f125edbc640a36622bae3ad0c67b80, not stripped --
> Tim
> 
> Note that I get dynamically linked binaries using any of the configure lines
> that were in previous and current versions of fuego_test.sh.

Sorry, I am not a  expert of compile theory.

Do u mean that u build with  static   option and  also  got a  dynamically linked binarie ?


> 
> I applied Li's patch (back in July), but let me know if I should revert it, or add -
> -enable-static.
> 
>  -- Tim
> 


--Li