ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: Evgeny Sinelnikov <sin@altlinux.org>
To: ALT Linux Team development discussions <devel@lists.altlinux.org>
Subject: Re: [devel] Новая схема ведения исходников ядра
Date: Mon, 3 Jan 2022 10:17:42 +0400
Message-ID: <CAK42-GqE3tPEkWzVkyWpcu2pty46cmKhHv6Mbn-Z968M0qYROw@mail.gmail.com> (raw)
In-Reply-To: <CAGvFrt0ePtRVJf6-dD4Ka+TaHDs5v3F2ZKGmSY=qek5seY2nnA@mail.gmail.com>

пн, 3 янв. 2022 г. в 09:41, Aleksey Novodvorsky <aen@basealt.ru>:
>
>
>
>
>
> пн, 3 янв. 2022 г., 08:08 Evgeny Sinelnikov <sin@altlinux.org>:
>>
>> С новым годом!
>
>
> И вас!
>>
>>
>> чт, 23 дек. 2021 г. в 10:08, Andrey Savchenko <bircoph@altlinux.org>:
>> >
>> > On Thu, 23 Dec 2021 06:28:34 +0300 Mikhail Novosyolov wrote:
>> > >
>> > > 06.12.2021 15:12, Anton Farygin пишет:
>> > > > On 06.12.2021 14:41, Anton V. Boyarshinov wrote:
>> > > >> В Mon, 6 Dec 2021 15:27:11 +0400
>> > > >> Alexey Sheplyakov <asheplyakov@basealt.ru> пишет:
>> > > >>
>> > > >>> 2) Некий логически монолитный кусок кода (например, LSM модуль AltHa, или
>> > > >>>     драйвер dwmac-baikal) оказывается размазанным тонким слоем по N коммитам.
>> > > >>>     При переносе на более свежее ядро из этих N коммитов всё равно надо сделать
>> > > >>>     один патч (а затем доработать его вслед за изменившимися API).
>> > > >> Да, эти "стабильные" патчи живут в отдельных ветках feat-* и fix-*
>>
>> Где их искать?
>> Под какие мажорные релизы они поддерживаются?
>> Можно ли рассчитывать, что патчи из этих веток лягут без конфликтов на
>> более старшие минорые версии тех мажорных, под которые они ведутся?
>>
>> Здесь я их не наблюдаю:
>> - http://git.altlinux.org/gears/k/kernel-image-std-def.git
>> - http://git.altlinux.org/gears/k/kernel-image-un-def.git
>>
>> Наверное, это вычисляется. Но для этого потребуется:
>> - вытянуть гигабайт исходников;
>> - выяснить какой командой это вычисляется;
>> - то есть потратить кучу времени без гарантии, что не ошибся и всё
>> правильно сделал;
>> - без возможности быстро посмотреть через веб-интерфейс ту часть
>> истории, которая (казалось бы) должна быть доступна.
>>
>>
>> > > >> И, если я правильно понимаю, упоминаемым партнёрам они не очень
>> > > >> интересны. Зачем их при каждой сборке вытаскивать наверх ценой
>> > > >> испорченной истории мне не ясно.
>> > > >
>> > > > Если мне гит при каждом merge ядра будет ругаться на unrelated history, то я буду громко и сильно ругаться на тех людей, которые так делают.
>> > > >
>> > > > Давайте ещё раз подумаем и придумаем удобное решение.
>> > > >
>> > > > А как ведёт ядро та же убунта ? Почему у них нет этой проблемы ?
>> > > > https://git.launchpad.net/~canonical-kernel/ubuntu/+source/linux-oem/+git/focal?h=oem-5.10
>> > > >
>> > > По моим наблюдениям примерно годичной давности, убунта регулярно форс-пушит в свой гит, который часто не соответствует ушедшим в репозиторий исходникам. С таким гитом работать невозможно. Точно не пример для подражания.
>>
>> Ну, наверное, убунта это делает из каких-то соображений. Не из прихоти
>> же? Субъективные предпочтения тоже бывают, но тут не тот случай, когда
>> всё к этому сводится.
>>
>>
>> > Согласен. За push --force в репозитории, которые в разработке
>> > используются другими людьми, нужно руки отрывать.
>>
>> Не согласен. Смотря в каком контексте идёт речь о разработке. Пачка
>> запутанных merge-коммитов - это интересная история для пакета, но не
>> для разработки того исходного кода, который собирается в этом пакете.
>>
>> О какой, вообще, разработке идёт речь? Что вы называете разработкой,
>> если отсутствует плановый сценарий передачи исходников в апстрим и не
>> заложен подход взаимодействия с другими разработчиками, которые тоже
>> взаимодействуют с апстримом и этот апстрим не мы?
>>
>> "Плановый сценарий передачи исходников в апстрим" отсутствует, в
>> данном случае, потому, что "выковыривать" размазанные по истории патчи
>> - это антипродуктивный, нерабочий сценарий для любого разработчика,
>> кроме автора пакета, да и для него тоже неудобная;
>> "Подход взаимодействия с другими разработчиками, которые тоже
>> взаимодействуют с апстримом" не заложен, в данном случае, потому, что
>> сторонним разработчикам не менее нужны наши патчи в доступном виде,
>> чем апстриму, если мы хотим этой самой совместной разработки.
>>
>> При работе с апстримом и попытках передать астриму исходники
>> приходится делать регулярный git rebase и git push --force. Это же
>> очевидно. Вариант - передать патч в электронном сообщении только
>> скрывает отсутствие этого --force, который, фактически, всё равно,
>> имеет место быть, поскольку этот патч равносилен цепочке git
>> cherry-pick && git rebase, после которой неизбежен --force или
>> параллельная история с кучей merge-коммитов, которая никому не
>> интересна для разработки, даже нам самим.
>>
>> Не стоит называть любую сборку пакета в репозиторий участием в
>> разработке. Это иногда так, но очень и очень редко. Во многих случаях
>> авторы даже не в курсе, что мы их проект собираем. И разница тут
>> именно в том, что для передачи исходников и для совместного ведения
>> исходного кода приходится чем-то платить. В нашем случае, одним из
>> удачных вариантов является вариант с разделением веток:
>> - master - апстрим;
>> - altlinux - наши патчи;
>> - sisyphus/p10/p9/... - ветки с gear-правилами и нашим spec-файлом.
>>
>> Если патчи не нужно отдавать и наш сборочный репозиторий никому не
>> нужен, то, всё равно, такая схема для разработки удобна, поскольку в
>> историю изменений исходников не входит ещё и история изменений нашего
>> spec-файла.
>>
>> Всё это актуально только в том случае, если:
>> - мы рассматриваем не личный репозиторий, а репозиторий формата
>> git.altlinux.org/gears/p/package-name.git, как источник для
>> взаимодействия с другими разработчиками, включая нас самих;
>> - мы исходим из того, что у нас имеется общий апстрим (ветка master),
>> который, как первоисточник, используется всеми разработчиками;
>> - задача, которую мы совместно решаем, состоит в том, чтобы
>> максимально быстро передавать наши наработки, при необходимости, в
>> апстрим или друг другу;
>> - а длительная поддержка наших наработок предполагает, чтобы эта
>> возможность не утрачивалась в угоду мнимому или объективному, но
>> весьма условному, удобству.
>>
>> Длительная поддержка наших изменений в апстримных проектах тоже, в
>> каком-то смысле - разработка. Но совершенно другая задача, если
>> взаимодействие планируется налаживать на регулярной основе. Если мы
>> ведём все исходники в git-репах, то неплохо бы его и использовать (ну,
>> хотелось бы). В том виде, в котором мы ведём сборку сейчас, во многих
>> случаях, нашими наработками воспользоваться, мягко говоря, довольно
>> затруднительно. Если мы не ставим себе такой задачи, то у меня нет
>> больше никаких вопросов.
>
>
> У нас есть по крайней мере две разные задачи: регулярная сборка ядра из единых исходников для основных платформ и разработка для апстрима и партнёров. И это две разные сферы ответственности.
> То есть задача не только техническая, но управленческая и кадровая. Тянуть одеяло на себя -- самое плохое дело в этой ситуации. Решение нужно комплексное и согласованное, простым оно не будет. Но сейчас есть время и, возможно, ресурсы, подумать над ней.

