Re: [kvm-unit-tests PATCH 2/2] powerpc/ppc64: Makefile cleanup

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

 




On 10/05/2016 08:35, Andrew Jones wrote:
> On Tue, May 10, 2016 at 08:19:36AM +0200, Thomas Huth wrote:
>> On 09.05.2016 16:01, Andrew Jones wrote:
>>> This does the same cleanup as 'x86: Makefile refine'
>>>
>>> Signed-off-by: Andrew Jones <drjones@xxxxxxxxxx>
>>> ---
>>>  powerpc/Makefile.common | 13 ++++---------
>>>  1 file changed, 4 insertions(+), 9 deletions(-)
>>>
>>> diff --git a/powerpc/Makefile.common b/powerpc/Makefile.common
>>> index 4449aec616853..60d545c98e38c 100644
>>> --- a/powerpc/Makefile.common
>>> +++ b/powerpc/Makefile.common
>>> @@ -22,6 +22,9 @@ CFLAGS += -I lib -I lib/libfdt
>>>  CFLAGS += -Wa,-mregnames
>>>  CFLAGS += -fpie
>>>  
>>> +# We want to keep intermediate files: %.elf and %.o
>>> +.PRECIOUS: %.elf %.o
>>
>> If you just want to keep intermediate files, wouldn't it be better to
>> use .SECONDARY instead of .PRECIOUS? .PRECIOUS seems to contain some
>> more magic that comes into action when make is killed by a signal... not
>> sure if we want to have that here?
> 
> I should extend the commit message to explain that. Wei Yang found
> (and I confirmed) that .SECONDARY doesn't seem to work[*] - we tried it,
> but the intermediate files still got removed. We both agree it would be
> the more appropriate choice though. Did you have better luck with it?

I don't understand why you want to keep intermediate files. If they have
to be kept, they should be primary targets.

It seems for, at least ppc64, we don't need the .PRECIOUS.

Do you have an example?

Laurent
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux