Bug 41495 - Сделать исполняемым /etc/profile.d/flatpak.sh
Summary: Сделать исполняемым /etc/profile.d/flatpak.sh
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: flatpak (show other bugs)
Version: unstable
Hardware: x86_64 Linux
: P5 blocker
Assignee: Yuri N. Sedunov
QA Contact: qa-sisyphus
URL:
Keywords:
: 41566 (view as bug list)
Depends on:
Blocks:
 
Reported: 2021-12-03 11:47 MSK by Andrey Cherepanov
Modified: 2021-12-16 13:21 MSK (History)
8 users (show)

See Also:


Attachments
Фото падения (114.23 KB, image/jpeg)
2021-12-03 14:19 MSK, Anton Farygin
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Andrey Cherepanov 2021-12-03 11:47:34 MSK
Так как /etc/profile запускает только исполняемые файлы из /etc/profile.d/*.sh, прошу сделать его исполняемым, чтобы переменная XDG_DATA_DIRS могла содержать нужные каталоги. Также прошу скопировать в p10.
Comment 1 Repository Robot 2021-12-03 12:25:49 MSK
flatpak-1.12.2-alt2 -> sisyphus:

 Fri Dec 03 2021 Yuri N. Sedunov <aris@altlinux> 1.12.2-alt2
 - /etc/profile.d/flatpak.sh: made executable (ALT #41495)
Comment 2 Sergey V Turchin 2021-12-03 12:43:58 MSK
А теперь вертайте взад.
Comment 3 Sergey V Turchin 2021-12-03 12:44:45 MSK
В следующий раз проверяйте, прежде, чем делать.
Comment 4 Sergey V Turchin 2021-12-03 12:49:25 MSK
/etc/profile.d/flatpak.sh сломан
Comment 5 Yuri N. Sedunov 2021-12-03 13:37:07 MSK
(Ответ для Sergey V Turchin на комментарий #3)
> В следующий раз проверяйте, прежде, чем делать.

На кедах проверять? У меня всё работает.
Comment 6 Sergey V Turchin 2021-12-03 13:45:31 MSK
> На кедах проверять?
Например.

> У меня всё работает.
Покажи $XDG_DATA_DIRS
Comment 7 Yuri N. Sedunov 2021-12-03 13:48:32 MSK
(Ответ для Sergey V Turchin на комментарий #6)
> > На кедах проверять?
> Например.
Твои кеды меня не интересуют.

> > У меня всё работает.
> Покажи $XDG_DATA_DIRS

$ echo $XDG_DATA_DIRS
/home/aris/.local/share/flatpak/exports/share:/var/lib/flatpak/exports/share:/usr/local/share/:/usr/share/
Comment 8 Sergey V Turchin 2021-12-03 14:06:32 MSK
(Ответ для Yuri N. Sedunov на комментарий #7)
> Твои кеды меня не интересуют.
А должны.
Comment 9 Yuri N. Sedunov 2021-12-03 14:12:28 MSK
(Ответ для Sergey V Turchin на комментарий #8)
> (Ответ для Yuri N. Sedunov на комментарий #7)
> > Твои кеды меня не интересуют.
> А должны.

Если твои кеды можно обрушить чем-то в $XDG_DATA_DIRS, их надо обязательно чинить.

Эта бага закрыта.
Comment 10 Anton Farygin 2021-12-03 14:17:22 MSK
после того, как этот файл я сделал исполняемым - вход в KDE падает. Это регрессия, такое недопустимо.

Надо разобраться что происходит и починить.
Comment 11 Sergey V Turchin 2021-12-03 14:18:25 MSK
(Ответ для Yuri N. Sedunov на комментарий #9)
> Если твои кеды можно обрушить чем-то в $XDG_DATA_DIRS, их надо обязательно
> чинить.
Бред. Гном так же ломается при желании.
Comment 12 Anton Farygin 2021-12-03 14:19:08 MSK
Created attachment 10023 [details]
Фото падения
Comment 13 Sergey V Turchin 2021-12-03 14:21:43 MSK
Скрипт /etc/profile.d/flatpak.sh плюёт на содержимое $XDG_DATA_DIRS, которое было до его деятельности и перекрывает своими значениями.
Он не должен _ничего_ оттуда удалять.
Comment 14 Sergey V Turchin 2021-12-03 14:22:36 MSK
(Ответ для Anton Farygin на комментарий #12)
> Фото падения
Это просто параллельно не повезло.
Comment 15 Sergey V Turchin 2021-12-03 14:31:49 MSK
Хотя, возможно, этот баг на меня.
Comment 16 Yuri N. Sedunov 2021-12-03 14:34:52 MSK
(Ответ для Sergey V Turchin на комментарий #13)
> Скрипт /etc/profile.d/flatpak.sh плюёт на содержимое $XDG_DATA_DIRS, которое
> было до его деятельности и перекрывает своими значениями.
> Он не должен _ничего_ оттуда удалять.

А, ты попробуй сделать так, чтобы kde5.sh выполнялся до flatpak.sh.
Comment 17 AEN 2021-12-03 14:37:13 MSK
Андрей, прошу Вас посмотреть что происходит и разобраться как можно исправить.
Comment 18 Олег Соловьев 2021-12-03 14:42:51 MSK
Что происходит:

В /etc/profile.d/flatpack.sh происходит следующее:

XDG_DATA_DIRS="${new_dirs:+${new_dirs}:}${XDG_DATA_DIRS:-/usr/local/share:/usr/share}"

К $XDG_DATA_DIRS что-то добавляется в самое начало.
но если переменная пуста - вместо неё подставляется дефолтное "/usr/local/share:/usr/share".

А в plasma5-workspace:
+if (!qEnvironmentVariableIsSet("XDG_DATA_DIRS")) {
+    qputenv("XDG_DATA_DIRS", "/usr/share/kf5:/usr/share");
+}

Сам plasma-workspace устанавливает переменную, только если она пуста, но после flatpack.sh она больше не пуста и потому не модифицируется.
Comment 19 Олег Соловьев 2021-12-03 14:43:36 MSK
s/пуста/объявлена/g
Comment 20 Sergey V Turchin 2021-12-03 14:46:09 MSK
(Ответ для Олег Соловьев на комментарий #18)
> Сам plasma-workspace устанавливает переменную, только если она пуста, но
> после flatpack.sh она больше не пуста и потому не модифицируется.
Да. Я до этого и догадался. Только от этого вход в систему ломаться не перестаёт.
Comment 21 AEN 2021-12-03 14:54:23 MSK
Анлрей посмотрит в понедельник.
Comment 22 Sergey V Turchin 2021-12-03 15:00:16 MSK
Уже всё посмотрели. Нужно откатить изменения flatpak до обновления plasma-workspace, т.к. в plasma-workspace уже топором нехорошо махать, поэтом не моментальное исправление.
Comment 23 AEN 2021-12-03 15:04:21 MSK
(Ответ для Sergey V Turchin на комментарий #22)
> Уже всё посмотрели. Нужно откатить изменения flatpak до обновления
> plasma-workspace, т.к. в plasma-workspace уже топором нехорошо махать,
> поэтом не моментальное исправление.

Спасибо.
Подожлем Андрея.
Comment 24 Anton Farygin 2021-12-03 15:13:52 MSK
Исправлять точно надо в kde, тут вопрос в том, что это надо исправить сегодня, что бы завтра изменение flatpack не сломало вход в систему всем пользователям KDE.
Comment 25 Sergey V Turchin 2021-12-03 15:18:14 MSK
Я ж написал, откатите пока flatpak.
Comment 26 AEN 2021-12-03 15:19:17 MSK
(Ответ для Anton Farygin на комментарий #24)
> Исправлять точно надо в kde, тут вопрос в том, что это надо исправить
> сегодня, что бы завтра изменение flatpack не сломало вход в систему всем
> пользователям KDE.

Понятно. У Андрея ВКС, он псомтрит через полчаса и отпишется. У него тоже KDE в системе.
Юрий, спасибо за оперативность и извините за накладки. Разберемся.
Comment 27 Sergey V Turchin 2021-12-03 15:29:14 MSK
(Ответ для AEN на комментарий #26)
> Андрея ВКС, он псомтрит
У него есть kde5-profile, на который посмотреть.
Comment 28 Sergey V Turchin 2021-12-03 16:07:16 MSK
А так похоже, что это изменение сломает всех, кто не стрижен под горшок.
Comment 29 Sergey V Turchin 2021-12-03 16:42:04 MSK
(Ответ для Yuri N. Sedunov на комментарий #7)
> $ echo $XDG_DATA_DIRS
> /home/aris/.local/share/flatpak/exports/share:/var/lib/flatpak/exports/share:
> /usr/local/share/:/usr/share/
Слеши в конце путей лучше убрать.
Comment 30 Andrey Cherepanov 2021-12-03 17:37:44 MSK
(Ответ для AEN на комментарий #17)
> Андрей, прошу Вас посмотреть что происходит и разобраться как можно
> исправить.

Уже давно разобрался. И починил в kde5-profile-1.3-alt1
У меня на KDE из Альт Образование 10.0rc ничего не упало.
Comment 31 Sergey V Turchin 2021-12-03 18:11:47 MSK
(Ответ для Andrey Cherepanov на комментарий #30)
> ничего не упало.
Этого не всегда достаточно для корректной работы.

Запуск KDE я починил в plasma5-workspace.git=5.23.4-alt4 build#291235
Comment 32 Sergey V Turchin 2021-12-03 18:17:44 MSK
(Ответ для Sergey V Turchin на комментарий #28)
> А так похоже, что это изменение сломает всех, кто не стрижен под горшок.

Будем считать, что исправлено.
Comment 33 Andrey Cherepanov 2021-12-03 18:39:28 MSK
(Ответ для Sergey V Turchin на комментарий #32)
> (Ответ для Sergey V Turchin на комментарий #28)
> > А так похоже, что это изменение сломает всех, кто не стрижен под горшок.
> 
> Будем считать, что исправлено.

Исправил также в kde5-profile-1.4-alt1 заполнение при пустых значениях.
Comment 34 Sergey V Turchin 2021-12-04 11:21:22 MSK
Ещё пришлось поправить скрипт из plasma5-discover-flatpak.
Наверняка всех затронет, кто меняет XDG_DATA_DIRS.
Comment 35 Sergey V Turchin 2021-12-16 13:21:18 MSK
*** Bug 41566 has been marked as a duplicate of this bug. ***