Именно с целью подумать и что-то придумать я и начал эту переписку.

Поскольку уже прошло некоторое время и были высказаны различные точки
зрения, хочу обобщить некоторые тезисы:
- в идеале нужен общий подход, подходящий для регулярной сборки ядра
из единых исходников для основных платформ и для разработки для
апстрима и партнёров;
- если это по каким-то причинам невозможно, то требуется
автоматизируемый сценарий получения исходников для разработки в
точности совпадающий с исходниками для регулярной сборки.

Честно говоря, я не очень понимаю как, и можно ли вообще, добиться
второго варианта. Кроме того, разработка для апстрима и партнёров в
равной (если не в большей) требует выявления регрессий на всех
основных платформах при разработке под какую-то одну.



> Rgrds, Алексей
>>
>>
>> PS: Кстати, у федоры тоже есть git-репы для своих пакетов:
>> - https://src.fedoraproject.org/rpms/kernel/
>> Пример, ядра не очень показателен, но даже в нём сразу понятно чем
>> отличается их ядро от апстримного. А про наше нужно ещё догадаться,
>> изучив и разобрав gear-правила, которые для каждого пакета свои. Стоит
>> отметить, что gear на другие дистрибутивы не портирован, хотя у меня
>> есть свой порт на centos.
>>
>> PPS: Понятно, что предложенная схема имеет свои издержки. Но "отрывать
>> руки" - это уже как-то за гранью. Я если отрывать руки за то, что
>> сборка есть, а патчей нет? Ну, то есть они как бы есть, но передать их
>> в апстрим требует нетривиальной работы?
>>
>>
>> --
>> Sin (Sinelnikov Evgeny)
>> _______________________________________________
>> Devel mailing list
>> Devel@lists.altlinux.org
>> https://lists.altlinux.org/mailman/listinfo/devel
>
> _______________________________________________
> Devel mailing list
> Devel@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/devel



