Bug 12333 - Depends on compiz-gtk and emerald
Summary: Depends on compiz-gtk and emerald
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: gnome-session (show other bugs)
Version: unstable
Hardware: all Linux
: P2 normal
Assignee: Yuri N. Sedunov
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-07-14 12:05 MSD by Sir Raorn
Modified: 2008-11-08 15:57 MSK (History)
1 user (show)

See Also:

Attachments
gnome-session/gnome-wm (2.21 KB, text/plain)
2007-07-14 15:28 MSD, Sir Raorn 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Sir Raorn 2007-07-14 12:05:08 MSD 
Жуткого вида скрипт gnome-session/gnome-wm порождает жёсткие зависимости на
compiz-gtk и emerald.
Comment 1 Alexey Rusakov 2007-07-14 13:57:45 MSD 
Э... И как же быть? gnome-wm перебирает доступное, мало того - только если
поддерживается Composite. Как можно сделать, чтобы этот список не попадал в
генерируемые зависимости?
Comment 2 Sir Raorn 2007-07-14 14:40:31 MSD 
Зависимость появляется не в момент перебора доступных WM, а в момент запуска
декоратора, если WM'ом является compiz или beryl.

Я перепишу этот скрипт и приаттачу сюда через пару часов.
Comment 3 Sir Raorn 2007-07-14 15:28:39 MSD 
Created attachment 2094 [details]
gnome-session/gnome-wm

Ну собсно убрана ересь в разборе параметров, проверки наличия WM, оторвано
использование basename.  Зависимость на декоратор должна пропасть.
Comment 4 Alexey Rusakov 2007-07-15 00:57:00 MSD 
К сути бага, как я понимаю, относится только последний исправленный кусок
(начиная с DESORATOR, который на самом деле DECORATOR). Я не уверен, что этот
кусок удастся так просто пробить в апстрим, но у нас я его всё равно добавляю
(ждите g-s 2.18.3-alt4).
Что касается остальных изменений, у меня есть вопросы перед тем как я их тоже
попробую довести до апстрима:
1. while :; и далее ненадёжен. Указание аргументов, отличных от --sm-client-id и
--default-wm приводит в ветку * и прекращает разбор, хотя следом могут быть и
валидные аргументы. Кроме этого, я не знаю, насколько переносим код,
использующий эквивалентность отсутствующего $2 и пустого $2 (ситуация с
недописанным параметром).
2. Вместо which я бы предпочёл видеть нечто более переносимое (например, type
-p). В ALT Linux, если не определять алиасов, он, конечно, работает, но даже в
man which есть довольно много предложений по алиасам для which.
Comment 5 Sir Raorn 2007-07-16 00:28:41 MSD 
1) $2 ненадёжен точно также как и оригинальный вариант.  По хорошему тут надо бы
начать использовать getopt(1).  К тому же остальные параметры больше нигде не
используются.

2) Почему-то увидев which бинарником, подумал что это что-то стандартное.  Надо
перечитать SUSv3.  Основная мысль была избавиться от того жуткого перебора $PATH.

А ещё, глядя на FIXME, хочется вынести поддержку гномовьих оконых манагеров в
отдельные конфиги, типа /etc/X11/wmsession.d.*, но это явно не в этой жизни...
Comment 6 Alexey Rusakov 2007-08-13 00:38:13 MSD 
Следите за новостями, 2.18.3-alt4 отправился в путь.
Comment 7 Sir Raorn 2008-11-07 00:13:24 MSK 
gnome-session-2.24.1-alt1 - опять то же самое.
Comment 8 Sir Raorn 2008-11-08 15:40:22 MSK 
Погорячился я, конечно, с severity...
Comment 9 Yuri N. Sedunov 2008-11-08 15:57:51 MSK 
gnome-session-2.24.1-alt2