Tarantool development patches archive
 help / color / mirror / Atom feed
From: Cyrill Gorcunov <gorcunov@gmail.com>
To: Konstantin Osipov <kostja@tarantool.org>
Cc: tml <tarantool-patches@freelists.org>,
	Alexander Turenko <alexander.turenko@tarantool.org>,
	Kirill Yukhin <kyukhin@tarantool.org>
Subject: [tarantool-patches] Re: [PATCH 2/3] box/lua/console: Add support for lua output format
Date: Sat, 13 Jul 2019 00:44:49 +0300	[thread overview]
Message-ID: <20190712214449.GG4602@uranus.lan> (raw)
In-Reply-To: <20190712204527.GB31930@atlas>

On Fri, Jul 12, 2019 at 11:45:27PM +0300, Konstantin Osipov wrote:
> * Cyrill Gorcunov <gorcunov@gmail.com> [19/07/12 22:08]:
> > Historically we use YAML format to print results of operation to
> > a console. Moreover our test engine is aiming YAML as a primary format
> > to compare results of test runs. Still we need an ability to print
> > results in a different fasion, in particular one may need to use
> > the console in a REPL way so that the results would be copied and
> > pased back to further processing.
> 
> Can you please come up with a patch for test-run that changes the
> test engine to set the output explicitly in suite.ini, so that
> it's not broken by changing the default?

I think about it and this gonna be new patches on top of the series.
Look, the main purpose of the patchset is to be completely transparent
to current codebase and the patches must not break anything already
running (including tests). That's why modifying test engine is my
next step, also I need to provide tests for this new console output.
But again, my initial main purpose -- to not break anything now.

  reply	other threads:[~2019-07-12 21:44 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-07-12 19:04 [tarantool-patches] [PATCH v3 0/3] box/lua/console: Add initial " Cyrill Gorcunov
2019-07-12 19:04 ` [tarantool-patches] [PATCH 1/3] third_party/serpent: Add serpent repo Cyrill Gorcunov
2019-07-12 19:04 ` [tarantool-patches] [PATCH 2/3] box/lua/console: Add support for lua output format Cyrill Gorcunov
2019-07-12 20:45   ` [tarantool-patches] " Konstantin Osipov
2019-07-12 21:44     ` Cyrill Gorcunov [this message]
2019-07-12 19:04 ` [tarantool-patches] [PATCH 3/3] box/lua/console: Verify remote protocol for compatibility Cyrill Gorcunov
2019-07-12 20:47   ` [tarantool-patches] " Konstantin Osipov
2019-07-12 20:49     ` Konstantin Osipov
2019-07-12 21:51       ` Cyrill Gorcunov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20190712214449.GG4602@uranus.lan \
    --to=gorcunov@gmail.com \
    --cc=alexander.turenko@tarantool.org \
    --cc=kostja@tarantool.org \
    --cc=kyukhin@tarantool.org \
    --cc=tarantool-patches@freelists.org \
    --subject='[tarantool-patches] Re: [PATCH 2/3] box/lua/console: Add support for lua output format' \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox