[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC v4 1/7] Makefile: Remove code to smooth transition
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [RFC v4 1/7] Makefile: Remove code to smooth transition to config.status |
Date: |
Fri, 24 May 2019 13:50:38 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux) |
Philippe Mathieu-Daudé <address@hidden> writes:
> Hi Markus,
>
> On 5/23/19 10:15 AM, Markus Armbruster wrote:
>> When commit bdf523e6923 made configure generate config.status, it
>> added a fallback to Makefile to smooth the transition, with a TODO
>> "code can be removed after QEMU 1.7." It's been more than five years.
>> Remove it.
>>
>> Signed-off-by: Markus Armbruster <address@hidden>
>> ---
>> Makefile | 9 +--------
>> 1 file changed, 1 insertion(+), 8 deletions(-)
>>
>> diff --git a/Makefile b/Makefile
>> index 155f066a20..8ec833a5fb 100644
>> --- a/Makefile
>> +++ b/Makefile
>> @@ -69,14 +69,7 @@ CONFIG_ALL=y
>>
>> config-host.mak: $(SRC_PATH)/configure $(SRC_PATH)/pc-bios
>> $(SRC_PATH)/VERSION
>> @echo $@ is out-of-date, running configure
>> - @# TODO: The next lines include code which supports a smooth
>> - @# transition from old configurations without config.status.
>> - @# This code can be removed after QEMU 1.7.
>> - @if test -x config.status; then \
>> - ./config.status; \
>> - else \
>> - sed -n "/.*Configured with/s/[^:]*: //p" $@ | sh; \
>> - fi
>> + ./config.status
>
> We could prepend a '@' since we already describe what this rule does
> ("config-host.mak is out-of-date, running configure").
You're right, my patch loses the @.
> Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
> Tested-by: Philippe Mathieu-Daudé <address@hidden>
Thanks!
[...]
- [Qemu-devel] [RFC v4 3/7] Makefile: Rename targets for make recursion, (continued)
- [Qemu-devel] [RFC v4 3/7] Makefile: Rename targets for make recursion, Markus Armbruster, 2019/05/23
- [Qemu-devel] [RFC v4 2/7] Makefile: Drop bogus cleaning of $(ALL_SUBDIRS)/qemu-options.def, Markus Armbruster, 2019/05/23
- [Qemu-devel] [RFC v4 1/7] Makefile: Remove code to smooth transition to config.status, Markus Armbruster, 2019/05/23
- [Qemu-devel] [RFC v4 7/7] tests: Make check-source cover generated headers, Markus Armbruster, 2019/05/23
- [Qemu-devel] [RFC v4 4/7] Makefile: Reuse all's recursion machinery for clean and install, Markus Armbruster, 2019/05/23
- [Qemu-devel] [RFC v4 5/7] tests: New make target check-source, Markus Armbruster, 2019/05/23
Re: [Qemu-devel] [RFC v4 5/7] tests: New make target check-source, Philippe Mathieu-Daudé, 2019/05/24