[Buildroot] [PATCH v2 2/2] package/dropbear: fix license infos
Yann E. MORIN
yann.morin.1998 at free.fr
Sun Jun 28 09:13:00 UTC 2020
François, All,
On 2020-06-28 06:44 +0200, Francois Perrad spake thusly:
> this package allows to use optionally bundled libraries (which is exceptional in BR).
> so, license infos must be conditional.
>
> Signed-off-by: Francois Perrad <francois.perrad at gadz.org>
Since this is a fix, it should go in before the version bump. That way,
we can easily backport the fix to the stable branch (as Thomas already
pointed out).
If you do the fix after the version bump, ten the backport is les
trivial, because there will be merge-conflicts due to version mismatch.
There are a few other issues with it; see below. Care to address them
and respin please?
> ---
> package/dropbear/dropbear.hash | 2 ++
> package/dropbear/dropbear.mk | 5 +++++
> 2 files changed, 7 insertions(+)
>
> diff --git a/package/dropbear/dropbear.hash b/package/dropbear/dropbear.hash
> index 4b22e0ddd..af69cec53 100644
> --- a/package/dropbear/dropbear.hash
> +++ b/package/dropbear/dropbear.hash
> @@ -3,3 +3,5 @@ sha256 d927941b91f2da150b2033f1a88b6a47999bf0afb1493a73e9216cffdb5d7949 dropbe
>
> # License file, locally computed
> sha256 a99ce657d790b761c132ee7e0de18edb437ae6361e536d991c6a12f36e770445 LICENSE
> +sha256 8f196cb13afd271f5e267fd29543fc454596382ad580e7592709492843996ac8 libtomcrypt/LICENSE
> +sha256 2fa64b163659f41965c9815882a8296d3d03ff546b76153e11445f9bdecf955a libtommath/LICENSE
> diff --git a/package/dropbear/dropbear.mk b/package/dropbear/dropbear.mk
> index 3120943c0..d0158f022 100644
> --- a/package/dropbear/dropbear.mk
> +++ b/package/dropbear/dropbear.mk
> @@ -7,8 +7,13 @@
> DROPBEAR_VERSION = 2020.80
> DROPBEAR_SITE = https://matt.ucc.asn.au/dropbear/releases
> DROPBEAR_SOURCE = dropbear-$(DROPBEAR_VERSION).tar.bz2
> +ifeq ($(BR2_PACKAGE_DROPBEAR_SMALL),y)
> +DROPBEAR_LICENSE = MIT, BSD-2-Clause, Public domain, Unlicense, WTFPL
> +DROPBEAR_LICENSE_FILES = LICENSE, libtommath/LICENSE, libtomcrypt/LICENSE
This has not been tested: this list is not comma-separated, otherwise
legal-info fails:
$ make dropbear-legal-info
ERROR: No hash found for LICENSE,
cp: cannot stat '/home/ymorin/dev/buildroot/O/build/dropbear-2020.79/LICENSE,': No such file or directory
> +else
> DROPBEAR_LICENSE = MIT, BSD-2-Clause, Public domain
> DROPBEAR_LICENSE_FILES = LICENSE
> +endif
The way we uasually do that, is to unconditionally define the "base"
licensing information, and conditionally append to it. And since we
already have a condition for _SMAL, you should re-use it, like:
DROPBEAR_LICENSE = MIT, BSD-2-Clause, Public domain
DROPBEAR_LICENSE_FILES = LICENSE
[...]
ifeq ($(BR2_PACKAGE_DROPBEAR_SMALL),y)
DROPBEAR_LICENSE += , Unlicense, WTFPL
DROPBEAR_LICENSE_FILES += libtommath/LICENSE libtomcrypt/LICENSE
DROPBEAR_CONF_OPTS += --disable-zlib --enable-bundled-libtom
else
[...]
endif
Regards,
Yann E. MORIN.
> DROPBEAR_TARGET_BINS = dropbearkey dropbearconvert scp
> DROPBEAR_PROGRAMS = dropbear $(DROPBEAR_TARGET_BINS)
>
> --
> 2.25.1
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
More information about the buildroot
mailing list