51809 2024-10-23 16:45:39 +0300 Невозможно войти из lightdm в классический gnome 2024-10-26 17:06:03 +0300 1 1 4 Development Sisyphus lightdm unstable all Linux CLOSED FIXED P5 normal --- 1 iv manowar aris golubevan liannnix manowar sin qa-sisyphus oldest_to_newest 253370 0 iv 2024-10-23 16:45:39 +0300 Раньше было можно, теперь падает. - возьмите систему с lightdm - установите пакет gnome-regular - выберете сеанс "Классический GNOME c Xorg" - войдите в систему Ожидание: запустится классический GNOME Реальность: что-то падает и снова появляется приглашение входа в систему от lightdm. 253371 1 iv 2024-10-23 16:47:00 +0300 При этом у пользователя в ~/.xsession-errors:0 появляется что-то такое: Running Xsession[13468] env GNOME_SHELL_SESSION_MODE=classic gnome-session Agent pid 13511 runwm: window manager "env gnome_shell_session_mode=classic gnome-session" not found. 253373 2 iv 2024-10-23 17:00:40 +0300 Примерная цепочка событий: Для старта сессии нужно запустить такую команду: # grep ^Exec /usr/share/xsessions/gnome-classic-xorg.desktop Exec=env GNOME_SHELL_SESSION_MODE=classic gnome-session lightdm по умолчанию пытается эту команду завернуть в Xsession: # grep ^session-wrapper /etc/lightdm/lightdm.conf session-wrapper=/etc/X11/Xsession Однако раз session-wrapper есть, вся строка из Exec попадает в один, первый аргумент session-wrapper'а (т.е. /etc/X11/Xsession), вместо того чтобы разделиться на отдельные аргументы: https://git.altlinux.org/gears/l/lightdm.git?p=lightdm.git;a=blob;f=lightdm/src/seat.c;h=09c78004f4ee72cf09be075b34f35527f71c7adf#l942 /etc/X11/Xsession (пакет xinitrc) видит, что его первый аргумент -- это не абсолютный путь к бинарнику, и передаёт его в rundm https://git.altlinux.org/gears/x/xinitrc.git?a=blob;f=xinitrc/install/etc/X11/Xsession;h=4dc829853de54fa74f77d0435539cfa2dfde23ea#l219 rundm, естественно, к такому не готов. 253374 3 iv 2024-10-23 17:06:03 +0300 > Примерная цепочка событий: [...] То есть, сейчас получается, что lightdm работает, только если в Exec указан полный путь к исполняемому файлу, причём без аргументов. Спецификация на desktop-файлы[1] разрешает указывать имена без пути (чтобы их поискали в $PATH) и аргументы через пробел. Так что если в /usr/share/xsessions лежат обычные настоящие desktop-файлы без каких-либо дополнительных ограничений, lightdm неправ в том, как он их обрабатывает. [1] https://specifications.freedesktop.org/desktop-entry-spec/latest/exec-variables.html 253382 4 manowar 2024-10-24 13:15:55 +0300 Да, как единую строку читает "Exec": https://git.altlinux.org/gears/l/lightdm.git?p=lightdm.git;a=blob;f=lightdm/src/session-config.c;h=e0bca920eea15a6afe62457abb25468645054a1f;hb=0c7668cc9d018f80cd3e3b602ae1f76f63150288#l32 Нужно сделать разбиение на аргументы командной строки, вероятно, с учётом возможного экранирования. Нет ли на этот счёт известного готового решения? 253399 5 iv 2024-10-24 23:35:16 +0300 (In reply to manowar@altlinux.org from comment #4) > Нужно сделать разбиение на аргументы командной строки, вероятно, с учётом > возможного экранирования. Нет ли на этот счёт известного готового решения? В том же get_session_argv чуть ниже используется g_shell_parse_argv из glib. На первый взгляд оно делает всё что нужно. 253414 6 manowar 2024-10-25 13:13:59 +0300 Спасибо, и правда. Осталось понять тогда, почему в lightdm в случае, когда есть обёртка, всё укладывается в один аргумент. То есть, как будто бы специально это сделано. 253422 7 manowar 2024-10-25 14:35:06 +0300 Да, я нашёл комит специально отменяющий правильное, на наш взгляд, поведение: https://git.altlinux.org/people/manowar/packages/?p=lightdm.git;a=commitdiff;h=964619ba615cd6c57070ad59cdcca8f132d9dfb1 . 253423 8 manowar 2024-10-25 15:03:46 +0300 https://github.com/canonical/lightdm/issues/381 253439 9 manowar 2024-10-25 16:46:07 +0300 Ну как бы вот: это что-то убунто-специфическое и старое: --- a/tests/src/lightdm-session +++ b/tests/src/lightdm-session @@ -1,5 +1,4 @@ #!/bin/sh -# Run the session expecting the full command to be the first argument. It has -# to be like this for backwards compatibility instead of having a normal argument list -exec $1 +# Run the session expecting an argument list. +exec "$@" Но мы вполне можем сделать по своему, мне кажется. 253440 10 manowar 2024-10-25 16:59:49 +0300 Сделал: https://git.altlinux.org/people/manowar/packages/?p=lightdm.git;a=commitdiff;h=1055b1486093458dbaf2dedc53f267597479e1ee Но не особо полегчало-то. :) Теперь: runwm: window manager "env" not found. Оно и понятно, раз > Exec=env GNOME_SHELL_SESSION_MODE=classic gnome-session и > /etc/X11/Xsession (пакет xinitrc) видит, что его первый > аргумент -- это не абсолютный путь к бинарнику, и передаёт > его в rundm 253445 11 iv 2024-10-25 17:08:52 +0300 > runwm: window manager "env" not found. А разве "env" не должен привратиться в /usr/bin/env? То есть, разве g_find_program_in_path (argv[0]) не нужен и в ветке if-a c session-wrapper? 253446 12 manowar 2024-10-25 17:21:08 +0300 Так он там есть: + if (session_wrapper) + { + path = g_find_program_in_path (session_wrapper); + prepend_argv (&argv, path ? path : session_wrapper); + } 253448 13 manowar 2024-10-25 17:37:59 +0300 А, не сразу понял твою мысль, извини. :) 253461 14 repository-robot 2024-10-25 19:28:10 +0300 lightdm-1.32.0-alt9 -> sisyphus: Fri Oct 25 2024 Paul Wolneykien <manowar@altlinux> 1.32.0-alt9 - Fix: Invoke session wrapper with splitted arguments (closes: 51809). 253472 15 iv 2024-10-26 17:06:03 +0300 Спасибо!