[Tarantool-patches] [PATCH v3 0/2] Adapt luarocks for tarantoolctl

Vladislav Shpilevoy v.shpilevoy at tarantool.org
Fri Apr 10 03:00:36 MSK 2020


Hi!

Thanks for the patchset!

LGTM.

But please, next time don't silently respond with a new patch
on my comments. I may be wrong sometimes. Honestly I am wrong
quite often. Like Vladimir said once, "mailing list exists for
discussions".

I added Yaroslav to CC since he expressed a will to join the
review.

On 09/04/2020 08:33, Leonid Vasiliev wrote:
> https://github.com/tarantool/tarantool/issues/4629
> https://github.com/tarantool/luarocks/tree/lvasiliev/gh-4629-add-chdir-to-make
> See corresponding patch for tarantoolctl
> (https://github.com/tarantool/tarantool/tree/lvasiliev/gh-4629-forward-flags)
> 
> Motivation:
> Imperfect integretion of the Luarocks to tarantoolctl
> (malfunctioning flags, disabled commands)
> 
> Was done:
> whitelist of tarantoolctl don't used for luarocks flags
> Option chdir has been moved from tarantoolctl to luarocks
> 
> @ChangeLog - see a comment in tarantool
> 
> Leonid Vasiliev (2):
>   Add the chdir option for make
>   Delete flags which can't be used with tarantoolctl rocks
> 
>  src/luarocks/cmd/help.lua | 5 -----
>  src/luarocks/cmd/make.lua | 8 ++++++++
>  src/luarocks/util.lua     | 5 +----
>  3 files changed, 9 insertions(+), 9 deletions(-)
> 


More information about the Tarantool-patches mailing list