[sisyphus] url_handler.sh bugfix and bugzilla

[Dmitry V. Levin]

On Thu, Apr 22, 2004 at 10:24:04PM +0300, Ilia wrote:
> Доброго времени суток!
> Заметил неудобство "конфигурабельности" с url_handler.sh, а именно, отсутствие 
> обработки переменной среды $BROWSER. Нашёл аналогичную заметку в багзилле:
> http://bugzilla.altlinux.ru/show_bug.cgi?id=3856
> Даже исправил у себя, но как запостить bugfix?

Просто добавьте патч, который вы предлагаете, в bugzilla
(attachментом к #3856).

> И в какие сроки он попадёт в сизиф?

Если maintainer согласится с вашим предложением, то, скорее всего, быстро.

> Если кому интересно, патч прилагается. Запускать:
> patch -Np1 <url_handler.diff
> 
> Ниже идет словесное описание патча:
> 1. В файле /etc/urlview/url_handlers
> вставить строку про $BROWSER перед galeon:
> https_prgs=(
>     "$BROWSER":VT
>     /usr/bin/galeon:PW 

Это предложение спорно.  А вдруг $BROWSER не предназначен для работы по
https?

> 2. Заменить 71 строку в файле /etc/urlview/url_handler.sh
> if [[ ! -x "${prog%% *}" ]]; then
> на
> if ! which "${prog%% *}" &>/dev/null; then

А вот это мне больше нравится, особенно если вместо which использовать
заменитель, аналогичный тому, что использован в /etc/init.d/functions.


-- 
ldv
----------- следующая часть -----------
Было удалено вложение не в текстовом формате...
Имя     : =?iso-8859-1?q?=CF=D4=D3=D5=D4=D3=D4=D7=D5=C5=D4?=
Тип     : application/pgp-signature
Размер  : 189 байтов
Описание: =?iso-8859-1?q?=CF=D4=D3=D5=D4=D3=D4=D7=D5=C5=D4?=
Url     : <http://lists.altlinux.org/pipermail/sisyphus/attachments/20040423/d476ce5c/attachment-0003.bin>