From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: To: devel@lists.altlinux.org References: <95db8005-f164-dbfc-434b-79673fb33cec@altlinux.org> <20191024185111.9fed14d1555cf6980401c1b3@altlinux.org> <5d36547b-547a-135b-158f-6beaa6c24d19@altlinux.org> <20191024191719.c28fc34f3a3d5fec0d4b8bd2@altlinux.org> <87tv7y11yk.fsf@asia.home.dd> <20191024234125.6ca4853d9f4ba199a57faa48@altlinux.org> <20191024214427.GA10569@imap.altlinux.org> <735cba2e-b3eb-1907-9a12-08e8e19c2d08@basealt.ru> <20191025153029.ef6c72a8114d1905917756db@altlinux.org> <46a17bb5-b32b-e159-4f3c-5670635bf78e@basealt.ru> <20191025154121.d3b8cd4853b95d0d623c3104@altlinux.org> From: Anton Farygin X-Opacus-Archived: none Organization: BaseALT Message-ID: <3f6d5fac-c8e8-976b-4e32-d60ab550e945@basealt.ru> Date: Fri, 25 Oct 2019 15:46:25 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.8.0 MIME-Version: 1.0 In-Reply-To: <20191025154121.d3b8cd4853b95d0d623c3104@altlinux.org> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Content-Language: ru Subject: Re: [devel] APT development /code review tool X-BeenThere: devel@lists.altlinux.org X-Mailman-Version: 2.1.12 Precedence: list Reply-To: ALT Linux Team development discussions List-Id: ALT Linux Team development discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 25 Oct 2019 12:46:26 -0000 Archived-At: List-Archive: List-Post: On 25.10.2019 15:41, Andrey Savchenko wrote: > On Fri, 25 Oct 2019 15:32:08 +0300 Anton Farygin wrote: >> On 25.10.2019 15:30, Andrey Savchenko wrote: >>> On Fri, 25 Oct 2019 08:03:51 +0300 Anton Farygin wrote: >>>> On 25.10.2019 0:44, Michael Shigorin wrote: >>>>> Поэтому если желающие развернут тот же gerrit для добровольного >>>>> участия в рецензировании кода -- в добрый путь. >>>> у желающих давно уже развёрнуто. Множество различных по величине >>>> проектов активно используют те или иные инструменты для online code review. >>>> >>>> Собственно сам процесс code review (наравне с CI) часто является одним >>>> из этапов для прохождения любых патчей в проект. >>>> >>>> Попробуй ради интереса сделать патчик к FreeIPA, LibreOffice, KDE, MySQL >>>> или QT  и отправить его в апстрим. >>> Действительно, попробуй что отправить через gerrit и т.п. code >>> review в ядро или glibc. О результатах напиши. >>> >> Через gerrit вообще невозможно что-то отправить. У него другой workflow. > Давай буквоедством не заниматься. Очевидно, что имелось ввиду > "засабмитить свой патч в апстрим как конечный результат обсуждения > с апстримом и возможной модификации предложенного патча по > результатам данного обсуждения". Если ты затребуешь gerrit для code > review твоих изменений ядра со словами "нихацу использовать > e-mail", тебя пошлют далеко и надолго, или просто проигнорят. > Естественно. Но у нас согласись немного другая ситуация. А именно - отсутствие внятных договорённостей между участниками разработки о средствах и инструментах этой разработки. Т.е. - если бы у нас было бы утверждёное policy, регламентирующие процесс взаимодействия по code review между участниками разработки, то данный вопрос даже бы не поднимался.