ALT Linux Distributions development
 help / color / mirror / Atom feed
From: Alexey Shabalin <a.shabalin@gmail.com>
To: Distributions development <devel-distro@lists.altlinux.org>
Subject: Re: [devel-distro] installer-feature-* и rootfs
Date: Fri, 6 Dec 2019 17:16:07 +0300
Message-ID: <CAEdvWkR=W3RU0ZWWrxGiFBvh9BFbSB1pw5dZFh6H2hFhiX9LfQ@mail.gmail.com> (raw)
In-Reply-To: <20191206131223.tzqyuxmdc6m36z4q@titan.localdomain>

пт, 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

  parent reply	other threads:[~2019-12-06 14:16 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 [this message]
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
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='CAEdvWkR=W3RU0ZWWrxGiFBvh9BFbSB1pw5dZFh6H2hFhiX9LfQ@mail.gmail.com' \
    --to=a.shabalin@gmail.com \
    --cc=devel-distro@lists.altlinux.org \
    --cc=shaba@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