<div dir="ltr">Hi, guys.<br><br>Recently we talked to Leonid in chat and discussed the necessity of<br>removing flags. I've offered Leonid to check behavior, so here is what<br>I've found.<br><br>My point of view (now and then) is that they're neither useful nor<br>harmful. Earlier I did agree to blacklist them, but looking at Leonids<br><div>patch set this week I've changed my mind as the second commit seems</div><div>just worthless to me.</div><br>Inline help says:<br><br>```<br><span style="font-family:monospace">--lua-dir=<prefix>     Which Lua installation to use.<br>--lua-version=<ver>    Which Lua version to use.<br>--global               Use the system tree when `local_by_default` is `true`.<br>--local                Use the tree in the user's home directory.<br>                       To enable it, see '/opt/tarantool-install/bin/tarantoolctl help path'.<br><br>Rocks trees in use:                  <br>        /tmp/tmp.JY1fgy8La9/.rocks ("user") <br>        /opt/tarantool-install ("system")    </span><br>```<br><br>* Tarantool does define `LOCAL_BY_DEFAULT = true`,<br>  (i.e. `--local` is our default behavior)<br><br>* "System tree" is the place where tarantool is installed<br>  (`-DCMAKE_INSTALL_PREFIX`).<br><br>* "Home directory" in tarantool is overriden with `./.rocks`.<br><br>* See "rocks: update luarocks to 3.1.1" <a href="https://github.com/tarantool/tarantool/commit/4222c1f6451973f446f9c8be28c7012435416fa3" target="_blank">4222c1f6</a>.<br><br>* Both `--local` and `--global` are only shortcuts for predefined<br>  `--tree=<tree>`, which isn't going to be removed.<br><br>* Speaking of `--lua-dir` and `--lua-version`, they look to be neither<br>  useful nor harmful.<br><br>Please, notice that command-line help refers to `tarantoolctl help<br>path`, but<br><br>1. It should be `tarantoolctl rocks help` (I guess there are many of<br>   such places)<br>2. `rocks path` command is disabled in your patch.<br><br>I'm a lazy developer and I don't like unnecessary restrictions. If I<br>were you, I'd drop the second commit ("Delete flags which can't be used<br>with tarantoolctl rocks" 1b51b2f) and enable back `rocks path` command.<br>Let people decide whether they need it or not.<br><br>Finally, a few words about documentation. I've tried to write down all<br>supported commands and flags, and I found it's really difficult and<br>verbose. I'm withdrawing my suggestion to clarify it, I don't know what<br>to do. Perhaps, documentation should just refer to inline help, as it's the<br>only up-to-date information, and still correct enough.<br><br><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, 12 Apr 2020 at 20:25, lvasiliev <<a href="mailto:lvasiliev@tarantool.org" target="_blank">lvasiliev@tarantool.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
<br>
On 11.04.2020 17:23, Vladislav Shpilevoy wrote:<br>
<br>
>> As of the second one ("Delete flags which can't be used with<br>
>> tarantoolctl rocks" 1b51b2f <<a href="https://github.com/tarantool/luarocks/commit/1b51b2fe1c194719a7a8a74220a954a91c21f2a7" rel="noreferrer" target="_blank">https://github.com/tarantool/luarocks/commit/1b51b2fe1c194719a7a8a74220a954a91c21f2a7</a>>), I'm worried that our fork every day goes<br>
>> farther from upstream. You didn't say a word about necessity to /remove/<br>
>> it, but it looks like another postponed problem.<br>
> <br>
> Leonid, btw, why do we remove them, again? I was always thinking it<br>
> is a part of the ticket, but I don't see it there except in your<br>
> comments.<br>
Because after talking with Yaroslav, we decided that the user should not <br>
be able to use these flags.<br>
When I worked on task, I was still too young and did not know that all <br>
summaries of discussions should be written down in comments on task.<br>
</blockquote></div><br clear="all"><br>-- <br><div dir="ltr"><div dir="ltr"><span><div><div dir="ltr">С уважением. <br>Дынников Ярослав.<br></div></div></span></div></div>