Re: [PR PATCH] [Updated] Trim down lua*-devel dependencies

[ericonr <ericonr@users.noreply.github.com>]

[-- Attachment #1: Type: text/plain, Size: 1606 bytes --]

There is an updated pull request by ericonr against master on the void-packages repository

https://github.com/ericonr/void-packages luadeps
https://github.com/void-linux/void-packages/pull/30982

Trim down lua*-devel dependencies
<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [ ] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->

@q66

A patch file from https://github.com/void-linux/void-packages/pull/30982.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-luadeps-30982.patch --]
[-- Type: text/x-diff, Size: 4678 bytes --]

From 2751e237df7df9ec17ecd5c5b0ae15459727c5d0 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=89rico=20Nogueira?= <erico.erc@gmail.com>
Date: Tue, 18 May 2021 23:46:49 -0300
Subject: [PATCH 1/4] lua53: -devel only depends on self.

Only the lua binary depends on libreadline and ncurses, not the library,
so lua53-devel doesn't need to depend on their devel packages.

The package has listed these dependencies explicitly since at least
f0105c662bd1be3c1e84271fb6a63da7e6bf3752 (2013), so it being there until
today seems to be more of a historical artifact then for an actual
reason.
---
 srcpkgs/lua53/template | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/srcpkgs/lua53/template b/srcpkgs/lua53/template
index 098d700bac82..0761abc9bc17 100644
--- a/srcpkgs/lua53/template
+++ b/srcpkgs/lua53/template
@@ -1,7 +1,7 @@
 # Template file for 'lua53'
 pkgname=lua53
 version=5.3.5
-revision=6
+revision=7
 wrksrc=lua-${version}
 makedepends="ncurses-devel readline-devel"
 short_desc="Powerful, fast, lightweight, embeddable scripting language (5.3.x)"
@@ -58,7 +58,7 @@ do_install() {
 }
 
 lua53-devel_package() {
-	depends="${makedepends} lua53>=${version}_${revision}"
+	depends="lua53>=${version}_${revision}"
 	short_desc+=" - development files"
 	alternatives="
 	 lua-devel:lua:/usr/share/doc/lua5.3

From 3251af71bc766d47ee538ca746ea26ec74d74191 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=89rico=20Nogueira?= <erico.erc@gmail.com>
Date: Tue, 18 May 2021 23:49:10 -0300
Subject: [PATCH 2/4] lua51: -devel only depends on self.

Only the lua binary depends on libreadline and ncurses, not the library,
so lua51-devel doesn't need to depend on their devel packages.
---
 srcpkgs/lua51/template | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/srcpkgs/lua51/template b/srcpkgs/lua51/template
index 12da0983a617..7aad299f0757 100644
--- a/srcpkgs/lua51/template
+++ b/srcpkgs/lua51/template
@@ -1,7 +1,7 @@
 # Template file for 'lua51'
 pkgname=lua51
 version=5.1.5
-revision=12
+revision=13
 wrksrc="lua-${version}"
 makedepends="ncurses-devel readline-devel"
 short_desc="Powerful, fast, lightweight, embeddable scripting language (5.1.x)"
@@ -52,7 +52,7 @@ do_install() {
 }
 
 lua51-devel_package() {
-	depends="${makedepends} lua51>=${version}_${revision}"
+	depends="lua51>=${version}_${revision}"
 	short_desc+=" - development files"
 	alternatives="
 	 lua-devel:lua:/usr/share/doc/lua5.1

From 515690214e3f5454757fb6593fd0157e30fb2ab9 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=89rico=20Nogueira?= <erico.erc@gmail.com>
Date: Tue, 18 May 2021 23:49:34 -0300
Subject: [PATCH 3/4] lua52: -devel only depends on self.

Only the lua binary depends on libreadline and ncurses, not the library,
so lua52-devel doesn't need to depend on their devel packages.
---
 srcpkgs/lua52/template | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/srcpkgs/lua52/template b/srcpkgs/lua52/template
index b26e54aba8c5..ecdf8e35b1c0 100644
--- a/srcpkgs/lua52/template
+++ b/srcpkgs/lua52/template
@@ -1,7 +1,7 @@
 # Template file for 'lua52'
 pkgname=lua52
 version=5.2.4
-revision=11
+revision=12
 wrksrc=lua-${version}
 makedepends="ncurses-devel readline-devel"
 short_desc="Powerful, fast, lightweight, embeddable scripting language (5.2.x)"
@@ -52,7 +52,7 @@ do_install() {
 }
 
 lua52-devel_package() {
-	depends="${makedepends} lua52>=${version}_${revision}"
+	depends="lua52>=${version}_${revision}"
 	short_desc+=" - development files"
 	alternatives="
 	 lua-devel:lua:/usr/share/doc/lua5.2

From 744cd2b9e2bd3c27a3bb09d1b1086fca50d6b1cd Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=89rico=20Nogueira?= <erico.erc@gmail.com>
Date: Tue, 18 May 2021 23:49:53 -0300
Subject: [PATCH 4/4] lua54: -devel only depends on self.

Only the lua binary depends on libreadline and ncurses, not the library,
so lua54-devel doesn't need to depend on their devel packages.
---
 srcpkgs/lua54/template | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/srcpkgs/lua54/template b/srcpkgs/lua54/template
index 47d4425cd63f..a897183fa2ed 100644
--- a/srcpkgs/lua54/template
+++ b/srcpkgs/lua54/template
@@ -1,7 +1,7 @@
 # Template file for 'lua54'
 pkgname=lua54
 version=5.4.3
-revision=1
+revision=2
 wrksrc=lua-${version}
 makedepends="ncurses-devel readline-devel"
 short_desc="Powerful, fast, lightweight, embeddable scripting language (5.4.x)"
@@ -53,7 +53,7 @@ do_install() {
 }
 
 lua54-devel_package() {
-	depends="${makedepends} lua54>=${version}_${revision}"
+	depends="lua54>=${version}_${revision}"
 	short_desc+=" - development files"
 	alternatives="
 	 lua-devel:lua:/usr/share/doc/lua5.4