From mboxrd@z Thu Jan 1 00:00:00 1970 From: Luca Ceresoli Date: Wed, 18 Jun 2014 23:25:06 +0200 Subject: [Buildroot] [PATCH 3/5] legal-info: add a comment about what packages we save the tarballs of In-Reply-To: <406f594430b168c7553033b561da7a89c3cdaa70.1402758331.git.yann.morin.1998@free.fr> References: <406f594430b168c7553033b561da7a89c3cdaa70.1402758331.git.yann.morin.1998@free.fr> Message-ID: <53A203B2.7070105@lucaceresoli.net> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net Dear Yann, Yann E. MORIN wrote: > From: "Yann E. MORIN" > > As the legal-info infra only (rightfully) saves the tarballs of packages > that: > - we want to redistribute, > - and are not local, > - and are not overriden, > > add a comment stating so. > > This should clarify the code-block, which although trivial to read, > was not easy to interpret without thinking thouroughly about it. > > Signed-off-by: "Yann E. MORIN" > Cc: Luca Ceresoli > Cc: Thomas De Schampheleire > Cc: Thomas Petazzoni > Cc: Fabio Porcedda > --- > package/pkg-generic.mk | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/package/pkg-generic.mk b/package/pkg-generic.mk > index a978e63..d16908a 100644 > --- a/package/pkg-generic.mk > +++ b/package/pkg-generic.mk > @@ -602,6 +602,9 @@ ifneq ($$($(2)_LICENSE_FILES),) > $(1)-legal-info: $(1)-extract > endif > > +# We only save the sources of packages we want to redistribute, that are > +# non-local, and non-overriden. So only store, in the manifest, the tarball > +# name of those packages. > ifeq ($$($(2)_REDISTRIBUTE),YES) > ifneq ($$($(2)_SITE_METHOD),local) > ifneq ($$($(2)_SITE_METHOD),override) > Reviewed-by: Luca Ceresoli -- Luca