-- 
Sin (Sinelnikov Evgeny)

  parent reply	other threads:[~2022-01-03  6:17 UTC|newest]

Thread overview: 92+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-12-03 19:41 Evgeny Sinelnikov
2021-12-03 23:44 ` Andrey Savchenko
2021-12-05 23:22   ` Evgeny Sinelnikov
2021-12-06 13:36     ` Dmitry V. Levin
2021-12-06 18:28     ` Andrey Savchenko
2021-12-06 23:07       ` Evgeny Sinelnikov
2021-12-07  7:38         ` Anton V. Boyarshinov
2021-12-06  8:29   ` Alexey Sheplyakov
2021-12-06 17:32     ` Andrey Savchenko
2021-12-06 20:08       ` Vladimir D. Seleznev
2021-12-06  7:42 ` Anton V. Boyarshinov
2021-12-06  8:09   ` Alexey Sheplyakov
2021-12-06 10:29     ` Anton V. Boyarshinov
2021-12-06 10:26   ` Alexey Sheplyakov
2021-12-06 10:31     ` Anton V. Boyarshinov
2021-12-06 10:47       ` Evgeny Sinelnikov
2021-12-06 11:03         ` Anton V. Boyarshinov
2021-12-06 10:55         ` Anton V. Boyarshinov
2021-12-06 13:25         ` Dmitry V. Levin
2021-12-06 13:35           ` Evgeny Sinelnikov
2021-12-06 13:41             ` Anton V. Boyarshinov
2021-12-06 13:48               ` Evgeny Sinelnikov
2021-12-06 13:51                 ` Anton V. Boyarshinov
2021-12-06 13:57                   ` Dmitry V. Levin
2021-12-06 14:04                   ` Evgeny Sinelnikov
2021-12-06 14:08                     ` Dmitry V. Levin
2021-12-06 14:40                     ` Anton V. Boyarshinov
2021-12-06 14:54                     ` Anton V. Boyarshinov
2021-12-06 15:02                       ` Dmitry V. Levin
2021-12-06 15:26                         ` Anton V. Boyarshinov
2021-12-06 15:53                           ` Dmitry V. Levin
2021-12-06 22:58                             ` Evgeny Sinelnikov
2021-12-07  7:47                               ` Anton V. Boyarshinov
2021-12-07 16:54                               ` Andrey Savchenko
2021-12-06 13:47             ` Dmitry V. Levin
2021-12-06 13:54               ` Evgeny Sinelnikov
2021-12-06 10:53     ` Anton V. Boyarshinov
2021-12-06 11:27       ` Alexey Sheplyakov
2021-12-06 11:41         ` Anton V. Boyarshinov
2021-12-06 12:12           ` Anton Farygin
2021-12-06 12:30             ` Alexey Sheplyakov
2021-12-06 14:53               ` Anton Farygin
2021-12-23  3:28             ` Mikhail Novosyolov
2021-12-23  6:07               ` Andrey Savchenko
2022-01-03  5:07                 ` Evgeny Sinelnikov
2022-01-03  5:33                   ` Evgeny Sinelnikov
2022-01-10  8:53                     ` Anton V. Boyarshinov
2022-01-11  8:01                     ` Anton V. Boyarshinov
2022-01-11  8:12                       ` Anton V. Boyarshinov
2022-01-11  8:55                         ` Paul Wolneykien
2022-01-11  9:10                           ` Sergey V Turchin
2022-01-11  9:33                             ` Anton Farygin
2022-01-11  9:37                               ` Alexey V. Vissarionov
2022-01-12  0:26                             ` [devel] PoC: gear-submodule-update Ex: " Vitaly Chikunov
2022-01-12  0:39                               ` Dmitry V. Levin
2022-01-12 12:20                                 ` Dmitry V. Levin
2022-01-12 12:40                                   ` Alexey Gladkov
2022-01-12 12:53                                     ` Alexey Gladkov
2022-01-12 13:10                                     ` Alexey Gladkov
2022-01-12 13:26                                       ` Dmitry V. Levin
2022-01-12 16:59                                   ` [devel] gear-store-submodules Dmitry V. Levin
2022-01-12 19:14                                     ` Alexey Gladkov
2022-01-12 20:06                                       ` Dmitry V. Levin
2022-01-12 21:14                                         ` Alexey Gladkov
2022-01-12  1:04                               ` [devel] PoC: gear-submodule-update Ex: Новая схема ведения исходников ядра Vladislav Zavjalov
2022-01-12  6:01                               ` Anton Farygin
2022-01-11 10:30                           ` [devel] " Vladimir D. Seleznev
2022-01-11 12:18                       ` Anton V. Boyarshinov
2022-01-19  9:53                         ` Vladimir D. Seleznev
2022-01-19  9:56                           ` Anton V. Boyarshinov
2022-01-19 10:15                             ` Anton V. Boyarshinov
2022-01-19 10:24                               ` Alexey V. Vissarionov
2022-01-19 10:58                                 ` Anton V. Boyarshinov
2022-01-19 10:31                             ` [devel] Автодекларация патчей в спеках (Was: Новая схема ведения исходников ядра) Vladimir D. Seleznev
2022-01-19 10:32                               ` Anton Farygin
2022-01-19 10:48                                 ` Vladimir D. Seleznev
2022-01-19 11:10                                   ` Anton Farygin
2022-01-19 10:57                               ` Anton V. Boyarshinov
2022-01-19 11:09                               ` Alexey V. Vissarionov
2022-01-19 10:52                             ` [devel] Новая схема ведения исходников ядра Anton V. Boyarshinov
2022-01-03  6:17                     ` Evgeny Sinelnikov [this message]
2022-01-03  6:25                       ` Aleksey Novodvorsky
2021-12-06 11:54       ` Alexey Sheplyakov
2021-12-06 12:35 ` Dmitry V. Levin
2021-12-06 13:29   ` Evgeny Sinelnikov
2021-12-06 13:48     ` Anton V. Boyarshinov
2021-12-08  8:07       ` [devel] Отделяя котлеты от мух (было Re: Новая схема ведения исходников ядра) Alexey Sheplyakov
2021-12-08  8:54         ` Anton V. Boyarshinov
2021-12-08 10:32           ` Alexey Sheplyakov
2021-12-06 14:31     ` [devel] Новая схема ведения исходников ядра Dmitry V. Levin
2021-12-06 14:55       ` Anton Farygin
2021-12-23  3:14       ` Mikhail Novosyolov

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=CAK42-GqE3tPEkWzVkyWpcu2pty46cmKhHv6Mbn-Z968M0qYROw@mail.gmail.com \
    --to=sin@altlinux.org \
    --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