[Fuego] [PATCH 3/6] Benchmark.iperf : fixed iperf build error

Tim.Bird at sony.com Tim.Bird at sony.com
Thu Jul 19 00:34:31 UTC 2018



> -----Original Message-----
> From: Li Xiaoming
> 
> Details : add a patch of modify headers.h

I would like a longer description here (in the commit message),
or in the body of the new patch, explaining why this is needed.
It would be good to indicate what's failing because of the current
behavior (e.g. a compiler error or something) that this fixes.
 
> Signed-off-by: Li Xiaoming <lixm.fnst at cn.fujitsu.com>
> ---
>  engine/tests/Benchmark.iperf/fixfor_iperf_math.patch | 12 ++++++++++++
>  engine/tests/Benchmark.iperf/fuego_test.sh           |  1 +
>  2 files changed, 13 insertions(+)
>  create mode 100644 engine/tests/Benchmark.iperf/fixfor_iperf_math.patch
> 
> diff --git a/engine/tests/Benchmark.iperf/fixfor_iperf_math.patch
> b/engine/tests/Benchmark.iperf/fixfor_iperf_math.patch
> new file mode 100644
> index 0000000..c30e677
> --- /dev/null
> +++ b/engine/tests/Benchmark.iperf/fixfor_iperf_math.patch
> @@ -0,0 +1,12 @@
> +Signed-off-by: jijl <jijl.fnst at cn.fujitsu.com>
> +
> +--- old/iperf-2.0.5/include/headers.h	2018-06-11 14:37:06.000000000 +0800
> ++++ new/iperf-2.0.5/include/headers.h	2018-06-12
> 09:30:59.528593622 +0800
> +@@ -82,7 +82,6 @@
> + #include <errno.h>
> + #include <string.h>
> + #include <time.h>
> +-#include <math.h>
Why does the math.h include need to be removed?

> +
> + #ifdef WIN32
> +
> diff --git a/engine/tests/Benchmark.iperf/fuego_test.sh
> b/engine/tests/Benchmark.iperf/fuego_test.sh
> index 21d0284..dc89023 100755
> --- a/engine/tests/Benchmark.iperf/fuego_test.sh
> +++ b/engine/tests/Benchmark.iperf/fuego_test.sh
> @@ -1,6 +1,7 @@
>  tarball=iperf-2.0.5.tar.gz
> 
>  function test_build {
> +    patch -p2 -N -s < $TEST_HOME/fixfor_iperf_math.patch
>      # get updated config.sub and config.guess files, so configure
>      # doesn't reject new toolchains
>      cp /usr/share/misc/config.{sub,guess} .
> --
> 2.7.4

Thanks.  Please give some more details.
 -- Tim



More information about the Fuego mailing list