Bug 10386 - Обращение к файлу options при использовании профилей
Summary: Обращение к файлу options при использовании профилей
Status: CLOSED NOTABUG
Alias: None
Product: Sisyphus
Classification: Development
Component: etcnet (show other bugs)
Version: unstable
Hardware: all Linux
: P2 normal
Assignee: Mikhail Efremov
QA Contact: qa-sisyphus
URL:
Keywords:
: 10186 (view as bug list)
Depends on:
Blocks:
 
Reported: 2006-12-09 17:21 MSK by Vitaly Lipatov
Modified: 2009-05-25 19:30 MSD (History)
7 users (show)

See Also:


Attachments
ifup-common patch (3.08 KB, patch)
2006-12-27 14:48 MSK, Andrew Kornilov
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Vitaly Lipatov 2006-12-09 17:21:52 MSK
В функции pickup_options имеется следующий код для подключения параметров:
# and finally source iface options
for of in $MYIFACEDIR/options $MYIFACEDIR/options.d/*; do
      [ -f "$of" -a "${of%.rpm*}" = "$of" -a "${of%\~}" = "$of" ] || continue
      SourceIfNotEmpty `profiled_filename $of`
done

Этот код не загружает файлы options#профиль, если отсутствует просто файл 
options.


То есть если у меня два профиля и файлы такие:
options#профиль1
options#профиль2

приходится добавлять ещё пустой файл options чтобы специфичные для профилей 
настройки воспринимались.
Comment 1 Andrew Kornilov 2006-12-27 14:48:07 MSK
Created attachment 1734 [details]
ifup-common patch
Comment 2 Andrew Kornilov 2006-12-27 14:53:51 MSK
Упс, патч не туда прилепил. Удалите, кто может, у меня только Edit есть.
Comment 3 Denis Ovsienko 2007-01-25 23:22:32 MSK
Да, замечание верное. Но необходимо отметить, что профили были придуманы как
отличие от основной конфигурации. Это значит, что хотя бы один вариант у вас
должен быть не в виде options#профильX, а в виде options. Если ваши потребности
вписываются в такую модель, то править ничего не нужно.
Comment 4 Vitaly Lipatov 2007-01-26 01:19:47 MSK
Не очень понимаю что для ноутбука, который перемещается между домом и работой 
является основной конфигурацией...
Мне кажется в случае нескольких профилей понятие "основная конфигурация" - 
лишнее. Само по себе нужно, но не вижу причин не загружать options#профиль в 
отсутствие options.
Comment 5 Denis Ovsienko 2007-01-28 18:23:32 MSK
Нужно это обдумать.
Comment 6 Denis Ovsienko 2007-02-04 12:33:32 MSK
*** Bug 10186 has been marked as a duplicate of this bug. ***
Comment 7 Denis Ovsienko 2009-05-24 21:12:37 MSD
Итак, более чем два года спустя выношу своё окончательное мнение. Из двух (или более) конфигураций одна обязана быть признанной человеком "основной", а другая (все остальные) --- "производной" (производными). При невозможности выбрать одну основную из двух следует воспользоваться монетой. Закрываю.
Comment 8 Vitaly Lipatov 2009-05-25 00:38:59 MSD
1. Я правильно понимаю, что настройки профилей составляются из того, что в options плюс из того что в options#профиль? То есть в случае профиля имеет место наследование параметров основной конфигурации? Тогда это имеет некий смысл.
2. CLOSED выставляет тот, кто повесил багу.
Comment 9 Denis Ovsienko 2009-05-25 01:26:49 MSD
1. Нет, это не так. Система специально устроена так, что если по тем или иным причинам используется содержимое файла options#something, то содержимое файла options не влияет вообще ни на что. И это тоже имеет смысл.
2. Я не знал.
Comment 10 Vitaly Lipatov 2009-05-25 01:32:44 MSD
(В ответ на комментарий №9)
> 1. Нет, это не так. Система специально устроена так, что если по тем или иным
> причинам используется содержимое файла options#something, то содержимое файла
> options не влияет вообще ни на что. И это тоже имеет смысл.
И это хорошо, что не влияет. Но почему требуется его наличие, если он не используется и  влияет? Вот это я считаю неправильным.
Comment 11 Denis Ovsienko 2009-05-25 19:30:49 MSD
На первый взгляд резонный вопрос, но я сейчас не в том положении, чтобы открывать ещё одну дискуссию. Не глядя менять этот алгоритм я не буду тоже. Предлагаю вернуться к этому в лучшие времена.