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

lvasiliev lvasiliev at tarantool.org
Fri Apr 10 11:41:08 MSK 2020


Hi! Thanks for the reviews.

On 10.04.2020 3:00, Vladislav Shpilevoy wrote:
> 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".
> 
OK
> 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