Make-initrd development discussion
 help / color / mirror / Atom feed
From: Leonid Krivoshein <klark.devel@gmail.com>
To: make-initrd@lists.altlinux.org
Subject: Re: [make-initrd] [PATCH v1 01/41] fork pipeline: split into 4 features (move files)
Date: Fri, 24 Sep 2021 22:03:28 +0300
Message-ID: <861d869c-88f1-30ec-3292-de391c0db8f2@gmail.com> (raw)
In-Reply-To: <20210924183443.kpsuna5f4pglmktc@example.org>


24.09.2021 21:34, Alexey Gladkov пишет:
> [...]
> Напомни, а зачем ты всё попилил на разные фичи ? Это кстати, не плохо бы в
> commit message занести.

Тут два момента. В форкнутом bootcahin это было сначала монолитной 
фичей, но потом я разделил на разные, чтобы минимизировать наполнение 
initramfs. Есть ядро (core), а есть методы загрузки. Например, getimage 
могла бы использовать внешний wget, я использую curl -- его точно не 
нужно во всех случаях, и его нет в busybox, кому-то нужен mount.cifs, 
ss, resolve, итд. По этому же принципу фичи делились на подпакеты... до 
сего дня.

Но при возврате в апстрим я понял, что pipeline это "фича из коробки". 
Чтобы не ломать ничего тем, у кого стоит pipeline, пришлось и эти три 
сделать частью самого make-initrd, а не make-initrd-чего-то-там. 
Варианты мы обсуждали, в частности, я предлагал сделать конфликт с фичей 
pipeline (он и был изначально), чтобы фичи друг-другу ничего не портили. 
Но ты сказал, что нет смысла оставлять pipeline, если bootchain его 
полностью заменяет. Я всё же сделал зависимость для фичи pipeline от 
bootchain'овских кусочков, но ещё не проверял, что будет, если по 
старинке поставить pipeline -- по идее разлом твои авто-тесты должны 
выловить.


-- 
Best regards,
Leonid Krivoshein.



  reply	other threads:[~2021-09-24 19:03 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-09-24 15:54 Leonid Krivoshein
2021-09-24 18:34 ` Alexey Gladkov
2021-09-24 19:03   ` Leonid Krivoshein [this message]
2021-09-26 18:52   ` Leonid Krivoshein

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=861d869c-88f1-30ec-3292-de391c0db8f2@gmail.com \
    --to=klark.devel@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