[Buildroot] [RFC 03/15] legal-info: save source tarballs for all packages

Luca Ceresoli luca at lucaceresoli.net
Wed Feb 1 21:38:07 UTC 2012


Arnout Vandecappelle wrote:
> On Sunday 29 January 2012 16:11:36 Luca Ceresoli wrote:
>> Signed-off-by: Luca Ceresoli<luca at lucaceresoli.net>
>   Darn, I've been adding Reviewed-by tags even though I said I wouldn't :-)
>
> [snip]
>> @@ -633,7 +635,8 @@ endif
>>
>>   clean:
>>   	rm -rf $(STAGING_DIR) $(TARGET_DIR) $(BINARIES_DIR) $(HOST_DIR) \
>> -		$(STAMP_DIR) $(BUILD_DIR) $(TOOLCHAIN_DIR) $(BASE_DIR)/staging
>> +		$(STAMP_DIR) $(BUILD_DIR) $(TOOLCHAIN_DIR) $(BASE_DIR)/staging \
>> +		$(LEGAL_INFO_DIR) $(REDIST_SOURCES_DIR)
>   $(REDIST_SOURCES_DIR) is a subdir of $(LEGAL_INFO_DIR), so doesn't need
> to be cleaned twice.

You're right.
Again a remnant of my initial drafts, when the two subdirs ere not one
inside the other...

>
>>
>>   distclean: clean
>>   ifeq ($(DL_DIR),$(TOPDIR)/dl)
>> diff --git a/package/Makefile.package.in b/package/Makefile.package.in
>> index e117223..0d7e081 100644
>> --- a/package/Makefile.package.in
>> +++ b/package/Makefile.package.in
>> @@ -653,10 +653,11 @@ $(1)-rsync:		$$($(2)_TARGET_RSYNC)
>>   $(1)-source:		$$($(2)_TARGET_RSYNC_SOURCE)
>>   endif
>>
>> -$(1)-legal-info:	$(1)-source $(LEGAL_INFO_DIR)
>> +$(1)-legal-info:	$(1)-source $(LEGAL_INFO_DIR) $(REDIST_SOURCES_DIR)
>>   			@echo -e "$(1)\t$$($(3)_VERSION)\t$$($(3)_LICENSE)"
>>   			@echo "$(1),$$($(3)_VERSION),$$($(3)_LICENSE)" \
>>   				>>$(LEGAL_MANIFEST_CSV)
>> +			@cp $(DL_DIR)/$$($(3)_SOURCE) $(REDIST_SOURCES_DIR)
>
>   Actually, for local and override packages (of which you said they are not
> supported at the moment), $(1)-source should create a tar file.

Yes, this is another point in my TODO list. I have a few implementation
options to choose from.

Luca


More information about the buildroot mailing list