[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 4/6] gnu: Add lua5.2-bitop.
From: |
Ricardo Wurmus |
Subject: |
Re: [PATCH 4/6] gnu: Add lua5.2-bitop. |
Date: |
Wed, 25 Jan 2017 10:17:21 +0100 |
User-agent: |
mu4e 0.9.18; emacs 25.1.1 |
Leo Famulari <address@hidden> writes:
> On Tue, Jan 24, 2017 at 03:40:13PM +0100, Ricardo Wurmus wrote:
>> * gnu/packages/lua.scm (lua5.2-bitop): New variable.
>
>> + (arguments
>> + `(#:test-target "test"
>> + #:make-flags
>> + (list "INSTALL=install -pD"
>> + (string-append "INSTALLPATH=printf "
>
> What's up with this "printf"? Does it actually execute the value of
> INSTALLPATH?
INSTALLPATH defaults to “lua somefile.lua”, and it is applied to the
name of the module (“bit”). If I just changed INSTALLPATH to a literal
path, the “install” target would try to execute it with “bit” as an
argument.
This is why I’m using “printf”, which when given a string literal
ignores any other arguments. The result is just the install path. I
chose this method over patching the Makefile.
--
Ricardo
GPG: BCA6 89B6 3655 3801 C3C6 2150 197A 5888 235F ACAC
https://elephly.net
- [PATCH 1/6] gnu: Add libmpack., Ricardo Wurmus, 2017/01/24
- [PATCH 2/6] gnu: Add lua-libmpack., Ricardo Wurmus, 2017/01/24
- [PATCH 3/6] gnu: Add lua5.2-libmpack., Ricardo Wurmus, 2017/01/24
- [PATCH 4/6] gnu: Add lua5.2-bitop., Ricardo Wurmus, 2017/01/24
- [PATCH 5/6] gnu: Add lua5.2-lpeg., Ricardo Wurmus, 2017/01/24
- [PATCH 6/6] gnu: Add neovim., Ricardo Wurmus, 2017/01/24