[bug#29562] [PATCH] gnu: lua5.1-socket: Update to 3.0-rc1 (for IPv6 support).

[bug#29562] [PATCH] gnu: lua5.1-socket: Update to 3.0-rc1 (for IPv6 support).

[Clément Lassieur]

* gnu/packages/lua.scm (lua5.1-socket): Update to 3.0-rc1.
[source]: Change upstream URI.
[arguments]: Set INSTALL_TOP environment variable instead of INSTALL_TOP_SHARE
and INSTALL_TOP_LIB.
---
 gnu/packages/lua.scm | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

diff --git a/gnu/packages/lua.scm b/gnu/packages/lua.scm
index d5f55b963..3a528599f 100644
--- a/gnu/packages/lua.scm
+++ b/gnu/packages/lua.scm
@@ -173,21 +173,20 @@ language.")
 (define-public lua5.1-socket
   (package
     (name "lua5.1-socket")
-    (version "2.0.2")
+    (version "3.0-rc1")
     (source (origin
               (method url-fetch)
-              (uri (string-append "http://files.luaforge.net/releases/"
-                                  "luasocket/luasocket/luasocket-"
-                                  version "/luasocket-" version ".tar.gz"))
+              (uri (string-append
+                    "https://github.com/diegonehab/luasocket/archive/v"
+                    version ".tar.gz"))
               (sha256
                (base32
-                "19ichkbc4rxv00ggz8gyf29jibvc2wq9pqjik0ll326rrxswgnag"))))
+                "0j8jx8bjicvp9khs26xjya8c495wrpb7parxfnabdqa5nnsxjrwb"))))
     (build-system gnu-build-system)
     (arguments
      `(#:make-flags
        (let ((out (assoc-ref %outputs "out")))
-         (list (string-append "INSTALL_TOP_SHARE=" out "/share/lua/5.1")
-               (string-append "INSTALL_TOP_LIB=" out "/lib/lua/5.1")))
+         (list (string-append "INSTALL_TOP=" out)))
        #:phases
        (modify-phases %standard-phases
          (delete 'configure)
-- 
2.15.1

[bug#29562] [PATCH] gnu: lua5.1-socket: Update to 3.0-rc1 (for IPv6 support).

[Leo Famulari]

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

On Mon, Dec 04, 2017 at 04:14:48PM +0100, Clément Lassieur wrote:
> * gnu/packages/lua.scm (lua5.1-socket): Update to 3.0-rc1.
> [source]: Change upstream URI.
> [arguments]: Set INSTALL_TOP environment variable instead of INSTALL_TOP_SHARE
> and INSTALL_TOP_LIB.

Typically we don't package release candidates or beta software.
Sometimes we do, if there are critical bugs fixed in the RC that we
can't backport, or if the upstream development seems to have stalled
without issuing a final release.

Can you give more detail about this case?

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

[bug#29562] [PATCH] gnu: lua5.1-socket: Update to 3.0-rc1 (for IPv6 support).

[Clément Lassieur]

Leo Famulari <leo@famulari.name> writes:

> On Mon, Dec 04, 2017 at 04:14:48PM +0100, Clément Lassieur wrote:
>> * gnu/packages/lua.scm (lua5.1-socket): Update to 3.0-rc1.
>> [source]: Change upstream URI.
>> [arguments]: Set INSTALL_TOP environment variable instead of INSTALL_TOP_SHARE
>> and INSTALL_TOP_LIB.
>
> Typically we don't package release candidates or beta software.
> Sometimes we do, if there are critical bugs fixed in the RC that we
> can't backport, or if the upstream development seems to have stalled
> without issuing a final release.
>
> Can you give more detail about this case?

Sure.  This patch adds support for IPv6 to Prosody.  See
https://prosody.im/doc/ipv6 and
https://lists.gnu.org/archive/html/guix-devel/2017-12/msg00050.html.

Even though it's a release candidate, it's four years old, it's
recommended by Prosody, and I believe IPv6 is really important.

[bug#29562] [PATCH] gnu: lua5.1-socket: Update to 3.0-rc1 (for IPv6 support).

[Leo Famulari]

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

On Mon, Dec 04, 2017 at 11:08:16PM +0100, Clément Lassieur wrote:
> Leo Famulari <leo@famulari.name> writes:
> > Can you give more detail about this case?
> 
> Sure.  This patch adds support for IPv6 to Prosody.  See
> https://prosody.im/doc/ipv6 and
> https://lists.gnu.org/archive/html/guix-devel/2017-12/msg00050.html.
> 
> Even though it's a release candidate, it's four years old, it's
> recommended by Prosody, and I believe IPv6 is really important.

Given that this RC is 4 years old, implements a critical feature (IPv6),
and is recommended by one of our major packages which is also its own
user, LGTM :)

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

[bug#29562] [PATCH] gnu: lua5.1-socket: Update to 3.0-rc1 (for IPv6 support).

[Clément Lassieur]

Leo Famulari <leo@famulari.name> writes:

> On Mon, Dec 04, 2017 at 11:08:16PM +0100, Clément Lassieur wrote:
>> Leo Famulari <leo@famulari.name> writes:
>> > Can you give more detail about this case?
>> 
>> Sure.  This patch adds support for IPv6 to Prosody.  See
>> https://prosody.im/doc/ipv6 and
>> https://lists.gnu.org/archive/html/guix-devel/2017-12/msg00050.html.
>> 
>> Even though it's a release candidate, it's four years old, it's
>> recommended by Prosody, and I believe IPv6 is really important.
>
> Given that this RC is 4 years old, implements a critical feature (IPv6),
> and is recommended by one of our major packages which is also its own
> user, LGTM :)

Haha, thank you for the review :-)