Hi, Leonid.
The first patch ("Add the chdir option for make"
8e8db7f) looks good to
me. I hope you'll be able to find time and propose it to upstream, in my
opinion it'll helpful there too.
As of the second one ("Delete flags which can't be used with
tarantoolctl rocks"
1b51b2f), I'm worried that our fork every day goes
farther from upstream. You didn't say a word about necessity to remove
it, but it looks like another postponed problem. Luarocks version is
already 3.3.1, while we still basing 3.1.1. And every such commit makes
it harder to rebase.
The last time we were upgrading luarocks we've spent several weeks of
worktime to resolve all the problems. And, disappointingly, those
changes weren't merged to 1.10, so my colleagues usually come back with
the same problems as 2 years ago. Please, don't screw over our future
selves.
Speaking about the patch in tarantool itself ("rocks: forward options to
luarocks"
28a3b55f), I've run some commands and it seems to be OK.
But I agree with Vlad that documentation should be more detailed. You
can't just link the luarocks wiki (at first, because our version
differs, but, moreover, because it's imperfect too). I don't believe
anybody but you would be able to tell supported features from
unsupported and find mistakes in wiki, so I suggest to clarify it
somewhere (in the commit message or in any other place you like).