Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Fiona Klute via buildroot <buildroot@buildroot.org>
To: "Alexis Lothoré" <alexis.lothore@bootlin.com>, buildroot@buildroot.org
Cc: Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
	Nicolas Carrier <nicolas.carrier@nav-timing.safrangroup.com>
Subject: Re: [Buildroot] [PATCH] package/localedef: add license data
Date: Mon, 20 Jul 2026 17:52:53 +0200	[thread overview]
Message-ID: <de53fcb2-c43f-49ed-86ed-2c9745d4b6f0@gmx.de> (raw)
In-Reply-To: <20260715-legal-info-localdef-v1-1-5c44106ea59d@bootlin.com>

Hi Alexis,

thanks for the patch, I bumped into this recently, too!

Am 15.07.26 um 14:30 schrieb Alexis Lothoré via buildroot:
> When running `make legal-info` for a buil based on glibc and configured
> to generate some locale data (eg: BR2_GENERATE_LOCALE="C.UTF-8"), the
> following warning is printed:
> 
>    WARNING: localedef-2.43-45-gdae425b554207f7c4599c7fac707ad4c08545674: \
>    cannot save license (HOST_LOCALEDEF_LICENSE_FILES not defined)
> 
> Define the missing LOCALEDEF_LICENSE and LOCALEDEF_LICENSE_FILES to get
> rid of the warning. The header in locale/programs/localedef.c states
> that the program is released under GPL, so use GPLv2, as also st stated
> by glibc.mk in the glibc package in buildroot.

You're obviously correct about locale/programs/localedef.c, but it 
includes/links additional parts of glibc, and it's not obvious to me 
which. Do you have a way to verify? And of course the source archive 
includes everything anyway.

glibc.mk currently lists:

GLIBC_LICENSE = GPL-2.0+ (programs), LGPL-2.1+, BSD-3-Clause, MIT (library)
GLIBC_LICENSE_FILES = COPYINGv2 COPYING.LESSERv2 LICENSES

Side note: /usr/share/doc/libc-bin/copyright from the Debian libc-bin 
package (which includes localedef) mentions that sysdeps/htl/raise.c is 
licensed under LGPL-3.0+, which is not currently in our list. As far as 
I understand it's only *used* on Hurd, but it's still present in the 
source archive.

Best regards,
Fiona

> Signed-off-by: Alexis Lothoré <alexis.lothore@bootlin.com>
> ---
> Up to my knowledge this package never had the needed license data, so
> all currently maintained versions are affected by the issue.
> ---
>   package/localedef/localedef.mk | 2 ++
>   1 file changed, 2 insertions(+)
> 
> diff --git a/package/localedef/localedef.mk b/package/localedef/localedef.mk
> index 732ed6d4ad1d..aa7dba2b5170 100644
> --- a/package/localedef/localedef.mk
> +++ b/package/localedef/localedef.mk
> @@ -11,6 +11,8 @@ LOCALEDEF_VERSION = 2.43-45-gdae425b554207f7c4599c7fac707ad4c08545674
>   LOCALEDEF_SOURCE = glibc-$(LOCALEDEF_VERSION)$(BR_FMT_VERSION_git).tar.gz
>   LOCALEDEF_SITE = https://sourceware.org/git/glibc.git
>   LOCALEDEF_SITE_METHOD = git
> +LOCALEDEF_LICENSE = GPL-2.0+
> +LOCALEDEF_LICENSE_FILES = COPYINGv2
>   HOST_LOCALEDEF_DL_SUBDIR = glibc
>   
>   HOST_LOCALEDEF_DEPENDENCIES = \
> 
> ---
> base-commit: 643f480f4379afe5677ec399c802b8d33734b601
> change-id: 20260715-legal-info-localdef-cc9caf1a3f3a
> 
> Best regards,
> --
> Alexis Lothoré <alexis.lothore@bootlin.com>
> 
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

  reply	other threads:[~2026-07-20 15:53 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-07-15 12:30 [Buildroot] [PATCH] package/localedef: add license data Alexis Lothoré via buildroot
2026-07-20 15:52 ` Fiona Klute via buildroot [this message]
2026-08-03 14:06   ` Alexis Lothoré via buildroot

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=de53fcb2-c43f-49ed-86ed-2c9745d4b6f0@gmx.de \
    --to=buildroot@buildroot.org \
    --cc=alexis.lothore@bootlin.com \
    --cc=fiona.klute@gmx.de \
    --cc=nicolas.carrier@nav-timing.safrangroup.com \
    --cc=thomas.petazzoni@bootlin.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox