ALT Linux Distributions development
 help / color / mirror / Atom feed
From: Mikhail Efremov <sem@altlinux.org>
To: devel-distro@lists.altlinux.org
Subject: Re: [devel-distro] installer-feature-* и rootfs
Date: Mon, 9 Dec 2019 17:19:54 +0300
Message-ID: <20191209171954.1ea5bc5e@sem.office.basealt.ru> (raw)
In-Reply-To: <5e619d0b-3842-e24a-44a2-b65f7cd95b0c@ya.ru>

On Fri, 6 Dec 2019 23:21:40 +0700 Антон Мидюков wrote:
> 06.12.2019 21:48, Антон Мидюков пишет:
> > 06.12.2019 21:21, Mikhail Efremov пишет:  
> >> On Fri, 6 Dec 2019 17:12:23 +0400 Ivan A. Melnikov wrote:
> >> [...]  
> >>> Как думаете, не стоит ли включить такой хук непосредственно
> >>> в alterator-setup, а не в отдельный пакет?  
> >> Лучше отдельный пакет, см. выше. Хуки в installer-feature-* писались
> >> для использования в инсталляторе и могут работать неправильно при таком
> >> нештатном использовании. Т.е. installer-features и
> >> rootfs-installer-features - это грязные хаки, нельзя бездумно
> >> использовать с ними произвольную installer-feature. Всегда нужно
> >> внимательно эту installer-feature прочитать и убедиться, что ее можно
> >> так использовать.  
> >
> > Я думаю, что не надо включать в собираемый rootfs installer-features, 
> > которые не предназначены для этой rootfs.

И также отдельно включать {rootfs,livecd}-installer-features явно и
только если хорошо знаешь, что делаешь. Пусть живет отдельно, иначе
может создаться впечатление, что это штатная вещь, а не грязный хак,
которым это на самом деле является.

> > Ограничить типом собираемой цели в mkimage-profiles не проблема.
> >  
> Мысль о бездумном использовании правильная. Давайте введём новую 
> категорию features-common, которые подходят и для installer, и для 
> livecd-install, и для alterator-setup. Поместим их скрипты, например, в 
> /usr/share/install-common/. Научим выполнять оттуда скрипты всех троих. 
> А затем останется постепенно выделить общие фичи, проверив, что они 
> подходят всем троим. Тогда точно никому ничего не поломаем. А в названии 
> feature укажем -common. Как идея?

Вот только проблемы хотя бы с тем, какие стадии вообще есть и когда
какие хуки должны запускаться. В alterator-setup вообще нет установки
пакетов, например. Поэтому не очевидно когда должны запускаться
preinstall-хуки. И это далеко не единственное различие.
Нужно продумать какие нужны хуки, в какой момент они должны
запускаться, какие аргументы/переменные им должны передаваться и как
они будут называться. И все это должно быть более-менее согласованно для
installer, livecd-install и alterator-setup.

> Не такая уж и большая работа, как мне кажется. Но результат будет не 
> здесь и не сейчас, конечно.

Я бы предпочел сначала хорошенько все продумать прежде чем что-то
делать. Иначе нагородим еще один жуткий набор костылей.

-- 
WBR, Mikhail Efremov


  reply	other threads:[~2019-12-09 14:19 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-12-06 13:12 Ivan A. Melnikov
2019-12-06 13:23 ` Антон Мидюков
2019-12-06 14:16 ` Alexey Shabalin
2019-12-06 14:29   ` Mikhail Efremov
2019-12-06 20:23   ` [devel-distro] installer-feature-symlinks-from-sbin Dmitry V. Levin
2019-12-08  9:45     ` Andrey Cherepanov
2019-12-08 10:16       ` Dmitry V. Levin
2019-12-06 14:21 ` [devel-distro] installer-feature-* и rootfs Mikhail Efremov
2019-12-06 14:48   ` Антон Мидюков
2019-12-06 16:21     ` Антон Мидюков
2019-12-09 14:19       ` Mikhail Efremov [this message]
2019-12-10  7:06         ` Ivan A. Melnikov
2019-12-24 16:45 ` Mikhail Efremov
2019-12-25  8:21   ` Ivan A. Melnikov

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=20191209171954.1ea5bc5e@sem.office.basealt.ru \
    --to=sem@altlinux.org \
    --cc=devel-distro@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 Distributions development

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/devel-distro/0 devel-distro/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-distro devel-distro/ http://lore.altlinux.org/devel-distro \
		devel-distro@lists.altlinux.org devel-distro@lists.altlinux.ru devel-distro@lists.altlinux.com
	public-inbox-index devel-distro

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


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