[Buildroot] [PATCH 10/51] core/legal-info: install source archives in their own sub-dir

Luca Ceresoli luca at lucaceresoli.net
Thu Dec 3 17:25:57 UTC 2015



Il 03/12/2015 18:16, Luca Ceresoli ha scritto:
> Yann,
>
> Yann E. MORIN wrote:
>> Currently, we put all source archives side-by-side in the same
>> directory.
>>
>> Since we're about to also save individual patches that were applied
>> on those sources, we don't want to make that directory a complete
>> mess of unassorted files.
>>
>> So, we install each source archive in its own sub-directory, where
>> we'll later store the patches too. Store that location in a variable,
>> so it can be re-used later on (to install patches in a future commit).
>>
>> Signed-off-by: "Yann E. MORIN" <yann.morin.1998 at free.fr>
>> Cc: Luca Ceresoli <luca at lucaceresoli.net>
>> ---
>>   package/pkg-generic.mk | 4 +++-
>>   1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/package/pkg-generic.mk b/package/pkg-generic.mk
>> index 98db3ba..bb3a6da 100644
>> --- a/package/pkg-generic.mk
>> +++ b/package/pkg-generic.mk
>> @@ -482,6 +482,8 @@ endif
>>
>>   $(2)_REDISTRIBUTE        ?= YES
>>
>> +$(2)_REDIST_SOURCES_DIR = $$(REDIST_SOURCES_DIR_$$(call
>> UPPERCASE,$(4)))/$$($(2)_RAWNAME)-$$($(2)_VERSION)
>> +
>>   # When a target package is a toolchain dependency set this variable to
>>   # 'NO' so the 'toolchain' dependency is not added to prevent a circular
>>   # dependency
>> @@ -812,7 +814,7 @@ endif
>>   # Copy the source tarball
>>       $$(Q)$$(call hardlink-copy,\
>>                $$(DL_DIR)/$$($(2)_ACTUAL_SOURCE_TARBALL),\
>> -             $$(REDIST_SOURCES_DIR_$$(call UPPERCASE,$(4))))
>> +             $$($(2)_REDIST_SOURCES_DIR))
>>   endif # redistribute
>>
>>   endif # other packages
>>
>
> Acked-by: Luca Ceresoli <luca at lucaceresoli.net>
> Tested-by: Luca Ceresoli <luca at lucaceresoli.net>
>
> Although this makes little sense without the further patches up to:
>
>    [PATCH 12/51] core/apply-patches: store full path of applied patches

Sorry, I actually meant:

   [PATCH 13/51] core/legal-info: also save patches

-- 
Luca


More information about the buildroot mailing list