qemu-trivial
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-trivial] [Qemu-devel] [PATCH] pc-bios/s390-ccw: use STRIP vari


From: Laurent Vivier
Subject: Re: [Qemu-trivial] [Qemu-devel] [PATCH] pc-bios/s390-ccw: use STRIP variable in Makefile
Date: Wed, 31 May 2017 15:15:56 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.1.0

On 30/05/2017 18:56, Greg Kurz wrote:
> On Tue, 30 May 2017 18:34:46 +0200
> Christian Borntraeger <address@hidden> wrote:
> 
>> On 05/30/2017 05:04 PM, Greg Kurz wrote:
>>> The address@hidden target fails with:
>>>
>>> strip --strip-unneeded s390-ccw.elf -o s390-ccw.img
>>> strip: Unable to recognise the format of the input file `s390-ccw.elf'
>>>
>>> The configure script defines a STRIP makefile variable whose default
>>> value is ${cross_prefix}strip. Let's use it.
>>>
>>> Signed-off-by: Greg Kurz <address@hidden>
>>> ---
>>>  pc-bios/s390-ccw/Makefile |    2 +-
>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/pc-bios/s390-ccw/Makefile b/pc-bios/s390-ccw/Makefile
>>> index 79a46b67356f..7af57dad109c 100644
>>> --- a/pc-bios/s390-ccw/Makefile
>>> +++ b/pc-bios/s390-ccw/Makefile
>>> @@ -22,7 +22,7 @@ s390-ccw.elf: $(OBJECTS)
>>>     $(call quiet-command,$(CC) $(LDFLAGS) -o $@ 
>>> $(OBJECTS),"BUILD","$(TARGET_DIR)$@")
>>>
>>>  s390-ccw.img: s390-ccw.elf
>>> -   $(call quiet-command,strip --strip-unneeded $< -o 
>>> $@,"STRIP","$(TARGET_DIR)$@")
>>> +   $(call quiet-command,$(STRIP) --strip-unneeded $< -o 
>>> $@,"STRIP","$(TARGET_DIR)$@")
>>>
>>>  $(OBJECTS): Makefile
>>>
>>>   
>>
>> I was going to apply this, but it fails with --disable-strip like
>>
>>     /bin/sh: --strip-unneeded: command not found
>>     Makefile:25: recipe for target 's390-ccw.img' failed
>>     make[1]: *** [s390-ccw.img] Error 127
>>     Makefile:354: recipe for target 'romsubdir-s390-ccw' failed
>>     make: *** [romsubdir-s390-ccw] Error 2
>>
>> Not yet sure whats going on.
>>
> 
> configure doesn't generate STRIP when --disable-strip or --enable-debug... so
> I guess we shouldn't use it here, but rather generate a s390_cross_prefix like
> in roms/Makefile ?
> 

I experimented the problem when I played with docker, and fixed the
problem as you did (with STRIP).

But I think it's a good idea to compile s390 rom as it is done for ppc64
and ppc roms.

Laurent



reply via email to

[Prev in Thread] Current Thread [Next in Thread]