* [Buildroot] make legal-info fails when override is active
@ 2012-10-11 8:19 Stephan Hoffmann
2012-10-14 14:41 ` [Buildroot] [PATCH] pkg-infra: rename <pkg>-rsync to <pkg>-extract Arnout Vandecappelle
0 siblings, 1 reply; 12+ messages in thread
From: Stephan Hoffmann @ 2012-10-11 8:19 UTC (permalink / raw)
To: buildroot
Hello all,
in my setup using the override srcdir feature for the kernel make
legal-info fails.
> make legal-info
> >>> Collecting legal info
> make: *** No rule to make target `linux-extract', needed by
> `linux-legal-info'. Stop.
Tested with actual git head (526badc94)
Kind regards
Stephan
--
reLinux - Stephan Hoffmann
Am Schmidtgrund 124 50765 K?ln
Tel. +49.221.95595-19 Fax: -64
www.reLinux.de sho at reLinux.de
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: defconfig
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20121011/573c0fce/attachment.ksh>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: local.mk
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20121011/573c0fce/attachment-0001.ksh>
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Buildroot] [PATCH] pkg-infra: rename <pkg>-rsync to <pkg>-extract
2012-10-11 8:19 [Buildroot] make legal-info fails when override is active Stephan Hoffmann
@ 2012-10-14 14:41 ` Arnout Vandecappelle
2012-10-14 18:01 ` Thomas Petazzoni
2012-11-12 8:27 ` [Buildroot] [PATCH] pkg-infra: rename <pkg>-rsync to <pkg>-extract Stephan Hoffmann
0 siblings, 2 replies; 12+ messages in thread
From: Arnout Vandecappelle @ 2012-10-14 14:41 UTC (permalink / raw)
To: buildroot
The legal-info target (and possibly others as well) depends on
<pkg>-extract to make sure the license file is available. However,
when <PKG>_OVERRIDE_SRCDIR is active, the <pkg>-extract target
doesn't exist.
To solve this, we just call the target that rsyncs the source
<pkg>-extract.
While we're at it, also add a <pkg>-patch target (which is equivalent
to <pkg>-extract). That avoids the same problem in the future if
something starts depending on <pkg>-patch.
Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
package/pkg-generic.mk | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/package/pkg-generic.mk b/package/pkg-generic.mk
index ffe7dfb..c2c1b06 100644
--- a/package/pkg-generic.mk
+++ b/package/pkg-generic.mk
@@ -378,9 +378,11 @@ else
$(1)-configure: $(1)-depends \
$$($(2)_TARGET_CONFIGURE)
-$(1)-depends: $(1)-rsync $$($(2)_DEPENDENCIES)
+$(1)-depends: $(1)-extract $$($(2)_DEPENDENCIES)
-$(1)-rsync: $$($(2)_TARGET_RSYNC)
+$(1)-patch: $(1)-extract
+
+$(1)-extract: $$($(2)_TARGET_RSYNC)
$(1)-source: $$($(2)_TARGET_RSYNC_SOURCE)
endif
--
tg: (0132e36..) t/rsync-add-extract-target (depends on: )
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [Buildroot] [PATCH] pkg-infra: rename <pkg>-rsync to <pkg>-extract
2012-10-14 14:41 ` [Buildroot] [PATCH] pkg-infra: rename <pkg>-rsync to <pkg>-extract Arnout Vandecappelle
@ 2012-10-14 18:01 ` Thomas Petazzoni
2012-10-14 21:00 ` Arnout Vandecappelle
2012-11-12 8:27 ` [Buildroot] [PATCH] pkg-infra: rename <pkg>-rsync to <pkg>-extract Stephan Hoffmann
1 sibling, 1 reply; 12+ messages in thread
From: Thomas Petazzoni @ 2012-10-14 18:01 UTC (permalink / raw)
To: buildroot
Arnout,
On Sun, 14 Oct 2012 16:41:44 +0200, Arnout Vandecappelle
(Essensium/Mind) wrote:
> The legal-info target (and possibly others as well) depends on
> <pkg>-extract to make sure the license file is available. However,
> when <PKG>_OVERRIDE_SRCDIR is active, the <pkg>-extract target
> doesn't exist.
>
> To solve this, we just call the target that rsyncs the source
> <pkg>-extract.
>
> While we're at it, also add a <pkg>-patch target (which is equivalent
> to <pkg>-extract). That avoids the same problem in the future if
> something starts depending on <pkg>-patch.
>
> Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
>
> ---
> package/pkg-generic.mk | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/package/pkg-generic.mk b/package/pkg-generic.mk
> index ffe7dfb..c2c1b06 100644
> --- a/package/pkg-generic.mk
> +++ b/package/pkg-generic.mk
> @@ -378,9 +378,11 @@ else
> $(1)-configure: $(1)-depends \
> $$($(2)_TARGET_CONFIGURE)
>
> -$(1)-depends: $(1)-rsync $$($(2)_DEPENDENCIES)
> +$(1)-depends: $(1)-extract $$($(2)_DEPENDENCIES)
>
> -$(1)-rsync: $$($(2)_TARGET_RSYNC)
> +$(1)-patch: $(1)-extract
> +
> +$(1)-extract: $$($(2)_TARGET_RSYNC)
>
> $(1)-source: $$($(2)_TARGET_RSYNC_SOURCE)
> endif
I think I would prefer to keep the 1:1 mapping between virtual target
name and real target name, and only have aliases when it makes sense.
I.e:
$(1)-rsync: $$($(2)_TARGET_RSYNC)
# Possibly some comment here to explain what's going on
$(1)-extract: $(1)-rsync
$(1)-patch: $(1)-rsync
Otherwise, I'm fine with fixing this problem using this approach, looks
good to me.
Thomas
--
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Buildroot] [PATCH] pkg-infra: rename <pkg>-rsync to <pkg>-extract
2012-10-14 18:01 ` Thomas Petazzoni
@ 2012-10-14 21:00 ` Arnout Vandecappelle
2012-10-14 21:06 ` Thomas Petazzoni
0 siblings, 1 reply; 12+ messages in thread
From: Arnout Vandecappelle @ 2012-10-14 21:00 UTC (permalink / raw)
To: buildroot
On 14/10/12 20:01, Thomas Petazzoni wrote:
> I think I would prefer to keep the 1:1 mapping between virtual target
> name and real target name, and only have aliases when it makes sense.
> I.e:
>
> $(1)-rsync: $$($(2)_TARGET_RSYNC)
>
> # Possibly some comment here to explain what's going on
> $(1)-extract: $(1)-rsync
> $(1)-patch: $(1)-rsync
>
> Otherwise, I'm fine with fixing this problem using this approach, looks
> good to me.
The reason that I choose to rename the -rsync target is that I'm hesitant
to add more targets - they slow down make.
However, this would only be for the OVERRIDE packages, so it just adds a few
targets, not 1000. So I'll take your suggestion and resend.
Regards,
Arnout
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286540
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Buildroot] [PATCH] pkg-infra: rename <pkg>-rsync to <pkg>-extract
2012-10-14 21:00 ` Arnout Vandecappelle
@ 2012-10-14 21:06 ` Thomas Petazzoni
2012-10-14 21:19 ` [Buildroot] [PATCH v2] pkg-infra: add <pkg>-extract and <pkg>-patch targets to OVERRIDE support Arnout Vandecappelle
0 siblings, 1 reply; 12+ messages in thread
From: Thomas Petazzoni @ 2012-10-14 21:06 UTC (permalink / raw)
To: buildroot
On Sun, 14 Oct 2012 23:00:21 +0200, Arnout Vandecappelle wrote:
> > Otherwise, I'm fine with fixing this problem using this approach, looks
> > good to me.
>
> The reason that I choose to rename the -rsync target is that I'm hesitant
> to add more targets - they slow down make.
>
> However, this would only be for the OVERRIDE packages, so it just adds a few
> targets, not 1000. So I'll take your suggestion and resend.
Ok, thanks!
Thomas
--
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Buildroot] [PATCH v2] pkg-infra: add <pkg>-extract and <pkg>-patch targets to OVERRIDE support
2012-10-14 21:06 ` Thomas Petazzoni
@ 2012-10-14 21:19 ` Arnout Vandecappelle
2012-10-14 21:42 ` Thomas Petazzoni
2012-10-15 12:38 ` Luca Ceresoli
0 siblings, 2 replies; 12+ messages in thread
From: Arnout Vandecappelle @ 2012-10-14 21:19 UTC (permalink / raw)
To: buildroot
The legal-info target (and possibly others as well) depends on
<pkg>-extract to make sure the license file is available. However,
when <PKG>_OVERRIDE_SRCDIR is active, the <pkg>-extract target
doesn't exist.
To solve this, we add <pkg>-extract which depends on <pkg>-rsync.
While we're at it,we do the same for <pkg>-patch. That avoids the
same problem in the future if something starts depending on
<pkg>-patch.
Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
v2: As ThomasP suggested, don't remove <pkg>-rsync.
package/pkg-generic.mk | 3 +++
1 file changed, 3 insertions(+)
diff --git a/package/pkg-generic.mk b/package/pkg-generic.mk
index ffe7dfb..27db5a5 100644
--- a/package/pkg-generic.mk
+++ b/package/pkg-generic.mk
@@ -380,6 +380,9 @@ $(1)-configure: $(1)-depends \
$(1)-depends: $(1)-rsync $$($(2)_DEPENDENCIES)
+$(1)-patch: $(1)-rsync
+$(1)-extract: $(1)-rsync
+
$(1)-rsync: $$($(2)_TARGET_RSYNC)
$(1)-source: $$($(2)_TARGET_RSYNC_SOURCE)
--
tg: (0132e36..) t/rsync-add-extract-target (depends on: master)
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [Buildroot] [PATCH v2] pkg-infra: add <pkg>-extract and <pkg>-patch targets to OVERRIDE support
2012-10-14 21:19 ` [Buildroot] [PATCH v2] pkg-infra: add <pkg>-extract and <pkg>-patch targets to OVERRIDE support Arnout Vandecappelle
@ 2012-10-14 21:42 ` Thomas Petazzoni
2012-10-15 12:38 ` Luca Ceresoli
1 sibling, 0 replies; 12+ messages in thread
From: Thomas Petazzoni @ 2012-10-14 21:42 UTC (permalink / raw)
To: buildroot
On Sun, 14 Oct 2012 23:19:18 +0200, Arnout Vandecappelle
(Essensium/Mind) wrote:
> The legal-info target (and possibly others as well) depends on
> <pkg>-extract to make sure the license file is available. However,
> when <PKG>_OVERRIDE_SRCDIR is active, the <pkg>-extract target
> doesn't exist.
>
> To solve this, we add <pkg>-extract which depends on <pkg>-rsync.
> While we're at it,we do the same for <pkg>-patch. That avoids the
> same problem in the future if something starts depending on
> <pkg>-patch.
>
> Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
Acked-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
--
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Buildroot] [PATCH v2] pkg-infra: add <pkg>-extract and <pkg>-patch targets to OVERRIDE support
2012-10-14 21:19 ` [Buildroot] [PATCH v2] pkg-infra: add <pkg>-extract and <pkg>-patch targets to OVERRIDE support Arnout Vandecappelle
2012-10-14 21:42 ` Thomas Petazzoni
@ 2012-10-15 12:38 ` Luca Ceresoli
1 sibling, 0 replies; 12+ messages in thread
From: Luca Ceresoli @ 2012-10-15 12:38 UTC (permalink / raw)
To: buildroot
Arnout Vandecappelle (Essensium/Mind) wrote:
> The legal-info target (and possibly others as well) depends on
> <pkg>-extract to make sure the license file is available. However,
> when <PKG>_OVERRIDE_SRCDIR is active, the <pkg>-extract target
> doesn't exist.
>
> To solve this, we add <pkg>-extract which depends on <pkg>-rsync.
> While we're at it,we do the same for <pkg>-patch. That avoids the
> same problem in the future if something starts depending on
> <pkg>-patch.
>
> Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
> ---
> v2: As ThomasP suggested, don't remove <pkg>-rsync.
>
> package/pkg-generic.mk | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/package/pkg-generic.mk b/package/pkg-generic.mk
> index ffe7dfb..27db5a5 100644
> --- a/package/pkg-generic.mk
> +++ b/package/pkg-generic.mk
> @@ -380,6 +380,9 @@ $(1)-configure: $(1)-depends \
>
> $(1)-depends: $(1)-rsync $$($(2)_DEPENDENCIES)
>
> +$(1)-patch: $(1)-rsync
> +$(1)-extract: $(1)-rsync
> +
> $(1)-rsync: $$($(2)_TARGET_RSYNC)
>
> $(1)-source: $$($(2)_TARGET_RSYNC_SOURCE)
>
Acked-by: Luca Ceresoli <luca@lucaceresoli.net>
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Buildroot] [PATCH] pkg-infra: rename <pkg>-rsync to <pkg>-extract
2012-10-14 14:41 ` [Buildroot] [PATCH] pkg-infra: rename <pkg>-rsync to <pkg>-extract Arnout Vandecappelle
2012-10-14 18:01 ` Thomas Petazzoni
@ 2012-11-12 8:27 ` Stephan Hoffmann
2012-11-13 23:48 ` Arnout Vandecappelle
1 sibling, 1 reply; 12+ messages in thread
From: Stephan Hoffmann @ 2012-11-12 8:27 UTC (permalink / raw)
To: buildroot
Am 14.10.2012 16:41, schrieb Arnout Vandecappelle (Essensium/Mind):
> The legal-info target (and possibly others as well) depends on
> <pkg>-extract to make sure the license file is available. However,
> when <PKG>_OVERRIDE_SRCDIR is active, the <pkg>-extract target
> doesn't exist.
>
> To solve this, we just call the target that rsyncs the source
> <pkg>-extract.
>
> While we're at it, also add a <pkg>-patch target (which is equivalent
> to <pkg>-extract). That avoids the same problem in the future if
> something starts depending on <pkg>-patch.
>
> Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
>
> ---
> package/pkg-generic.mk | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/package/pkg-generic.mk b/package/pkg-generic.mk
> index ffe7dfb..c2c1b06 100644
> --- a/package/pkg-generic.mk
> +++ b/package/pkg-generic.mk
> @@ -378,9 +378,11 @@ else
> $(1)-configure: $(1)-depends \
> $$($(2)_TARGET_CONFIGURE)
>
> -$(1)-depends: $(1)-rsync $$($(2)_DEPENDENCIES)
> +$(1)-depends: $(1)-extract $$($(2)_DEPENDENCIES)
>
> -$(1)-rsync: $$($(2)_TARGET_RSYNC)
> +$(1)-patch: $(1)-extract
> +
> +$(1)-extract: $$($(2)_TARGET_RSYNC)
>
> $(1)-source: $$($(2)_TARGET_RSYNC_SOURCE)
> endif
Hello,
this does not fix my issue, not even after doing a "make clean".
> LANG=C make legal-info
> mkdir -p /home/stephan/Dokumente/BeagleBone/buildroot/output/target
> if ! [ -d
> "/home/stephan/Dokumente/BeagleBone/buildroot/output/target/bin" ]; then \
> if [ -d
> "/home/stephan/Dokumente/BeagleBone/buildroot/fs/skeleton" ]; then \
> cp -fa
> /home/stephan/Dokumente/BeagleBone/buildroot/fs/skeleton/*
> /home/stephan/Dokumente/BeagleBone/buildroot/output/target/; \
> fi; \
> fi
> find /home/stephan/Dokumente/BeagleBone/buildroot/output/target -type
> d -name CVS -print0 -o -name .svn -print0 | xargs -0 rm -rf
> find /home/stephan/Dokumente/BeagleBone/buildroot/output/target -type
> f \( -name .empty -o -name '*~' \) -print0 | xargs -0 rm -rf
> touch /home/stephan/Dokumente/BeagleBone/buildroot/output/build/.root
> >>> Collecting legal info
<snip>
> >>> linux custom Syncing from source dir
> /home/stephan/Dokumente/BeagleBone/kernel/kernel
> rsync -au /home/stephan/Dokumente/BeagleBone/kernel/kernel/
> /home/stephan/Dokumente/BeagleBone/buildroot/output/build/linux-custom
> cp: cannot stat `/home/stephan/Dokumente/dl/linux-custom.tar.gz': No
> such file or directory
> make: *** [linux-legal-info] Error 1
Regards
Stephan
--
reLinux - Stephan Hoffmann
Am Schmidtgrund 124 50765 K?ln
Tel. +49.221.95595-19 Fax: -64
www.reLinux.de sho at reLinux.de
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: defconfig
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20121112/c4961bf1/attachment.ksh>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: local.mk
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20121112/c4961bf1/attachment-0001.ksh>
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Buildroot] [PATCH] pkg-infra: rename <pkg>-rsync to <pkg>-extract
2012-11-12 8:27 ` [Buildroot] [PATCH] pkg-infra: rename <pkg>-rsync to <pkg>-extract Stephan Hoffmann
@ 2012-11-13 23:48 ` Arnout Vandecappelle
2012-11-14 8:20 ` Stephan Hoffmann
0 siblings, 1 reply; 12+ messages in thread
From: Arnout Vandecappelle @ 2012-11-13 23:48 UTC (permalink / raw)
To: buildroot
On 12/11/12 09:27, Stephan Hoffmann wrote:
> Am 14.10.2012 16:41, schrieb Arnout Vandecappelle (Essensium/Mind):
[snip]
>> +$(1)-extract: $$($(2)_TARGET_RSYNC)
>>
>> $(1)-source: $$($(2)_TARGET_RSYNC_SOURCE)
>> endif
> Hello,
>
> this does not fix my issue, not even after doing a "make clean".
[snip]
>>>>> linux custom Syncing from source dir
>> /home/stephan/Dokumente/BeagleBone/kernel/kernel
>> rsync -au /home/stephan/Dokumente/BeagleBone/kernel/kernel/
>> /home/stephan/Dokumente/BeagleBone/buildroot/output/build/linux-custom
>> cp: cannot stat `/home/stephan/Dokumente/dl/linux-custom.tar.gz': No
>> such file or directory
>> make: *** [linux-legal-info] Error 1
One step at a time :-)
The .tar.gz doesn't exist for overridden packages. So what should we do -
exclude the source for overridden packages?
Regards,
Arnout
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286540
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Buildroot] [PATCH] pkg-infra: rename <pkg>-rsync to <pkg>-extract
2012-11-13 23:48 ` Arnout Vandecappelle
@ 2012-11-14 8:20 ` Stephan Hoffmann
2012-12-03 9:58 ` Stephan Hoffmann
0 siblings, 1 reply; 12+ messages in thread
From: Stephan Hoffmann @ 2012-11-14 8:20 UTC (permalink / raw)
To: buildroot
Am 14.11.2012 00:48, schrieb Arnout Vandecappelle:
> On 12/11/12 09:27, Stephan Hoffmann wrote:
>> Am 14.10.2012 16:41, schrieb Arnout Vandecappelle (Essensium/Mind):
> [snip]
>>> +$(1)-extract: $$($(2)_TARGET_RSYNC)
>>>
>>> $(1)-source: $$($(2)_TARGET_RSYNC_SOURCE)
>>> endif
>> Hello,
>>
>> this does not fix my issue, not even after doing a "make clean".
> [snip]
>>>>>> linux custom Syncing from source dir
>>> /home/stephan/Dokumente/BeagleBone/kernel/kernel
>>> rsync -au /home/stephan/Dokumente/BeagleBone/kernel/kernel/
>>> /home/stephan/Dokumente/BeagleBone/buildroot/output/build/linux-custom
>>> cp: cannot stat `/home/stephan/Dokumente/dl/linux-custom.tar.gz': No
>>> such file or directory
>>> make: *** [linux-legal-info] Error 1
>
> One step at a time :-)
>
> The .tar.gz doesn't exist for overridden packages. So what should we
> do -
> exclude the source for overridden packages?
Good point. At the first glance, I'd suggest to create the tar.gz using
the x_OVERRIDE_SRCDIR. That seems to be better than nothing.
After creating a linux-custom.tar.gz, "make legal-info" works for me,
but it does not seem to make sence to copy a .tar.gz file that has not
been extracted to build the packet.
Kind regards
Stephan
>
> Regards,
> Arnout
>
--
reLinux - Stephan Hoffmann
Am Schmidtgrund 124 50765 K?ln
Tel. +49.221.95595-19 Fax: -64
www.reLinux.de sho at reLinux.de
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Buildroot] [PATCH] pkg-infra: rename <pkg>-rsync to <pkg>-extract
2012-11-14 8:20 ` Stephan Hoffmann
@ 2012-12-03 9:58 ` Stephan Hoffmann
0 siblings, 0 replies; 12+ messages in thread
From: Stephan Hoffmann @ 2012-12-03 9:58 UTC (permalink / raw)
To: buildroot
Am 14.11.2012 09:20, schrieb Stephan Hoffmann:
> Am 14.11.2012 00:48, schrieb Arnout Vandecappelle:
>> On 12/11/12 09:27, Stephan Hoffmann wrote:
>>> Am 14.10.2012 16:41, schrieb Arnout Vandecappelle (Essensium/Mind):
>> [snip]
>>>> +$(1)-extract: $$($(2)_TARGET_RSYNC)
>>>>
>>>> $(1)-source: $$($(2)_TARGET_RSYNC_SOURCE)
>>>> endif
>>> Hello,
>>>
>>> this does not fix my issue, not even after doing a "make clean".
>> [snip]
>>>>>>> linux custom Syncing from source dir
>>>> /home/stephan/Dokumente/BeagleBone/kernel/kernel
>>>> rsync -au /home/stephan/Dokumente/BeagleBone/kernel/kernel/
>>>> /home/stephan/Dokumente/BeagleBone/buildroot/output/build/linux-custom
>>>> cp: cannot stat `/home/stephan/Dokumente/dl/linux-custom.tar.gz': No
>>>> such file or directory
>>>> make: *** [linux-legal-info] Error 1
>> One step at a time :-)
>>
>> The .tar.gz doesn't exist for overridden packages. So what should we
>> do -
>> exclude the source for overridden packages?
> Good point. At the first glance, I'd suggest to create the tar.gz using
> the x_OVERRIDE_SRCDIR. That seems to be better than nothing.
>
> After creating a linux-custom.tar.gz, "make legal-info" works for me,
> but it does not seem to make sence to copy a .tar.gz file that has not
> been extracted to build the packet.
Hello,
I just started looking at this again and found in package/pkg-generic.mk:
> # legal-info: produce legally relevant info.
> $(1)-legal-info:
> # Packages without a source are assumed to be part of Buildroot, skip
> them.
> ifneq ($(call qstrip,$$($(2)_SOURCE)),)
> ifeq ($$($(2)_SITE_METHOD),local)
> # Packages without a tarball: don't save and warn
> @$(call legal-warning-pkg-savednothing,$$($(2)_RAWNAME),local)
> else ifeq ($$($(2)_SITE_METHOD),override)
> @$(call legal-warning-pkg-savednothing,$$($(2)_RAWNAME),override)
> else
So there should just be a warning, but no make failing. Obviously, the
test for "override" does not work as expected. A patch changing this to
ifneq ($$($(2)_OVERRIDE_SRCDIR),)
follows, but I do not think that SITE_METHOD ever gets set to "override"
at all. So other parts of pkg-generic.mk might also need a closer look.
> Kind regards
>
> Stephan
>> Regards,
>> Arnout
>>
>
--
reLinux - Stephan Hoffmann
Am Schmidtgrund 124 50765 K?ln
Tel. +49.221.95595-19 Fax: -64
www.reLinux.de sho at reLinux.de
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2012-12-03 9:58 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-10-11 8:19 [Buildroot] make legal-info fails when override is active Stephan Hoffmann
2012-10-14 14:41 ` [Buildroot] [PATCH] pkg-infra: rename <pkg>-rsync to <pkg>-extract Arnout Vandecappelle
2012-10-14 18:01 ` Thomas Petazzoni
2012-10-14 21:00 ` Arnout Vandecappelle
2012-10-14 21:06 ` Thomas Petazzoni
2012-10-14 21:19 ` [Buildroot] [PATCH v2] pkg-infra: add <pkg>-extract and <pkg>-patch targets to OVERRIDE support Arnout Vandecappelle
2012-10-14 21:42 ` Thomas Petazzoni
2012-10-15 12:38 ` Luca Ceresoli
2012-11-12 8:27 ` [Buildroot] [PATCH] pkg-infra: rename <pkg>-rsync to <pkg>-extract Stephan Hoffmann
2012-11-13 23:48 ` Arnout Vandecappelle
2012-11-14 8:20 ` Stephan Hoffmann
2012-12-03 9:58 ` Stephan Hoffmann
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox