ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: Igor Vlasenko <vlasenko@imath.kiev.ua>
To: devel@lists.altlinux.org
Subject: [devel] RFC: wayland session wrapper script
Date: Thu, 2 Dec 2021 20:35:23 +0200
Message-ID: <20211202183523.GA29870@dad.imath.kiev.ua> (raw)

Уважаемые коллеги!

Как показал #40910
https://bugzilla.altlinux.org/40910
нам срочно нужен wayland session wrapper script.

Выношу на обсуждение, что туда хотелось бы запихнуть.

I/Часть один. Что хотелось бы иметь в таком скрипте.

1) Самый минимум такого скрипта - это

№!/bin/bash -login
# wayland session wrapper script.
exec "$@"

Этого минимума хватит, чтобы втянуть /etc/profile and /etc/profile.d
и закрыть #40910.
Но этого недостаточно по сравнению с нашей инфраструктурой над /etc/X11/Xsession.

Поэтому вот что еще хочу туда добавить:

2) обертки
SELINUX_WRAPPER=...
DBUS_WRAPPER=
# we already have user's dbus session or we are about to run it
[ -n "$DBUS_SESSION_BUS_ADDRESS" -o "$1" = "dbus-run-session" ] || DBUS_WRAPPER=dbus-run-session

exec $SELINUX_WRAPPER $DBUS_WRAPPER "$@"

DBUS_WRAPPER - для запуска на системах без systemd.

3) XDG_SESSION_TYPE.

принудительно устанавливать, если вдруг не установлен:
XDG_SESSION_TYPE=${XDG_SESSION_TYPE:-wayland}

То же самое, кстати, надо будет сделать и в /etc/X11/Xsession:
XDG_SESSION_TYPE=${XDG_SESSION_TYPE:-x11}

Это нужно для поддержки старых и кривых DM.

4) redirect stderr to a file

   Наверное, тоже надо.
   Куда-то, к примеру, в "$HOME/.local/share/wayland-session.log",
   имена предлагайте.

   Возможно, проверить сначала, что мы не запущены под GDM, sddm, ...
   которые сами делают stderr redirect
   и посмотреть в /proc/self/fd/2, это -f или -с

# redirect stderr to a file, if it is not yet redirected by DM
if [ ! -e /proc/self -o -c /proc/self/fd/2 ]; then
for errfile in "$HOME/.local/share/wayland-session.log"; do
	if install -m600 /dev/null "$errfile" 2>/dev/null; then
		exec &>"$errfile"
		break
	fi
done

4) <wayland>/profile.d.

Хоть мы уже втянули /etc/profile.d, но для
wayland session wrapper script нужен и свой отдельный

   <wayland>/profile.d

Смысл его существования в том, что ряд тулкитов и приложений
поддерживают wayland, но без волшебного понуждения в виде
магических переменных вида THIS_APP_PLEASE_DO_USE_WAYLAND=1
все равно по умолчанию запускаются под XWayland.
Такие костыли можно как раз в <wayland>/profile.d/ и сложить.

Кстати, проявилась тенденция переходить на использование
в качестве такого костыля
XDG_SESSION_TYPE=wayland
почему я и хочу форсить значение этой переменной в п.3.


5) <sessions>/profile.d.

Исторически сложилось, что у нас есть /etc/X11/profile.d
и в нем есть ряд скриптов.

Сейчас там 5 скриптов.

Из них только 1 X11 specific:
/etc/X11/profile.d/xapp-gtk3-module.sh

Остальные не совсем понятна логика, почему они в
/etc/X11/profile.d, а не просто в /etc/profile.d.

В случае
/etc/X11/profile.d/ssh-agent.sh
логика понятна, не хочется вызывать этот скрипт в
случае удаленного логина по ssh.
Но там вроде бы достаточно дополнительно проверить,
есть ли $SSH_CONNECTION, и с такой проверкой
можно смело переносить в /etc/profile.d
(поправьте, если не так, знающие люди!)

Что касается оставшихся скриптов
/etc/X11/profile.d/xdg-user-dirs.sh
/etc/X11/profile.d/zdg-user-dirs-install.sh
/etc/X11/profile.d/zdg-move-templates.sh
то мне не очень понятно, почему они не в /etc/profile.d.
(расскажите, знающие люди!)

Если их все же нельзя перенести в /etc/profile.d,
то придется дополнительно вводить каталог
<sessions>/profile.d,
Откуда будут читать и wayland-session,
и патченый /etc/X11/Xsession.

Если их можно перенести в /etc/profile.d, то перенести нужно,
и тогда они будут доступны и в wayland-session,
и в Xsession.

Также в /etc/X11/Xsession есть
TryXBrowser() and TryTextBrowser() stuff and
export BROWSER HELP_BROWSER

Этот кусок тоже логично выпилить и перенести в /etc/profile.d.

6. init.d
Для X есть /etc/X11/xinit.d.
Мне сначала показалось, что в 'wayland session wrapper script'
init.d не нужен. Напомню, что для xsession
DM __СНАЧАЛА__ запускает X, потом в запущенном X исполняется Xsession
и скрипты из xinit.d.

Для обертки wayland-session это не так. Во время его работы
wayland сервера еще нет и, возможно, никогда и не будет
(если, к примеру, запускается fbterm).
Поэтому иметь <wayland>/init.d бессмысленно.

Однако, анализ скриптов /etc/X11/xinit.d/ показал, что есть тема,
общая между Xsession, wayland compositor и fbterm, но не для
удаленного логина по ssh.

Это внесение пользовательских настроек в железо локальной машины,
на которой открывается сессия. Это может быть
* загрузка параметров alsa для не-pulse юзера,
* пользовательские настройки видеокарты
  #/etc/X11/xinit.d/nvidia-settings.sh     nvidia-settings
* пользовательские настройки для multiseat
  #/etc/X11/xinit.d/90-multiseat-*  setup-multiseat-*
и т. д.

Поэтому имеет смысл иметь <user-hardware>/init.d,
который запускать и из wayland-session, и из
/etc/X11/Xsession.

Прошу обсуждать, дополнять, критиковать.

P.S.

Про "II/Часть два. Что где в каком каталоге расположить и как назвать".

Специально не стал писать, так как сначала нужно определиться,
что нам нужно, а что, вроде <wayland>/init.d - нет.

В зависимости от итога, можно будет сделать структуру каталогов проще
или потребуется усложнять.

-- 

I V


             reply	other threads:[~2021-12-02 18:35 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-12-02 18:35 Igor Vlasenko [this message]
2021-12-02 19:09 ` Alexey V. Vissarionov
2021-12-03  7:30   ` Sergey V Turchin
2021-12-03 10:58     ` Igor Vlasenko
2021-12-03 10:59   ` Igor Vlasenko
2021-12-03 18:41     ` Alexey V. Vissarionov
2021-12-03 12:12 ` Dmitry V. Levin
2021-12-03 12:52   ` Igor Vlasenko
2021-12-03 16:07   ` [devel] почему ssh-agent не имеет смысла на tty1 Arseny Maslennikov
2021-12-03 16:11     ` Dmitry V. Levin
2021-12-03 13:50 ` [devel] RFC: wayland session wrapper script Alexey Gladkov
2021-12-03 14:49   ` Sergey Bolshakov
2021-12-03 15:10     ` Alexey Gladkov
2021-12-03 16:05       ` [devel] DM (RFC: wayland session wrapper script) Arseny Maslennikov
2021-12-03 18:05         ` Alexey Gladkov
2021-12-03 19:59           ` [devel] DM (was: RFC: " Arseny Maslennikov
2021-12-03 23:00             ` Alexey Gladkov
2021-12-21 13:19       ` [devel] RFC: wayland session wrapper script Alexey Gladkov
2021-12-03 15:16   ` Arseny Maslennikov
2021-12-03 15:14 ` Arseny Maslennikov
2021-12-03 18:01   ` Igor Vlasenko
2021-12-03 18:16     ` Yuri Sedunov

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=20211202183523.GA29870@dad.imath.kiev.ua \
    --to=vlasenko@imath.kiev.ua \
    --cc=devel@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 Team development discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/devel/0 devel/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 devel/ http://lore.altlinux.org/devel \
		devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru
	public-inbox-index devel

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


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