[sisyphus] url_handler.sh bugfix and bugzilla

[sisyphus] url_handler.sh bugfix and bugzilla

[Ilia]

[-- Attachment #1: Type: text/plain, Size: 726 bytes --]

Доброго времени суток!
Заметил неудобство "конфигурабельности" с url_handler.sh, а именно, отсутствие 
обработки переменной среды $BROWSER. Нашёл аналогичную заметку в багзилле:
http://bugzilla.altlinux.ru/show_bug.cgi?id=3856
Даже исправил у себя, но как запостить bugfix?
И в какие сроки он попадёт в сизиф?


Если кому интересно, патч прилагается. Запускать:
patch -Np1 <url_handler.diff

Ниже идет словесное описание патча:
1. В файле /etc/urlview/url_handlers
вставить строку про $BROWSER перед galeon:
https_prgs=(
    "$BROWSER":VT
    /usr/bin/galeon:PW 

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

-- 
С уважением,
Илья.

[-- Attachment #2: url_handler.diff --]
[-- Type: text/x-diff, Size: 851 bytes --]

diff -Naur old/etc/urlview/url_handlers new/etc/urlview/url_handlers
--- old/etc/urlview/url_handlers	2004-04-22 22:11:45 +0300
+++ new/etc/urlview/url_handlers	2004-04-22 21:32:55 +0300
@@ -34,6 +34,7 @@
     /usr/bin/gopher:XT)
 # https_progs is a subset of http_progs, so defining https_progs first.
 https_prgs=(
+    "$BROWSER":VT
     /usr/bin/galeon:PW 
     /usr/bin/mozilla:PW 
     /usr/bin/netscape:PW 
diff -Naur old/etc/urlview/url_handler.sh new/etc/urlview/url_handler.sh
--- old/etc/urlview/url_handler.sh	2004-04-22 22:11:39 +0300
+++ new/etc/urlview/url_handler.sh	2004-04-22 22:12:01 +0300
@@ -68,7 +68,7 @@
 		tag="${ele##*:}"
 		prog="${ele%%:*}"
 		# Check that the executable is installed on this system:
-		if [[ ! -x "${prog%% *}" ]]; then
+		if ! which "${prog%% *}" &>/dev/null; then
 		    continue
 		fi
 		case "$tag" in

Re: [sisyphus] url_handler.sh bugfix and bugzilla

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 1223 bytes --]

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

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]