Bug 10950

Summary: неправильный путь к brctl
Product: Sisyphus Reporter: bee <bee>
Component: etcnetAssignee: Andrew Kornilov <hiddenman>
Status: CLOSED FIXED QA Contact: qa-sisyphus
Severity: normal    
Priority: P2 CC: ldv, rider, sem, shaba, shrek, vseleznv
Version: unstable   
Hardware: all   
OS: Linux   

Description bee 2007-02-27 16:07:23 MSK 
в functions прописано 
DEFAULT_BRCTL=/usr/sbin/brctl 
на самом деле:
rpm -ql bridge-utils |grep brctl
/sbin/brctl

 rpm -qa |grep bridge-utils
bridge-utils-1.2-alt2
 rpm -qa |grep etcnet      
etcnet-0.8.5-alt2

Actual Results:  
по умолчанию не поднимается мост, приходится делать симлинк
Comment 1 Valery Inozemtsev 2007-03-09 20:50:59 MSK 
*** Bug 11045 has been marked as a duplicate of this bug. ***
Comment 2 Andrew Kornilov 2007-03-10 03:57:48 MSK 
Спасибо, исправлено в trunk.
Comment 3 Valery Inozemtsev 2007-04-04 11:54:16 MSD 
в 0.8.6 не заметно
Comment 4 Andrew Kornilov 2007-04-04 12:54:11 MSD 
Я перед выпуском убрал это изменение в глобальной опции DEFAULT_BRCTL и добавил
строку BRCTL=/sbin/brctl в 50-Altlinux-desktop и server. Не помогает разве?
Comment 5 Valery Inozemtsev 2007-04-04 13:02:32 MSD 
$ rpmquery etcnet
etcnet-0.8.6-alt1

$ grep brctl /etc/net/* -r
/etc/net/options.d/50-ALTLinux-desktop:BRCTL=/sbin/brctl
/etc/net/scripts/functions:DEFAULT_BRCTL=/usr/sbin/brctl
/etc/net/scripts/setup-bri:PROF_BRCTL=`profiled_filename $MYIFACEDIR/brctl`

не думаю что BRCTL=/sbin/brctl поможет, в переменную BRCTL в скриптах вроде 
подставляется DEFAULT_BRCTL
Comment 6 Andrew Kornilov 2007-04-04 13:13:06 MSD 
Попробуй grep -ir brctl /etc/net
И почему все игнорируют опцию -i :)
Comment 7 Andrew Kornilov 2007-04-04 13:17:07 MSD 
Выражение ${BRCTL:=$DEFAULT_BRCTL}, afair, означает, что если не задана
переменная BRCTL, использовать DEFAULT_BRCTL
Comment 8 Valery Inozemtsev 2007-04-04 13:56:35 MSD 
не убедительно. DEFAULT_BRCTL и BRCTL указывающие в разные места вызывают по 
меньшей мере недоумение, а глядя на type brctl подозрение на слепоту 
разработчиков
Comment 9 Dmitry V. Levin 2007-04-04 14:04:11 MSD 
Reassigned to new maintainer.
Comment 10 Dmitry V. Levin 2007-04-04 14:07:44 MSD 
В etcnet bash-specific и так уже используется вовсю.
Так что нежелание использовать type в сочетании с "brctl" из $PATH
выглядит как нездоровый консерватизм.
Comment 11 Andrew Kornilov 2007-04-04 14:18:05 MSD 
Не понял с type, объясните, пожалуйста.
А разные BRCTL и DEFAULT_BRCTL чем смущают? Для этого и сделано. У всех(?) (и у
нас раньше тоже) brctl был где-то в /usr. etcnet Денисом не позиционировался как
for ALTLinux only, поэтому для каждого дистрибутива будут (и есть) свои такие
вот специфические опции.
Comment 12 Dmitry V. Levin 2007-04-04 14:26:53 MSD 
Я уже рассказывал, как можно использовать type на примере absolute(),
кажется в списке sisyphus@lists в прошлом месяце.

Я убеждён, что дистрибутиво-настриваемым являются только
$PATH, имя утилиты и семантика её использования.

А полное имя файла из вышеперечисленного вычисляется,
поэтому лучше его не настраивать.
Comment 13 Andrew Kornilov 2007-04-04 14:29:25 MSD 
А, да, это помню. Сделаем. Просто не хотелось в этом месте тоже менять, все
протестировать не успеем к релизу. Меня все-таки интересует, работает brctl
сейчас или нет?
Comment 14 Valery Inozemtsev 2007-04-04 14:34:35 MSD 
работает/неработает смогу проверить только дома. вечером отпишу
Comment 15 Denis Ovsienko 2007-04-04 21:13:46 MSD 
DEFAULT_BRCTL обеспечивает sane default, а BRCTL --- возможность его
переопределения в любом месте. Полные же пути к утилитам были использованы на
всякий случай, от них вреда сейчас я большого не вижу.
Comment 16 Dmitry V. Levin 2007-04-04 21:19:48 MSD 
В полных путях к утилитам есть определённый недостаток:
пути меняются чаще чем просто имена.
И этот bug report -- лишнее тому подтверждение.
Comment 17 Valery Inozemtsev 2007-04-05 09:44:37 MSD 
проверил. бридж поднимается. конкретно эту багу можно закрывать.
и можно тут же открывать следующую, на прибитые гвоздями пути к утилитам
Comment 18 Andrew Kornilov 2007-04-05 11:13:33 MSD 
Да, спасибо.

А новую открывай. Я вот все думаю, можно ли нашу bugzilla использовать для своих
todo и самому себе вешать баги? А то валяется вагон todo-шек всяких. Ну и свой
багтрекер есть еще. Неудобно/