From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from localhost (localhost [127.0.0.1]) by turing.freelists.org (Avenir Technologies Mail Multiplex) with ESMTP id 59C1D212AC for ; Wed, 24 Jul 2019 13:11:44 -0400 (EDT) Received: from turing.freelists.org ([127.0.0.1]) by localhost (turing.freelists.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id IijM5-M_T3R4 for ; Wed, 24 Jul 2019 13:11:44 -0400 (EDT) Received: from smtp32.i.mail.ru (smtp32.i.mail.ru [94.100.177.92]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by turing.freelists.org (Avenir Technologies Mail Multiplex) with ESMTPS id E1BE9212DA for ; Wed, 24 Jul 2019 13:11:43 -0400 (EDT) Date: Wed, 24 Jul 2019 20:11:41 +0300 From: Konstantin Osipov Subject: [tarantool-patches] Re: [PATCH 4/5] box/lua/console: Provide output_default function to setup default output Message-ID: <20190724171141.GD4764@atlas> References: <20190723223113.16084-1-gorcunov@gmail.com> <20190723223113.16084-5-gorcunov@gmail.com> <20190724171051.GC4764@atlas> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20190724171051.GC4764@atlas> Sender: tarantool-patches-bounce@freelists.org Errors-to: tarantool-patches-bounce@freelists.org Reply-To: tarantool-patches@freelists.org List-Help: List-Unsubscribe: List-software: Ecartis version 1.0.0 List-Id: tarantool-patches List-Subscribe: List-Owner: List-post: List-Archive: To: Cyrill Gorcunov Cc: tml , Alexander Turenko , Kirill Yukhin * Konstantin Osipov [19/07/24 20:10]: > * Cyrill Gorcunov [19/07/24 01:35]: > > A user might need to use lua output as default serializer so instead > > of requiring him setting up the output every new session that named > > default output mode may be used instead with help of command > > > > > require('console').set_default_output("lua") > > > > Also we provide > > > > > require('console').get_default_output("lua") > > > > to obtain current setting. > > > > Part-of #3834 > > there is a bug in this patch - it changes the output of an already > existing console. It should not happen, only new consoles should > be affected. > > Please submit a fix. The fix is to table.deepcopy() d from default. -- Konstantin Osipov, Moscow, Russia