From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gladkov.alexey@gmail.com>
Date: Sun, 26 Sep 2021 16:29:21 +0200
From: Alexey Gladkov <gladkov.alexey@gmail.com>
To: make-initrd@lists.altlinux.org
Message-ID: <20210926142921.qnmrkydotjtpq664@example.org>
References: <a9409f0a-207c-baa2-e6e8-8dbb850d5f87@gmail.com>
 <20210924190822.hkqkjmcsj3u3zazv@example.org>
 <a7bae7b2-6210-8044-bbef-33d3babd88b2@gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <a7bae7b2-6210-8044-bbef-33d3babd88b2@gmail.com>
Subject: Re: [make-initrd] [PATCH v1 00/41] fork pipeline
X-BeenThere: make-initrd@lists.altlinux.org
X-Mailman-Version: 2.1.12
Precedence: list
Reply-To: make-initrd@lists.altlinux.org
List-Id: <make-initrd.lists.altlinux.org>
List-Unsubscribe: <https://lists.altlinux.org/mailman/options/make-initrd>,
 <mailto:make-initrd-request@lists.altlinux.org?subject=unsubscribe>
List-Archive: <http://lists.altlinux.org/pipermail/make-initrd>
List-Post: <mailto:make-initrd@lists.altlinux.org>
List-Help: <mailto:make-initrd-request@lists.altlinux.org?subject=help>
List-Subscribe: <https://lists.altlinux.org/mailman/listinfo/make-initrd>,
 <mailto:make-initrd-request@lists.altlinux.org?subject=subscribe>
X-List-Received-Date: Sun, 26 Sep 2021 14:29:23 -0000
Archived-At: <http://lore.altlinux.org/make-initrd/20210926142921.qnmrkydotjtpq664@example.org/>
List-Archive: <http://lore.altlinux.org/make-initrd/>

On Sat, Sep 25, 2021 at 12:59:27AM +0300, Leonid Krivoshein wrote:
> 
> 24.09.2021 22:08, Alexey Gladkov пишет:
> > > На первой итерации, чтобы не флудить, сделал поменьше коммитов. Всё равно
> > > вспомнить "что за чем менялось" уже невозможно. На следующих итерациях
> > > постараюсь сделать это более понятным. Поэтому первые три коммита позволяют
> > > понять порядок перемещения, переименования и что создано нового. Заранее
> > > извиняюсь за порядок и оформление коммитов, готов исправиться.
> > Я начал читать, но так как кода очень много не успею за один день всё
> > осилить. Я буду ещё комментировать.
> 
> Да это понятно, смотри, по возможности. А я тут подумал, что bootchain не
> просто же так форкался и переделывался, это не только демон и API --
> основная часть его "клиентов" дойдёт до апстрима следом, но сейчас этого нет
> в удобном тебе виде. Может помочь черновик документации, что я выслал. Также
> прилагаю лог самой проблемной загрузки по FTP, сделанный в результате
> сегодняшней проверки отправленного в апстрим. Могу выслать такой же с
> локальной загрузкой, если надо. Где-то это может понять логику работы кода и
> подход к отладочным сообщениям. Ну и на вопросы буду рад ответить!

Я дальше смотреть пока не буду. Пожалуйста переделай патчи. Твои патчи не
разбиты логически, в них нет описания зачем делаются те или иные
изменения, в README я не увидел описаний.

Форк + rename можно сделать за один коммит. После этого уже идут
содержательные коммиты, которые приносят твой новый функционал или меняют
поведение.

Сейчас же есть пачка "fork pipeline: * modified", которые либо делают
переименование, либо за одно что-то правят.

-- 
Rgrds, legion