diff mbox

[20/20] Makefile.target: Targets now have to reside in the target/ folder

Message ID 1481285870-3396-21-git-send-email-thuth@redhat.com (mailing list archive)
State New, archived
Headers show

Commit Message

Thomas Huth Dec. 9, 2016, 12:17 p.m. UTC
Now that all target CPU folders have been moved to target/, we
do not need to support the old naming scheme of target-xxx
anymore.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 Makefile.target | 4 ----
 1 file changed, 4 deletions(-)

Comments

Paolo Bonzini Dec. 13, 2016, 4:20 p.m. UTC | #1
On 09/12/2016 13:27, Laurent Vivier wrote:
> Le 09/12/2016 à 13:17, Thomas Huth a écrit :
>> Now that all target CPU folders have been moved to target/, we
>> do not need to support the old naming scheme of target-xxx
>> anymore.
>>
>> Signed-off-by: Thomas Huth <thuth@redhat.com>
>> ---
>>  Makefile.target | 4 ----
>>  1 file changed, 4 deletions(-)
>>
>> diff --git a/Makefile.target b/Makefile.target
>> index 90b25ae..1da789c 100644
>> --- a/Makefile.target
>> +++ b/Makefile.target
>> @@ -7,11 +7,7 @@ include config-target.mak
>>  include config-devices.mak
>>  include $(SRC_PATH)/rules.mak
>>  
>> -ifneq ($(wildcard $(SRC_PATH)/target/$(TARGET_BASE_ARCH)),)
>>  TARGET_FOLDER=target/$(TARGET_BASE_ARCH)
>> -else
>> -TARGET_FOLDER=target-$(TARGET_BASE_ARCH)
>> -endif
>>  
>>  $(call set-vpath, $(SRC_PATH):$(BUILD_DIR))
>>  ifdef CONFIG_LINUX
>>
> Reviewed-by: Laurent Vivier <laurent@vivier.eu>
> 

Please undo the introduction of TARGET_FOLDER completely.

I would just squash all 20 patches together, and send a pull request.

Paolo
diff mbox

Patch

diff --git a/Makefile.target b/Makefile.target
index 90b25ae..1da789c 100644
--- a/Makefile.target
+++ b/Makefile.target
@@ -7,11 +7,7 @@  include config-target.mak
 include config-devices.mak
 include $(SRC_PATH)/rules.mak
 
-ifneq ($(wildcard $(SRC_PATH)/target/$(TARGET_BASE_ARCH)),)
 TARGET_FOLDER=target/$(TARGET_BASE_ARCH)
-else
-TARGET_FOLDER=target-$(TARGET_BASE_ARCH)
-endif
 
 $(call set-vpath, $(SRC_PATH):$(BUILD_DIR))
 ifdef CONFIG_LINUX