All of lore.kernel.org
 help / color / mirror / Atom feed
From: Baruch Siach <baruch@tkos.co.il>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH] pkg-generic: fix <pkg>-legal-info target
Date: Thu, 3 Sep 2015 17:30:43 +0300	[thread overview]
Message-ID: <20150903143043.GJ18583@tarshish> (raw)
In-Reply-To: <55E85556.4030103@lucaceresoli.net>

Hi Luca,

On Thu, Sep 03, 2015 at 04:12:38PM +0200, Luca Ceresoli wrote:
> Baruch Siach wrote:
> >When making the <pkg>-legal-info target before the main legal-info target, the
> >following error shows:
> >
> >/bin/sh: /home/baruch/git/buildroot/output/legal-info/licenses.txt: No such file or directory
> >package/freescale-imx/imx-vpu/imx-vpu.mk:39: recipe for target 'imx-vpu-legal-info' failed
> >make: *** [imx-vpu-legal-info] Error 1
> >
> >Make <pkg>-legal-info depend on legal-info-prepare to ensure that
> >$(LEGAL_INFO_DIR) exists when generating licenses.txt.
> >
> >Cc: Luca Ceresoli <luca@lucaceresoli.net>
> >Signed-off-by: Baruch Siach <baruch@tkos.co.il>
> >---
> >  package/pkg-generic.mk | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> >diff --git a/package/pkg-generic.mk b/package/pkg-generic.mk
> >index 6a7d97efdf02..24b473b13ff2 100644
> >--- a/package/pkg-generic.mk
> >+++ b/package/pkg-generic.mk
> >@@ -715,7 +715,7 @@ endif
> >  endif
> >
> >  # legal-info: produce legally relevant info.
> >-$(1)-legal-info:
> >+$(1)-legal-info: legal-info-prepare
> 
> Very weird, I can reproduce the bug, but only for packages under
> package/freescale/, like imx-vpu. I tested a dozen other packages, and
> they all work fine. My test command is:
> 
>   git clean -xdf && make defconfig && \
>     make BR2_DL_DIR=~/src ${PKG}-legal-info
> 
> Any idea of why these packages are special?

Which target creates the legal-info/ directory in the successful (non 
freescale) case?

baruch

-- 
     http://baruch.siach.name/blog/                  ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -

  reply	other threads:[~2015-09-03 14:30 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-09-02 17:11 [Buildroot] [PATCH] pkg-generic: fix <pkg>-legal-info target Baruch Siach
2015-09-03 14:12 ` Luca Ceresoli
2015-09-03 14:30   ` Baruch Siach [this message]
2015-09-03 22:19     ` Luca Ceresoli
2015-09-03 23:22       ` Arnout Vandecappelle
2015-09-04  7:18         ` Luca Ceresoli

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20150903143043.GJ18583@tarshish \
    --to=baruch@tkos.co.il \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.