ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: Anton Farygin <rider@basealt.ru>
To: devel@lists.altlinux.org
Subject: Re: [devel] APT development /code review tool
Date: Fri, 25 Oct 2019 15:46:25 +0300
Message-ID: <3f6d5fac-c8e8-976b-4e32-d60ab550e945@basealt.ru> (raw)
In-Reply-To: <20191025154121.d3b8cd4853b95d0d623c3104@altlinux.org>

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 между участниками разработки, то 
данный вопрос даже бы не поднимался.



  reply	other threads:[~2019-10-25 12:46 UTC|newest]

Thread overview: 87+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-10-24 15:27 [devel] APT development Ivan Zakharyaschev
2019-10-24 15:29 ` Aleksei Nikiforov
2019-10-24 15:37   ` Ivan Zakharyaschev
2019-10-24 15:48     ` Aleksei Nikiforov
2019-10-25  5:13       ` Mikhail Novosyolov
2019-10-25  7:11         ` Ivan Zakharyaschev
2019-10-24 15:51   ` Andrey Savchenko
2019-10-24 16:02     ` Aleksei Nikiforov
2019-10-24 16:17       ` Andrey Savchenko
2019-10-24 17:35         ` Дмитрий Державин
2019-10-24 17:52           ` Anton Farygin
2019-10-24 20:41             ` Andrey Savchenko
2019-10-24 21:44               ` Michael Shigorin
2019-10-25  4:04                 ` Alexey V. Vissarionov
2019-10-25  5:03                 ` [devel] APT development /code review tool Anton Farygin
2019-10-25 11:41                   ` Michael Shigorin
2019-10-25 11:48                     ` Anton Farygin
2019-10-25 12:30                   ` Andrey Savchenko
2019-10-25 12:32                     ` Anton Farygin
2019-10-25 12:41                       ` Andrey Savchenko
2019-10-25 12:46                         ` Anton Farygin [this message]
2019-10-25 12:47                           ` Michael Shigorin
2019-10-25 12:51                             ` Anton Farygin
2019-10-25 13:07                               ` [devel] [JT] полиси -- не панацея (was: APT development /code review tool) Michael Shigorin
2019-10-25 14:18                                 ` [devel] [JT] полиси -- не панацея Anton Farygin
2019-10-25 16:29                             ` [devel] APT development /code review tool Vladimir D. Seleznev
2019-10-25 12:32                     ` Aleksei Nikiforov
2019-10-25 12:40                       ` Dmitry V. Levin
2019-10-25 12:47                         ` Andrey Savchenko
2019-10-25 12:48                           ` Dmitry V. Levin
2019-10-25  7:27               ` [devel] APT development Aleksei Nikiforov
2019-10-25 12:29                 ` Andrey Savchenko
2019-10-25 12:42                   ` Aleksei Nikiforov
2019-10-25 12:45                     ` Andrey Savchenko
2019-10-25 12:49                       ` Anton Farygin
2019-10-25 12:45                     ` Michael Shigorin
2019-10-25  4:02             ` Alexey V. Vissarionov
2019-10-25  4:47               ` Anton Farygin
2019-10-25  7:24         ` Aleksei Nikiforov
2019-10-25 11:35           ` [devel] [JT] WEB vs IMAP Anton Farygin
2019-10-25 12:11             ` Andrey Savchenko
2019-10-25 12:21               ` Anton Farygin
2019-10-25 12:44               ` Sergey Afonin
2019-10-25 12:47                 ` Anton Farygin
2019-10-26 21:54         ` [devel] APT development Mikhail Efremov
2019-10-27  5:53           ` Anton Farygin
2019-10-28 21:29             ` Mikhail Novosyolov
2019-10-28 22:28               ` Vladimir D. Seleznev
2019-10-29  4:57                 ` Anton Farygin
2019-10-29 10:23                   ` Andrey Savchenko
2019-10-29 11:13                     ` Anton Farygin
2019-10-29 13:04                       ` Andrey Savchenko
2019-10-29 13:27                         ` Anton Farygin
2019-10-29 14:09                           ` Andrey Savchenko
2019-10-29 14:33                             ` Anton Farygin
2019-10-29 22:17                             ` [devel] FSF copyright assignment Dmitry V. Levin
2019-10-29 23:22                                 ` Dmitry V. Levin
2019-10-30 12:04                                   ` Alexey V. Vissarionov
2019-10-30  5:12                             ` [devel] APT development Anton Farygin
2019-10-30  8:57                               ` Andrey Savchenko
2019-10-30  9:01                                 ` Anton Farygin
2019-10-30 10:17                                   ` Andrey Savchenko
2019-10-30 10:59                                     ` Anton Farygin
2019-10-30 11:31                                       ` Andrey Savchenko
2019-10-30 11:38                                         ` Anton Farygin
2019-10-30 12:03                                           ` Andrey Savchenko
2019-10-31  4:21                                             ` Anton Farygin
2019-10-30 10:28                               ` [devel] CLA vs DCO Dmitry V. Levin
2019-10-30 10:57                                 ` Anton Farygin
2019-10-25  0:52   ` [devel] APT development Dmitry V. Levin
2019-10-25  4:55     ` Anton Farygin
2019-10-25  7:08       ` Ivan Zakharyaschev
2019-10-25 11:26         ` Anton Farygin
2019-10-25 12:01       ` [devel] про gerrit Dmitry V. Levin
2019-10-25 12:20         ` Anton Farygin
2019-10-25 12:43           ` Alexey Gladkov
2019-10-25 12:45             ` Anton Farygin
2019-10-25 12:55               ` [devel] про gitlab/github Alexey Gladkov
2019-10-25 13:16                 ` Andrey Savchenko
2019-10-25 14:04               ` [devel] про gerrit Alexey Gladkov
2019-10-25 14:16                 ` Anton Farygin
2019-10-26  8:05                   ` Konstantin Lepikhov
2019-10-25 12:48             ` Aleksei Nikiforov
2019-10-25 16:25             ` Vladimir D. Seleznev
2019-10-25  7:31     ` [devel] APT development Aleksei Nikiforov
2019-10-25 16:26     ` Vladimir D. Seleznev
2019-10-24 15:32 ` Ivan Zakharyaschev

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=3f6d5fac-c8e8-976b-4e32-d60ab550e945@basealt.ru \
    --to=rider@basealt.ru \
    --cc=devel@lists.altlinux.org \
    /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

ALT Linux Team development discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/devel/0 devel/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 devel devel/ http://lore.altlinux.org/devel \
		devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru
	public-inbox-index devel

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://lore.altlinux.org/org.altlinux.lists.devel


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git