a) I think it supports command history with arrow keys. b) It is somewhat different from the existing Sentry shell - one has a concept of a session and state, another works as a normal shell command-line tool. I think there is benefit in both. On the other hand, I am not a big fan of the existing shell interface - that's why a while ago I implemented a different version in GO (which unfortunately doesn't support Kerberos - it requires more work for that). I think that one way or another the current shell should be redone, but there is a value in both. c) Yes, it makes sense to add a script in the bin directory. I was mostly running it with the run_sentry script which runs things via maven which wouldn't work in real installations. d) slf4j jars - isn't it handled already - most of other things need it e) I plan to merge it to master - there is no reason it shouldn't work on master as well.
On Mon, Jun 26, 2017 at 4:50 AM, Colm O hEigeartaigh <[email protected]> wrote: > Thanks, it looks pretty cool. A few comments/questions: > > a) Is there a way to configure it to list again the previous command by > pressing the up arrow key? > b) Is this meant to deprecate the Sentry Shell? Or compliment it? > c) Will you add a script to the distribution bin directory to make it easy > to run it? > d) The distribution lib directory needs the slf4j jars in it as well to get > it to run there. > e) Will you merge this to master as well so that will be in 1.8.0? > > Colm. > > On Fri, Jun 23, 2017 at 6:27 PM, Alexander Kolbasov <[email protected]> > wrote: > > > Here is another doc about Cliche from the original developer: > > > > https://www.codeproject.com/Articles/55088/Create-Minimal- > > UI-in-No-Time-at-all-Using-Cliche-S > > > > On Fri, Jun 23, 2017 at 1:22 AM, Colm O hEigeartaigh < > [email protected]> > > wrote: > > > > > Is there a tutorial on any documentation on how it works? MIT license > is > > > fine, just make sure that we update our NOTICE and LICENSE file > > > accordingly, and make sure the MIT license is shipped with the > > > distribution. > > > > > > Colm. > > > > > > On Fri, Jun 23, 2017 at 12:27 AM, Alexander Kolbasov < > [email protected] > > > > > > wrote: > > > > > > > We had this discussion a while ago, I'd like to being it back on the > > > table. > > > > > > > > I implemented a simple Sentry interactive CLI that is based on Cliche > > > > library (https://github.com/budhash/cliche). > > > > > > > > The library is available under MIT license - I guess it shouldn't be > a > > > > problem. > > > > > > > > I opened SENTRY-1812 to track this issue and posted the initial patch > > > > there. > > > > > > > > Please let me know what do you think about it. > > > > > > > > - Alex > > > > > > > > > > > > > > > > -- > > > Colm O hEigeartaigh > > > > > > Talend Community Coder > > > http://coders.talend.com > > > > > > > > > -- > Colm O hEigeartaigh > > Talend Community Coder > http://coders.talend.com >
