diff mbox series

package/lpeg: add missing LPEG_LICENSE_FILES

Message ID 8ef2e2013a5e486034e602157d91b1de0d1ad374.camel@orolia.com
State Accepted
Headers show
Series package/lpeg: add missing LPEG_LICENSE_FILES | expand

Commit Message

Nicolas Carrier Jan. 13, 2023, 1:44 p.m. UTC 
The lpeg.html file declares the licensing terms at its bottom, so it
should work as a license file.

Comments

Yann E. MORIN Jan. 14, 2023, 9 p.m. UTC | #1 
Nicolas, All,

On 2023-01-13 13:44 +0000, Nicolas Carrier spake thusly:
> The lpeg.html file declares the licensing terms at its bottom, so it
> should work as a license file.

> From 12007e85d75daf941b8e82741e14ee1cb9c61893 Mon Sep 17 00:00:00 2001
> From: Nicolas Carrier <nicolas.carrier@orolia.com>
> Date: Fri, 13 Jan 2023 14:39:53 +0100
> Subject: [PATCH 1/1] package/lpeg: add missing LPEG_LICENSE_FILES
> 
> The lpeg.html file declares the licensing terms at its bottom, so it
> should work as a license file.
> 
> Signed-off-by: Nicolas Carrier <nicolas.carrier@orolia.com>
> ---
>  package/lpeg/lpeg.hash | 3 +++
>  package/lpeg/lpeg.mk   | 1 +
>  2 files changed, 4 insertions(+)
> 
> diff --git a/package/lpeg/lpeg.hash b/package/lpeg/lpeg.hash
> index 1ccaf242c4..f480ac6483 100644
> --- a/package/lpeg/lpeg.hash
> +++ b/package/lpeg/lpeg.hash
> @@ -1,2 +1,5 @@
>  # computed by luarocks/buildroot
>  sha256  e0d0d687897f06588558168eeb1902ac41a11edd1b58f1aa61b99d0ea0abbfbc  lpeg-1.0.2-1.src.rock
> +# computed locally
> +sha256  2b80a8cce793e008ba433e5e0184f20aca4a67ec741439fafd86a874a6a91c49  lpeg-1.0.2/lpeg.html
> +

    $ make check-package
    package/lpeg/lpeg.hash:5: empty line at end of file

> diff --git a/package/lpeg/lpeg.mk b/package/lpeg/lpeg.mk
> index cdb2e81a5f..82c733904b 100644
> --- a/package/lpeg/lpeg.mk
> +++ b/package/lpeg/lpeg.mk
> @@ -6,5 +6,6 @@
>  
>  LPEG_VERSION = 1.0.2-1
>  LPEG_LICENSE = MIT
> +LPEG_LICENSE_FILES = lpeg-1.0.2/lpeg.html

I've changed that to use $(LPEG_SUBDIR), which is autocomputed by the
luarosk-package infra to contain the proper package-version string.

Applied to master with that fixed, thanks.

Regards,
Yann E. MORIN.

>  
>  $(eval $(luarocks-package))
> -- 
> 2.30.2
> 

> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot
Peter Korsgaard Jan. 16, 2023, 2:45 p.m. UTC | #2 
>>>>> "Nicolas" == Nicolas Carrier <Nicolas.Carrier@orolia.com> writes:

 > The lpeg.html file declares the licensing terms at its bottom, so it
 > should work as a license file.

Committed to 2022.11.x and 2022.02.x, thanks.
diff mbox series

Patch

From 12007e85d75daf941b8e82741e14ee1cb9c61893 Mon Sep 17 00:00:00 2001
From: Nicolas Carrier <nicolas.carrier@orolia.com>
Date: Fri, 13 Jan 2023 14:39:53 +0100
Subject: [PATCH 1/1] package/lpeg: add missing LPEG_LICENSE_FILES

The lpeg.html file declares the licensing terms at its bottom, so it
should work as a license file.

Signed-off-by: Nicolas Carrier <nicolas.carrier@orolia.com>
---
 package/lpeg/lpeg.hash | 3 +++
 package/lpeg/lpeg.mk   | 1 +
 2 files changed, 4 insertions(+)

diff --git a/package/lpeg/lpeg.hash b/package/lpeg/lpeg.hash
index 1ccaf242c4..f480ac6483 100644
--- a/package/lpeg/lpeg.hash
+++ b/package/lpeg/lpeg.hash
@@ -1,2 +1,5 @@ 
 # computed by luarocks/buildroot
 sha256  e0d0d687897f06588558168eeb1902ac41a11edd1b58f1aa61b99d0ea0abbfbc  lpeg-1.0.2-1.src.rock
+# computed locally
+sha256  2b80a8cce793e008ba433e5e0184f20aca4a67ec741439fafd86a874a6a91c49  lpeg-1.0.2/lpeg.html
+
diff --git a/package/lpeg/lpeg.mk b/package/lpeg/lpeg.mk
index cdb2e81a5f..82c733904b 100644
--- a/package/lpeg/lpeg.mk
+++ b/package/lpeg/lpeg.mk
@@ -6,5 +6,6 @@ 
 
 LPEG_VERSION = 1.0.2-1
 LPEG_LICENSE = MIT
+LPEG_LICENSE_FILES = lpeg-1.0.2/lpeg.html
 
 $(eval $(luarocks-package))
-- 
2.30.2