Subject: [bug#76770] [PATCH] gnu: lua: compile with readline support

From: Vagrant Cascadian <[email-masked]>
Date: Mon, 10 Mar 2025 15:46:48 -0700

On 2025-03-05, kimapr wrote:
> * gnu/packages/lua.scm (lua)[arguments]: change target to
>   "linux-readline".
> ---
>  gnu/packages/lua.scm | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/gnu/packages/lua.scm b/gnu/packages/lua.scm
> index 4c65bcbf12..b3d4cbf5d2 100644
> --- a/gnu/packages/lua.scm
> +++ b/gnu/packages/lua.scm
> @@ -109,7 +109,7 @@ (define-public lua
>               (string-append "CC=" ,(cc-for-target))
>               (string-append "SYSLIBS=-L" (assoc-ref %build-inputs
> "readline")
>                              "/lib")
> -             "linux")
> +             "linux-readline")
>         #:phases
>         (modify-phases %standard-phases
>           (delete 'configure)
> -- 
> 2.47.1

Patch was slightly malformed for some reason, but being a one-liner, was
easy to apply manually!

At first glance, it increases the results from guix size only slightly
from 77.0MB to 84.4MB, and has a smallish number of dependents...

But [email-masked] [email-masked] and [email-masked] inherit from lua, so the impact is much
larger than a quick glance suggests (especially [email-masked]):

    $ guix refresh --list-dependent [email-masked] [email-masked] [email-masked] [email-masked]
    Building the following 359 packages would ensure 604 dependent
    packages are rebuilt: [email-masked] [email-masked] [email-masked]
    [email-masked] [email-masked] ...

I did successfully build [email-masked] and it's four dependents.

https://guix.gnu.org/manual/devel/en/guix.html#Managing-Patches-and-Branches

  Changes which affect more than 300 dependent packages (see Invoking guix
  refresh) should first be pushed to a topic branch other than master

Would it make sense to exclude the older versions from the change? Then
it could go to the git master banch, in my opinion.

live well,
  vagrant