Bug 6275 - silent bootsplash support
: silent bootsplash support
Status: CLOSED FIXED
: Sisyphus
(All bugs in Sisyphus/service)
: unstable
: all Linux
: P2 normal
Assigned To:
:
:
:
:
: 6274
  Show dependency tree
 
Reported: 2005-03-17 18:14 by
Modified: 2005-07-13 15:47 (History)


Attachments
service-0.5.7-bootsplash-silent.patch (2.22 KB, patch)
2005-03-17 18:14, Anton Farygin
no flags Details | Diff
Proposed change (1.08 KB, patch)
2005-03-21 17:47, Dmitry V. Levin
no flags Details | Diff


Note

You need to log in before you can comment on or make changes to this bug.


Description From 2005-03-17 18:14:17
Необходимо добавить функции для поддержки silent режима bootsplash'а.
------- Comment #1 From 2005-03-17 18:14:36 -------
Created an attachment (id=773) [details]
service-0.5.7-bootsplash-silent.patch
------- Comment #2 From 2005-03-17 18:41:13 -------
А #6270 тут при чём?
------- Comment #3 From 2005-03-17 19:09:16 -------
Упс.. промахнулся.. конечно #6274
------- Comment #4 From 2005-03-18 15:40:11 -------
Этот патч добавляет код, написанный в другом стиле.
На то, чтобы интегрировать его в /etc/init.d/functions, у меня уйдёт некоторое
время.
------- Comment #5 From 2005-03-18 15:48:42 -------
В другом стиле - это ещё мягко сказано.
------- Comment #6 From 2005-03-18 19:54:53 -------
хорошо, у нас еще есть некоторое время на то, что бы ждать.
А может добавить а потом стиль менять ?
------- Comment #7 From 2005-03-18 21:11:06 -------
Мне важно, чтобы поведение системы в случае неиспользования bootsplash не
менялось совсем.  Поэтому патч надо доработать.
------- Comment #8 From 2005-03-20 20:56:59 -------
Я не могу разрешить экспортирование такой кучи переменных:
export nbservices res theme progress_enable progress text_x text_y text_color
text_size splash_rc LOGO_CONSOLE PROMPT

Пожалуйста, попробуй:
1. уменьшить этот список
2. переименовать те переменные, которые экспортировать необходимо
------- Comment #9 From 2005-03-20 20:59:49 -------
Очень большая просьба переименовать /sbin/splash.sh
во что-нибудь более благозвучное, пока мы не завязались
на использование этого имени.
------- Comment #10 From 2005-03-21 02:43:10 -------
Кстати, тот код, который ты предлагаешь добавить в init.d/functions,
очень сильно напоминает код в splash.sh
Обычно во избежание коллизий дублирования общий код выносят в отдельный файл.
------- Comment #11 From 2005-03-21 16:33:48 -------
(In reply to comment #8)
> Я не могу разрешить экспортирование такой кучи переменных:
> export nbservices res theme progress_enable progress text_x text_y text_color
> text_size splash_rc LOGO_CONSOLE PROMPT
> 
> Пожалуйста, попробуй:
> 1. уменьшить этот список
> 2. переименовать те переменные, которые экспортировать необходимо

Уменьшить попробую... последняя переменная (PROMPT) экспортируется для
выключения getkey.

А во что переименовать ?
Добавить перед переменной префикс ?
------- Comment #12 From 2005-03-21 17:47:48 -------
Created an attachment (id=775) [details]
Proposed change

Чтобы внедрение bootsplash не затягивалось и остальная система не страдала,
предлагаю поступить следующим образом:
Я добавляю минимально необходимую поддержку bootsplash в пакете service таким
образом, чтобы её можно было по мере необходимости дорабатывать в пакете
bootsplash.
Изменения в пакете startup при этом можно будет приложить с минимальными
изменениями.
------- Comment #13 From 2005-03-21 18:36:17 -------
Добро. Давай так и сделаем.
------- Comment #14 From 2005-03-22 14:50:28 -------
В service-0.5.8-alt1 я добавил 2 новые функции, splash_init и splash_update, с
той же семантикой вызова, что и у предложенных initsplash/rc_splash.

splash_init(), помимо базовых проверок и загрузки /etc/sysconfig/bootsplash,
загружает файл с определением функции(й) bootsplash.
По умолчанию это файл /etc/init.d/functions-bootsplash, по окончании загрузки
этого файла должна быть определена переменная splash_init (эксаортировать её не
надо), содержащая имя функции, которую надо вызвать по окончании splash_init().
Функцию инициализации можно назвать bootsplash_init, если но лучшее не хватит
фантазии.  По окончании работы этой функции должна быть определена переменная
splash_exe (эксаортировать её тоже не надо), содержащая имя программы, которую
будет запускать splash_update().
У меня только что возникла мысль, что имеет смысл разрешить $splash_exe быть не
только программой, но и функцией.  Если это так, напишите.