* Re: [devel-distro] installer-feature-* и rootfs
2019-12-06 13:12 [devel-distro] installer-feature-* и rootfs Ivan A. Melnikov
@ 2019-12-06 13:23 ` Антон Мидюков
2019-12-06 14:16 ` Alexey Shabalin
` (2 subsequent siblings)
3 siblings, 0 replies; 14+ messages in thread
From: Антон Мидюков @ 2019-12-06 13:23 UTC (permalink / raw)
To: devel-distro
06.12.2019 20:12, Ivan A. Melnikov пишет:
> [...]
> Как думаете, не стоит ли включить такой хук непосредственно
> в alterator-setup, а не в отдельный пакет?
>
Я думаю, что стоит включить прямо в alterator-setup.
--
С уважением, Антон Мидюков <antohami@altlinux.org>
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [devel-distro] installer-feature-* и rootfs
2019-12-06 13:12 [devel-distro] installer-feature-* и rootfs 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-06 14:21 ` [devel-distro] installer-feature-* и rootfs Mikhail Efremov
2019-12-24 16:45 ` Mikhail Efremov
3 siblings, 2 replies; 14+ messages in thread
From: Alexey Shabalin @ 2019-12-06 14:16 UTC (permalink / raw)
To: Distributions development
пт, 6 дек. 2019 г. в 16:12, Ivan A. Melnikov <iv@altlinux.org>:
>
> Привет.
>
> Как известно, для одноплатников вроде RPi (и завёрнутых в коробку
> одноплатников вроде Таволги) у нас система поставляется в виде
> образа rootfs и ставится без помощи installer'а.
>
> Это означает, что на таких системах не отрабатывают пакеты
> installer-feature-*. Мы с jqt4@ посмотрели, что же мы теряем.
> Вот интересные особенности simply, сделанные через
> installer-feature-* и соответсвенно *не* присутсвующие
> в сборках под, например, RPi4:
>
> - installer-feature-bell-off-stage3 -- выключает bell в X (xset -b,
> сам всегда так делал)
> - installer-feature-lightdm-stage3 -- настраивает lightdm gtk
> greeter (появляется индикатор раскладки клавиатуры) и делает первого
> пользователя выбранным после первой загрузки (маст хев,
> в alt-workstation-mipsel я делал костыль вместо этой фичи)
> - installer-feature-sudo-enable-by-default-stage3 -- sudo по
> умолчанию для всех в группе wheel, "как в Ubuntu";
Ага, мы красавчики. Типа в пакете мы такое не делаем - нехорошо, но
все дистрибутивы выпускаем с этой фичей, что бы было удобно. Внесите
уже это в пакет sudo по-умолчанию и не извращайтесь с installer.
> - installer-feature-symlinks-from-sbin -- симлинки для ip,
> sysctl, chkconfig, service, blkid, resolvconf, lsusb, lsof
> в $PATH простого пользователя.
Жесть какая. Я уже высказывал свое отношение cas@'у, что так делать
категорически нельзя - эти симлинки не принадлежат никакому пакету.
Сделайте честный пакет, типа root-utils-user и упакуйте туда эти
симлинки(что бы можно было это легко удалить). Но я бы вообще подверг
право на жизнь такого пакета(тем более installer-featute)
>
> Понятно, что эти фичи хотелось бы видеть на всех платформах,
> под которые мы выпускаем simply.
>
> Чтобы не перепиливать всё сразу, предлагается для начала
> сделать что-нибудь для desktop-like сборок, для первоначальной
> настройки которых используется alterator-setup. У него
> есть специальные хуки (setup-postinstall.d), так что можно
> можно после его работы запустить скрипты из installer-feature-*,
> чтобы они донастроили систему.
>
> Для начала, я реализовал такой хук в отдельном пакете:
>
> http://git.altlinux.org/people/iv/packages/?p=rootfs-installer-features.git
>
> Он пока проходит обкатку в тестовой таске #242450, первое
> тестирование продемонстрировало его работоспособность.
>
> Поскольку до совсем недавнего времени я с installer-feature-*
> не сталкивался, хотелось бы услышать от людей, которые в этом
> понимают (если такие есть) какие-то мысли по этому поводу
> и, возможно, code review.
>
> Как думаете, не стоит ли включить такой хук непосредственно
> в alterator-setup, а не в отдельный пакет?
Моё мнение, что все конфиги и настройки лучше упаковывать в пакеты,
все остальное - это неконтролируемые изменения системы. Потом никогда
не найдешь, кто же тебе так "помог".
Из недавнего в телеграм-канале:
Человек собрал rpm-пакет в системе(без hasher, а почему нет если пакет
простенький?). У него появилась зависимость на /bin/service, которую
не предоставляет ни один пакет. Симлинк /bin/service появился
благодаря installer-feature-symlinks-from-sbin.
Руки бы оторвал таким помощничкам.
Итог: старайтесь не создавать проблем пользователям из лучших
побуждений. Ну не зачем запускать service простому пользователю.
А то что su- минус не используют - допинайте уже мантейнера, что бы
этот минус был по-умолчанию. Никто не может объяснить, зачем нужен su
без минуса.
--
Alexey Shabalin
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [devel-distro] installer-feature-* и rootfs
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
1 sibling, 0 replies; 14+ messages in thread
From: Mikhail Efremov @ 2019-12-06 14:29 UTC (permalink / raw)
To: devel-distro
On Fri, 6 Dec 2019 17:16:07 +0300 Alexey Shabalin wrote:
> пт, 6 дек. 2019 г. в 16:12, Ivan A. Melnikov <iv@altlinux.org>:
> >
> > Привет.
> >
> > Как известно, для одноплатников вроде RPi (и завёрнутых в коробку
> > одноплатников вроде Таволги) у нас система поставляется в виде
> > образа rootfs и ставится без помощи installer'а.
> >
> > Это означает, что на таких системах не отрабатывают пакеты
> > installer-feature-*. Мы с jqt4@ посмотрели, что же мы теряем.
> > Вот интересные особенности simply, сделанные через
> > installer-feature-* и соответсвенно *не* присутсвующие
> > в сборках под, например, RPi4:
> >
> > - installer-feature-bell-off-stage3 -- выключает bell в X (xset -b,
> > сам всегда так делал)
> > - installer-feature-lightdm-stage3 -- настраивает lightdm gtk
> > greeter (появляется индикатор раскладки клавиатуры) и делает первого
> > пользователя выбранным после первой загрузки (маст хев,
> > в alt-workstation-mipsel я делал костыль вместо этой фичи)
> > - installer-feature-sudo-enable-by-default-stage3 -- sudo по
> > умолчанию для всех в группе wheel, "как в Ubuntu";
>
> Ага, мы красавчики. Типа в пакете мы такое не делаем - нехорошо, но
> все дистрибутивы выпускаем с этой фичей, что бы было удобно. Внесите
> уже это в пакет sudo по-умолчанию и не извращайтесь с installer.
Надеюсь, что не все. Раньше это вообще было только в Simply.
Не надо такое в пакет.
> > - installer-feature-symlinks-from-sbin -- симлинки для ip,
> > sysctl, chkconfig, service, blkid, resolvconf, lsusb, lsof
> > в $PATH простого пользователя.
>
> Жесть какая. Я уже высказывал свое отношение cas@'у, что так делать
> категорически нельзя - эти симлинки не принадлежат никакому пакету.
> Сделайте честный пакет, типа root-utils-user и упакуйте туда эти
> симлинки(что бы можно было это легко удалить). Но я бы вообще подверг
> право на жизнь такого пакета(тем более installer-featute)
Да, лучше так.
> Моё мнение, что все конфиги и настройки лучше упаковывать в пакеты,
> все остальное - это неконтролируемые изменения системы. Потом никогда
> не найдешь, кто же тебе так "помог".
Нельзя упаковать в пакеты все изменения, которые бывает нужно сделать в
конкретном дистрибутиве. Инсталлер - это набор хаков, всегда так было.
--
WBR, Mikhail Efremov
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [devel-distro] installer-feature-symlinks-from-sbin
2019-12-06 14:16 ` Alexey Shabalin
2019-12-06 14:29 ` Mikhail Efremov
@ 2019-12-06 20:23 ` Dmitry V. Levin
2019-12-08 9:45 ` Andrey Cherepanov
1 sibling, 1 reply; 14+ messages in thread
From: Dmitry V. Levin @ 2019-12-06 20:23 UTC (permalink / raw)
To: devel-distro
[-- Attachment #1: Type: text/plain, Size: 730 bytes --]
On Fri, Dec 06, 2019 at 05:16:07PM +0300, Alexey Shabalin wrote:
> пт, 6 дек. 2019 г. в 16:12, Ivan A. Melnikov <iv@altlinux.org>:
[...]
> > - installer-feature-symlinks-from-sbin -- симлинки для ip,
> > sysctl, chkconfig, service, blkid, resolvconf, lsusb, lsof
> > в $PATH простого пользователя.
>
> Жесть какая. Я уже высказывал свое отношение cas@'у, что так делать
> категорически нельзя - эти симлинки не принадлежат никакому пакету.
> Сделайте честный пакет, типа root-utils-user и упакуйте туда эти
> симлинки(что бы можно было это легко удалить). Но я бы вообще подверг
> право на жизнь такого пакета(тем более installer-featute)
Удалить installer-feature-symlinks-from-sbin немедленно!
--
ldv
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [devel-distro] installer-feature-symlinks-from-sbin
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
0 siblings, 1 reply; 14+ messages in thread
From: Andrey Cherepanov @ 2019-12-08 9:45 UTC (permalink / raw)
To: Distributions development, Dmitry V. Levin, devel-distro
6 декабря 2019 г. 23:23:35 GMT+03:00, "Dmitry V. Levin" <ldv@altlinux.org> пишет:
>On Fri, Dec 06, 2019 at 05:16:07PM +0300, Alexey Shabalin wrote:
>> пт, 6 дек. 2019 г. в 16:12, Ivan A. Melnikov <iv@altlinux.org>:
>[...]
>> > - installer-feature-symlinks-from-sbin -- симлинки для ip,
>> > sysctl, chkconfig, service, blkid, resolvconf, lsusb, lsof
>> > в $PATH простого пользователя.
>>
>> Жесть какая. Я уже высказывал свое отношение cas@'у, что так делать
>> категорически нельзя - эти симлинки не принадлежат никакому пакету.
>> Сделайте честный пакет, типа root-utils-user и упакуйте туда эти
>> симлинки(что бы можно было это легко удалить). Но я бы вообще подверг
>> право на жизнь такого пакета(тем более installer-featute)
>
>Удалить installer-feature-symlinks-from-sbin немедленно!
Не вопрос. Перенесите симлинки в пакеты и он станет просто не нужен. Делался-то этот пакет, чтобы не ждать годами некоторых мейнтейнеров.
--
Andrey Cherepanov
cas@altlinux.org
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [devel-distro] installer-feature-symlinks-from-sbin
2019-12-08 9:45 ` Andrey Cherepanov
@ 2019-12-08 10:16 ` Dmitry V. Levin
0 siblings, 0 replies; 14+ messages in thread
From: Dmitry V. Levin @ 2019-12-08 10:16 UTC (permalink / raw)
To: Andrey Cherepanov; +Cc: Distributions development
[-- Attachment #1: Type: text/plain, Size: 1215 bytes --]
On Sun, Dec 08, 2019 at 12:45:05PM +0300, Andrey Cherepanov wrote:
> 6 декабря 2019 г. 23:23:35 GMT+03:00, "Dmitry V. Levin" <ldv@altlinux.org> пишет:
> >On Fri, Dec 06, 2019 at 05:16:07PM +0300, Alexey Shabalin wrote:
> >> пт, 6 дек. 2019 г. в 16:12, Ivan A. Melnikov <iv@altlinux.org>:
> >[...]
> >> > - installer-feature-symlinks-from-sbin -- симлинки для ip,
> >> > sysctl, chkconfig, service, blkid, resolvconf, lsusb, lsof
> >> > в $PATH простого пользователя.
> >>
> >> Жесть какая. Я уже высказывал свое отношение cas@'у, что так делать
> >> категорически нельзя - эти симлинки не принадлежат никакому пакету.
> >> Сделайте честный пакет, типа root-utils-user и упакуйте туда эти
> >> симлинки(что бы можно было это легко удалить). Но я бы вообще подверг
> >> право на жизнь такого пакета(тем более installer-featute)
> >
> >Удалить installer-feature-symlinks-from-sbin немедленно!
>
> Не вопрос. Перенесите симлинки в пакеты и он станет просто не нужен. Делался-то этот пакет, чтобы не ждать годами некоторых мейнтейнеров.
Некоторые из этих симлинков бессмысленные, их не должно быть вообще.
Остальные переносите сами, вы такие же мантейнеры, как и все остальные.
--
ldv
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [devel-distro] installer-feature-* и rootfs
2019-12-06 13:12 [devel-distro] installer-feature-* и rootfs Ivan A. Melnikov
2019-12-06 13:23 ` Антон Мидюков
2019-12-06 14:16 ` Alexey Shabalin
@ 2019-12-06 14:21 ` Mikhail Efremov
2019-12-06 14:48 ` Антон Мидюков
2019-12-24 16:45 ` Mikhail Efremov
3 siblings, 1 reply; 14+ messages in thread
From: Mikhail Efremov @ 2019-12-06 14:21 UTC (permalink / raw)
To: devel-distro
On Fri, 6 Dec 2019 17:12:23 +0400 Ivan A. Melnikov wrote:
> Привет.
>
> Как известно, для одноплатников вроде RPi (и завёрнутых в коробку
> одноплатников вроде Таволги) у нас система поставляется в виде
> образа rootfs и ставится без помощи installer'а.
>
> Это означает, что на таких системах не отрабатывают пакеты
> installer-feature-*. Мы с jqt4@ посмотрели, что же мы теряем.
> Вот интересные особенности simply, сделанные через
> installer-feature-* и соответсвенно *не* присутсвующие
> в сборках под, например, RPi4:
>
> - installer-feature-bell-off-stage3 -- выключает bell в X (xset -b,
> сам всегда так делал)
> - installer-feature-lightdm-stage3 -- настраивает lightdm gtk
> greeter (появляется индикатор раскладки клавиатуры) и делает первого
> пользователя выбранным после первой загрузки (маст хев,
> в alt-workstation-mipsel я делал костыль вместо этой фичи)
> - installer-feature-sudo-enable-by-default-stage3 -- sudo по
> умолчанию для всех в группе wheel, "как в Ubuntu";
> - installer-feature-symlinks-from-sbin -- симлинки для ip,
> sysctl, chkconfig, service, blkid, resolvconf, lsusb, lsof
> в $PATH простого пользователя.
JFYI, https://bugzilla.altlinux.org/37503
Я думаю заменить installer-feature-symlinks-from-sbin на пакет.
> Понятно, что эти фичи хотелось бы видеть на всех платформах,
> под которые мы выпускаем simply.
Я собирался посмотреть в alterator-setup на этот счет еще при выпуске
workstation, но так и не добрался.
> Чтобы не перепиливать всё сразу, предлагается для начала
> сделать что-нибудь для desktop-like сборок, для первоначальной
> настройки которых используется alterator-setup. У него
> есть специальные хуки (setup-postinstall.d), так что можно
> можно после его работы запустить скрипты из installer-feature-*,
> чтобы они донастроили систему.
>
> Для начала, я реализовал такой хук в отдельном пакете:
>
> http://git.altlinux.org/people/iv/packages/?p=rootfs-installer-features.git
Да, я как раз думал про такой вариант как самый простой.
Когда-то я делал подобное для alterator-livecd, см.
http://git.altlinux.org/people/sem/packages/?p=livecd-installer-features.git
Вообще надо бы сделать единые хуки для installer, livecd и setup, но
это много работы...
> Он пока проходит обкатку в тестовой таске #242450, первое
> тестирование продемонстрировало его работоспособность.
>
> Поскольку до совсем недавнего времени я с installer-feature-*
> не сталкивался, хотелось бы услышать от людей, которые в этом
> понимают (если такие есть) какие-то мысли по этому поводу
> и, возможно, code review.
Я посмотрел, помимо хуков initinstall и postinstall есть еще preinstall.
В исталляторе они запускаются так:
initinstall - в самом начале, при запуске инсталлятора.
preinstall - после установки пакетов.
postinstall - в самом конце, после всех шагов исталлятора.
Можно ли все эти хуки запускать просто в setup-postinstall.d в общем
случае неизвестно, зависит от того, что делает конкретная
installer-feature. Например, preinstall-хук может проводить какую-то
подготовительную работу для последующих шагов инсталлятора.
> Как думаете, не стоит ли включить такой хук непосредственно
> в alterator-setup, а не в отдельный пакет?
Лучше отдельный пакет, см. выше. Хуки в installer-feature-* писались
для использования в инсталляторе и могут работать неправильно при таком
нештатном использовании. Т.е. installer-features и
rootfs-installer-features - это грязные хаки, нельзя бездумно
использовать с ними произвольную installer-feature. Всегда нужно
внимательно эту installer-feature прочитать и убедиться, что ее можно
так использовать.
--
WBR, Mikhail Efremov
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [devel-distro] installer-feature-* и rootfs
2019-12-06 14:21 ` [devel-distro] installer-feature-* и rootfs Mikhail Efremov
@ 2019-12-06 14:48 ` Антон Мидюков
2019-12-06 16:21 ` Антон Мидюков
0 siblings, 1 reply; 14+ messages in thread
From: Антон Мидюков @ 2019-12-06 14:48 UTC (permalink / raw)
To: devel-distro
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.
Ограничить типом собираемой цели в mkimage-profiles не проблема.
--
С уважением, Антон Мидюков <antohami@altlinux.org>
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [devel-distro] installer-feature-* и rootfs
2019-12-06 14:48 ` Антон Мидюков
@ 2019-12-06 16:21 ` Антон Мидюков
2019-12-09 14:19 ` Mikhail Efremov
0 siblings, 1 reply; 14+ messages in thread
From: Антон Мидюков @ 2019-12-06 16:21 UTC (permalink / raw)
To: devel-distro
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.
>
> Ограничить типом собираемой цели в mkimage-profiles не проблема.
>
Мысль о бездумном использовании правильная. Давайте введём новую
категорию features-common, которые подходят и для installer, и для
livecd-install, и для alterator-setup. Поместим их скрипты, например, в
/usr/share/install-common/. Научим выполнять оттуда скрипты всех троих.
А затем останется постепенно выделить общие фичи, проверив, что они
подходят всем троим. Тогда точно никому ничего не поломаем. А в названии
feature укажем -common. Как идея?
Не такая уж и большая работа, как мне кажется. Но результат будет не
здесь и не сейчас, конечно.
--
С уважением, Антон Мидюков <antohami@altlinux.org>
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [devel-distro] installer-feature-* и rootfs
2019-12-06 16:21 ` Антон Мидюков
@ 2019-12-09 14:19 ` Mikhail Efremov
2019-12-10 7:06 ` Ivan A. Melnikov
0 siblings, 1 reply; 14+ messages in thread
From: Mikhail Efremov @ 2019-12-09 14:19 UTC (permalink / raw)
To: devel-distro
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
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [devel-distro] installer-feature-* и rootfs
2019-12-09 14:19 ` Mikhail Efremov
@ 2019-12-10 7:06 ` Ivan A. Melnikov
0 siblings, 0 replies; 14+ messages in thread
From: Ivan A. Melnikov @ 2019-12-10 7:06 UTC (permalink / raw)
To: Distributions development
On Mon, Dec 09, 2019 at 05:19:54PM +0300, Mikhail Efremov wrote:
> 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.
Я просмотрел то, что используется в simply; там что-то применимое
для rootfs происходит только в postinstall. Так что я оставил запуск
только postinstall хуков и в таком виде отправлю это в Сизиф, p9
и применю.
> > Не такая уж и большая работа, как мне кажется. Но результат будет не
> > здесь и не сейчас, конечно.
>
> Я бы предпочел сначала хорошенько все продумать прежде чем что-то
> делать. Иначе нагородим еще один жуткий набор костылей.
Мне кажется, что по большому счёту существующих костылей вполне
достатночно, нужно просто ими аккуратнee воспользоваться.
Из того, что я увидел, большая часть всего очень специфична для
installer'а и не требуется или должна делаться совсем по-другому
для livecd и rootfs. Включение сервисов и настройки control логичнее
мне кажется превести на фичи m-p, там для этого есть более
логичные и переносимые механизмы (например,
installer-feature-sudo-enable-by-default или installer-feature-nfs*).
Оставшиеся немногие вещи (вроде подстановки первого пользователя
в lightdm) мне кажется можно перетащить хоть в /etc/firsttime.d
(если на она выполняется до alterator-setup, это надо в любом
случае поправить).
--
wbr,
iv m.
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [devel-distro] installer-feature-* и rootfs
2019-12-06 13:12 [devel-distro] installer-feature-* и rootfs Ivan A. Melnikov
` (2 preceding siblings ...)
2019-12-06 14:21 ` [devel-distro] installer-feature-* и rootfs Mikhail Efremov
@ 2019-12-24 16:45 ` Mikhail Efremov
2019-12-25 8:21 ` Ivan A. Melnikov
3 siblings, 1 reply; 14+ messages in thread
From: Mikhail Efremov @ 2019-12-24 16:45 UTC (permalink / raw)
To: devel-distro
On Fri, 6 Dec 2019 17:12:23 +0400 Ivan A. Melnikov wrote:
>
> Для начала, я реализовал такой хук в отдельном пакете:
>
> http://git.altlinux.org/people/iv/packages/?p=rootfs-installer-features.git
Может пора его в p9? Я бы тоже хотел его использовать на aarch64.
--
WBR, Mikhail Efremov
^ permalink raw reply [flat|nested] 14+ messages in thread