In lua-5.4.2/src/Makefile (release version), use './$(LUA_T)' instead of './lua' for a target 'test'

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

In lua-5.4.2/src/Makefile (release version), use './$(LUA_T)' instead of './lua' for a target 'test'

Lemures Lemniscati
Hi!

This is a small patch, just for consistency.

In lua-5.4.2/src/Makefile ( https://www.lua.org/ftp/lua-5.4.2.tar.gz ),

* A name of a lua executable is configurable by a variable LUA_T.
* But, in a target 'test', the name of the lua executable is hard-coded as './lua'.

And, the patch modifies it to use './$(LUA_T)' instead of './lua'.


================
diff -u -r a/lua-5.4.2/src/Makefile b/lua-5.4.2/src/Makefile
--- a/lua-5.4.2/src/Makefile 2020-11-11 03:39:37.000000000 +0900
+++ b/lua-5.4.2/src/Makefile 2021-01-23 11:16:35.000000000 +0900
@@ -67,7 +67,7 @@
  $(CC) -o $@ $(LDFLAGS) $(LUAC_O) $(LUA_A) $(LIBS)
 
 test:
- ./lua -v
+ ./$(LUA_T) -v
 
 clean:
  $(RM) $(ALL_T) $(ALL_O)
================

Regards,

--
Lem
Reply | Threaded
Open this post in threaded view
|

Re: In lua-5.4.2/src/Makefile (release version), use './$(LUA_T)' instead of './lua' for a target 'test'

Luiz Henrique de Figueiredo
> In lua-5.4.2/src/Makefile ( https://www.lua.org/ftp/lua-5.4.2.tar.gz ),
>
> * A name of a lua executable is configurable by a variable LUA_T.
> * But, in a target 'test', the name of the lua executable is hard-coded as './lua'.
>
> And, the patch modifies it to use './$(LUA_T)' instead of './lua'.

Thanks for the report and patch.