[Buildroot] [PATCH 4/5] legal-info: rename legal-warning-pkg-savednothing helper

Yann E. MORIN yann.morin.1998 at free.fr
Sat Jun 21 22:17:57 UTC 2014


Luca, All,

On 2014-06-18 17:32 +0200, Luca Ceresoli spake thusly:
> Yann E. MORIN wrote:
> >From: "Yann E. MORIN" <yann.morin.1998 at free.fr>
> >
> >This helper was called when none of the sources or license
> >files were saved.
> >
> >Now we handle license files separately from the sources,
> >this is no longer the case: they are only called when the
> >sources are not saved.
> >
> >Rename the handler and change the warning message accordingly.
> >
> >Signed-off-by: "Yann E. MORIN" <yann.morin.1998 at free.fr>
> >Cc: Luca Ceresoli <luca at lucaceresoli.net>
> >Cc: Thomas De Schampheleire <patrickdepinguin at gmail.com>
> >Cc: Thomas Petazzoni <thomas.petazzoni at free-electrons.com>
> >Cc: Fabio Porcedda <fabio.porcedda at gmail.com>
> >---
> >  package/pkg-generic.mk | 4 ++--
> >  package/pkg-utils.mk   | 4 ++--
> >  2 files changed, 4 insertions(+), 4 deletions(-)
> >
> >diff --git a/package/pkg-generic.mk b/package/pkg-generic.mk
> >index d16908a..8a32f9d 100644
> >--- a/package/pkg-generic.mk
> >+++ b/package/pkg-generic.mk
> >@@ -640,10 +640,10 @@ endif # license files
> >
> >  ifeq ($$($(2)_SITE_METHOD),local)
> >  # Packages without a tarball: don't save and warn
> >-	@$(call legal-warning-pkg-savednothing,$$($(2)_RAWNAME),local)
> >+	@$(call legal-license-nosource,$$($(2)_RAWNAME),local)
> >
> >  else ifneq ($$($(2)_OVERRIDE_SRCDIR),)
> >-	@$(call legal-warning-pkg-savednothing,$$($(2)_RAWNAME),override)
> >+	@$(call legal-license-nosource,$$($(2)_RAWNAME),override)
> >
> >  else
> >  # Other packages
> >diff --git a/package/pkg-utils.mk b/package/pkg-utils.mk
> >index 163ef8a..471f3ed 100644
> >--- a/package/pkg-utils.mk
> >+++ b/package/pkg-utils.mk
> >@@ -101,8 +101,8 @@ define legal-warning-pkg # pkg, text
> >  	echo "WARNING: $(1): $(2)" >>$(LEGAL_WARNINGS)
> >  endef
> >
> >-define legal-warning-pkg-savednothing # pkg, {local|override}
> >-	$(call legal-warning-pkg,$(1),sources and license files not saved ($(2) packages not handled))
> >+define legal-license-nosource # pkg, {local|override}
> 
> I'm OK with the idea, but I not with the name.
> Why "license"? This has to do with the source tarball.
> Also, I would keep "warning" in the name, as it's what it is.
> 
> So, I'd rename it to legal-warning-nosource.

Indeed. What was on my head when I did that?... :-/

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'


More information about the buildroot mailing list