Summary: | silent bootsplash support | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | Sisyphus | Reporter: | Anton Farygin <rider> | ||||||
Component: | service | Assignee: | placeholder <placeholder> | ||||||
Status: | CLOSED FIXED | QA Contact: | qa-sisyphus | ||||||
Severity: | normal | ||||||||
Priority: | P2 | CC: | glebfm, ldv, legion, placeholder, shrek | ||||||
Version: | unstable | ||||||||
Hardware: | all | ||||||||
OS: | Linux | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 6274 | ||||||||
Attachments: |
|
Description
Anton Farygin
2005-03-17 18:14:17 MSK
Created attachment 773 [details]
service-0.5.7-bootsplash-silent.patch
Этот патч добавляет код, написанный в другом стиле. На то, чтобы интегрировать его в /etc/init.d/functions, у меня уйдёт некоторое время. В другом стиле - это ещё мягко сказано. хорошо, у нас еще есть некоторое время на то, что бы ждать. А может добавить а потом стиль менять ? Мне важно, чтобы поведение системы в случае неиспользования bootsplash не менялось совсем. Поэтому патч надо доработать. Я не могу разрешить экспортирование такой кучи переменных: export nbservices res theme progress_enable progress text_x text_y text_color text_size splash_rc LOGO_CONSOLE PROMPT Пожалуйста, попробуй: 1. уменьшить этот список 2. переименовать те переменные, которые экспортировать необходимо Очень большая просьба переименовать /sbin/splash.sh во что-нибудь более благозвучное, пока мы не завязались на использование этого имени. Кстати, тот код, который ты предлагаешь добавить в init.d/functions, очень сильно напоминает код в splash.sh Обычно во избежание коллизий дублирования общий код выносят в отдельный файл. (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. А во что переименовать ? Добавить перед переменной префикс ? Created attachment 775 [details]
Proposed change
Чтобы внедрение bootsplash не затягивалось и остальная система не страдала,
предлагаю поступить следующим образом:
Я добавляю минимально необходимую поддержку bootsplash в пакете service таким
образом, чтобы её можно было по мере необходимости дорабатывать в пакете
bootsplash.
Изменения в пакете startup при этом можно будет приложить с минимальными
изменениями.
Добро. Давай так и сделаем. В 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 быть не только программой, но и функцией. Если это так, напишите. |