From: Alexey Gladkov <gladkov.alexey@gmail.com>
To: make-initrd@lists.altlinux.org
Subject: Re: [make-initrd] [PATCH v1 00/41] fork pipeline
Date: Sun, 26 Sep 2021 16:29:21 +0200
Message-ID: <20210926142921.qnmrkydotjtpq664@example.org> (raw)
In-Reply-To: <a7bae7b2-6210-8044-bbef-33d3babd88b2@gmail.com>
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
next prev parent reply other threads:[~2021-09-26 14:29 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-09-24 15:52 Leonid Krivoshein
2021-09-24 16:56 ` Alexey Gladkov
2021-09-24 18:42 ` Leonid Krivoshein
2021-09-24 19:06 ` Alexey Gladkov
2021-09-26 18:56 ` Leonid Krivoshein
2021-09-27 9:15 ` Alexey Gladkov
2021-09-27 13:17 ` Leonid Krivoshein
2021-09-27 13:55 ` Alexey Gladkov
2021-10-13 17:06 ` Leonid Krivoshein
2021-10-13 18:38 ` Alexey Gladkov
2021-10-13 18:48 ` Leonid Krivoshein
2021-09-24 19:08 ` Alexey Gladkov
2021-09-24 21:59 ` Leonid Krivoshein
2021-09-26 14:29 ` Alexey Gladkov [this message]
2021-09-26 20:09 ` Leonid Krivoshein
2021-09-27 9:23 ` Alexey Gladkov
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=20210926142921.qnmrkydotjtpq664@example.org \
--to=gladkov.alexey@gmail.com \
--cc=make-initrd@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
Make-initrd development discussion
This inbox may be cloned and mirrored by anyone:
git clone --mirror http://lore.altlinux.org/make-initrd/0 make-initrd/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 make-initrd make-initrd/ http://lore.altlinux.org/make-initrd \
make-initrd@lists.altlinux.org make-initrd@lists.altlinux.ru make-initrd@lists.altlinux.com
public-inbox-index make-initrd
Example config snippet for mirrors.
Newsgroup available over NNTP:
nntp://lore.altlinux.org/org.altlinux.lists.make-initrd
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git