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
next prev parent 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