Re: [PATCH 4/4] strip: Add --reloc-debug-sections-only option.

[Mark Wielaard <mark at klomp dot org>]

On Fri, 2018-10-26 at 23:50 +0200, Mark Wielaard wrote:
> diff --git a/tests/ChangeLog b/tests/ChangeLog
> index 751a081..0870d4c 100644
> --- a/tests/ChangeLog
> +++ b/tests/ChangeLog
> @@ -1,3 +1,7 @@
> +2018-10-26  Mark Wielaard  <mark@klomp.org>
> +
> +	* run-strip-reloc.sh: Add a test for --reloc-debug-sections-only.
> +
>  2018-10-12  Mark Wielaard  <mark@klomp.org>
>  
>  	* run-readelf-zdebug.sh: Adjust flags output.
> diff --git a/tests/run-strip-reloc.sh b/tests/run-strip-reloc.sh
> index bbc9f58..6e54ab4 100755
> --- a/tests/run-strip-reloc.sh
> +++ b/tests/run-strip-reloc.sh
> @@ -32,6 +32,8 @@ runtest() {
>    outfile2=out.stripped2
>    debugfile2=out.debug2
>  
> +  echo "runtest $infile"
> +
>    rm -f $outfile1 $debugfile1 $outfile2 $debugfile2
>  
>    testrun ${abs_top_builddir}/src/strip -o $outfile1 -f $debugfile1 $infile ||
> @@ -67,6 +69,15 @@ runtest() {
>  
>    testrun_compare cat readelf.out1 < readelf.out2 ||
>    { echo "*** failure readelf -w compare $infile"; status=1; }
> +
> +  testrun ${abs_top_builddir}/src/strip --reloc-debug-sections-only \
> +	  $debugfile1 ||
> +  { echo "*** failure strip --reloc-debug-sections-only $debugfile1"; \
> +    status=1; }
> +
> +  cmp $debugfile1 $debugfile2 ||
> +  { echo "*** failure --reloc-debug-sections[-only] $debugfile1 $debugfile2"; \
> +    status=1; }
>  }
>  
>  # Most simple hello world kernel module for various architectures.

The buildbot didn't sent any failure emails (I am still looking into
why), but this new cmp test failed on debian-armhf, fedora-ppc64 and
fedora-ppc64le:
https://builder.wildebeest.org/buildbot/#/builders/15/builds/81
https://builder.wildebeest.org/buildbot/#/builders/12/builds/242
https://builder.wildebeest.org/buildbot/#/builders/11/builds/244

It succeeds on everything else. Still investigating.