[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCHv2 RFC] makefile: detect corrupted elf files
From: |
Stefan Weil |
Subject: |
Re: [Qemu-devel] [PATCHv2 RFC] makefile: detect corrupted elf files |
Date: |
Wed, 22 May 2013 08:32:52 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux i686; rv:17.0) Gecko/20130510 Thunderbird/17.0.6 |
Am 22.05.2013 00:07, schrieb Michael S. Tsirkin:
> Once in a while make gets killed and doesn't
> clean up partial object files after it.
> Result is nasty errors from link.
> This hack checks object is well formed before linking,
> and rebuilds it if not.
>
> Signed-off-by: Michael S. Tsirkin <address@hidden>
> ---
>
> v1 actually prints some noise on stderr, this shuts it up.
>
> Makefile.target | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/Makefile.target b/Makefile.target
> index ce4391f..c506d4c 100644
> --- a/Makefile.target
> +++ b/Makefile.target
> @@ -191,3 +191,10 @@ endif
>
> GENERATED_HEADERS += config-target.h
> Makefile: $(GENERATED_HEADERS)
> +
> +.SECONDEXPANSION:
> +
> +.PHONY: CORRUPTBINARY
> +
> +$(all-obj-y): % : $$(if $$(shell size % 2>/dev/null), , CORRUPTBINARY)
> +
This modification adds build complexity and execution time for
each build, just to fix a potential problem which is very rare.
I personally prefer the current solution (remove bad object files
manually).
I did not test the patch, but won't cross builds be caught in an
endless loop when the native size command does not know the
cross object format?
Regards,
Stefan