Bug 28185 - url handling broke
Summary: url handling broke
Status: CLOSED DUPLICATE of bug 28728
Alias: None
Product: Sisyphus
Classification: Development
Component: xdg-utils (show other bugs)
Version: unstable
Hardware: all Linux
: P3 major
Assignee: Vitaly Lipatov
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks: 27685
  Show dependency tree
 
Reported: 2012-12-05 14:03 MSK by Michael Shigorin
Modified: 2014-05-24 23:36 MSK (History)
7 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Shigorin 2012-12-05 14:03:09 MSK
После обновления до 1.1.0-alt7 сломалось открытие ссылок: при попытке открыть в e.g. xchat ссылку http://www.altlinux.org вместо нового таба открывается новое окошко firefox с _двумя_ табами, в одном "http://http.com/" (видимо, было "http"), в другом -- "file:////www.altlinux.org" (видимо, было "//www.altlinux.org").
Comment 2 manowar@altlinux.org 2012-12-05 14:25:19 MSK
$ grep -r 'xchat' xdg-utils
Ничего не находит.

О каком скрипте из пакета идёт речь? Давайте сравним его содержимое между версиями.
Comment 3 Michael Shigorin 2012-12-05 14:58:03 MSK
Это была информация о том, как именно воспроизводится :)

Перед тем, как вешать, взял гит и глянул разницу с alt6 -- подозрительного не заметил, потому из начальной формулировки упоминание про alt6 убрал.  Затем всё же откатил и тогда уже уточнил.

Сейчас глянул чуть внимательней git diff 66a997ad1e038bb7e1d67244fbd0e3f4b0443bdd..f84dc0185d4e8b57281f3c6b52770a2c6dd93137 и продолжаю не замечать очевидных оснований для такого изменения поведения.

Проверил на всякий пересобранный http://ftp.altlinux.org/pub/distributions/archive/Sisyphus/2012/09/12/files/SRPMS/xdg-utils-1.1.0-alt6.src.rpm -- работает.
Comment 4 Michael Shigorin 2013-02-03 15:25:40 MSK
(В ответ на комментарий №2)
> О каком скрипте из пакета идёт речь?
Насколько понимаю, это xdg-open -- по крайней мере тот, что в alt6, работает как должен, а с тем, что в alt7, xdg-open http://localhost/ приводит к воспроизведению данной баги.

В том же regular-e17 это проявляется так, что при тычке в http://localhost в terminology получаем открывание двух ссылок -- "http://http.com/" и "file:////localhost" в firefox.

Пришлось и на новой машинке откатить да захолдить -- IMHO это p7 blocker всё же.
Comment 5 AEN 2013-02-03 15:28:17 MSK
(В ответ на комментарий №4)
> (В ответ на комментарий №2)
> > О каком скрипте из пакета идёт речь?
> Насколько понимаю, это xdg-open -- по крайней мере тот, что в alt6, работает
> как должен, а с тем, что в alt7, xdg-open http://localhost/ приводит к
> воспроизведению данной баги.
> 
> В том же regular-e17 это проявляется так, что при тычке в http://localhost в
> terminology получаем открывание двух ссылок -- "http://http.com/" и
> "file:////localhost" в firefox.
> 
> Пришлось и на новой машинке откатить да захолдить -- IMHO это p7 blocker всё
> же.

Не хочу вешать блокер, когда откат помогает. Прошу исправить как можно скорее.
Comment 6 AEN 2013-02-03 16:01:27 MSK
(В ответ на комментарий №5)
> (В ответ на комментарий №4)
> > (В ответ на комментарий №2)
> > > О каком скрипте из пакета идёт речь?
> > Насколько понимаю, это xdg-open -- по крайней мере тот, что в alt6, работает
> > как должен, а с тем, что в alt7, xdg-open http://localhost/ приводит к
> > воспроизведению данной баги.
> > 
> > В том же regular-e17 это проявляется так, что при тычке в http://localhost в
> > terminology получаем открывание двух ссылок -- "http://http.com/" и
> > "file:////localhost" в firefox.
> > 
> > Пришлось и на новой машинке откатить да захолдить -- IMHO это p7 blocker всё
> > же.
> 
> Не хочу вешать блокер, когда откат помогает. Прошу исправить как можно скорее.

Не вопроизводится в других приложениях, которые у меня под рукой. Нужно более детальное описание и изучение указанного mike@ коммита.
Comment 7 Michael Shigorin 2013-04-08 14:36:27 MSK
В общем, у меня на hold до сих пор.
Comment 8 Michael Shigorin 2013-04-08 17:53:41 MSK
<raorn> gvy: каким-то ЧУДОМ мановар заставил скрипты собираться из .in
<raorn> сделал он это, правда, черезжопу, но тем не менее
Comment 9 manowar@altlinux.org 2013-04-08 18:01:07 MSK
  «Но тем не менее» означает, что сборку из *.in оставляем и чиним? Или пытаемся откатить?
Comment 10 AEN 2013-04-08 18:04:08 MSK
(В ответ на комментарий №9)
>   «Но тем не менее» означает, что сборку из *.in оставляем и чиним? Или
> пытаемся откатить?

Багу исправляем. А как -- на усмотрение мейнтейнера.
Comment 11 AEN 2013-04-11 09:12:37 MSK
На последнего сборщика, который, судя по сообщению, внес ошибку.
Прошу подтвердить или отклонить ошибку, написать об этом сюда. После этого можно перевесить по согласованию.
Да, и приходится пока считать блокером для p7.
Comment 12 AEN 2013-04-21 17:45:14 MSK
Прошу мейнтейнера откликнуться.
Comment 13 Vitaly Lipatov 2013-04-24 18:07:38 MSK
(В ответ на комментарий №6)
> Не вопроизводится в других приложениях, которые у меня под рукой. Нужно более
> детальное описание и изучение указанного mike@ коммита.
Не вижу в этом коммите никаких проблем. Если что и стоит сравнить, так это содержимое пакетов alt6 и alt7. К сожалению
Архив Сизифа http://www.altlinux.org/Archive недоступен, так что сейчас проверить не могу.

Я не собирал этот пакет уже давно, но у меня не воспроизводится:

$ sh -x `which xdg-open` http://localhost/
+ check_common_commands http://localhost/
+ '[' 1 -gt 0 ']'
+ parm=http://localhost/
+ shift
+ case "$parm" in
+ '[' 0 -gt 0 ']'
+ '[' -z '' ']'
+ unset XDG_UTILS_DEBUG_LEVEL
+ '[' 0 -lt 1 ']'
+ xdg_redirect_output=' > /dev/null 2> /dev/null'
+ '[' xhttp://localhost/ '!=' x ']'
+ url=
+ '[' 1 -gt 0 ']'
+ parm=http://localhost/
+ shift
+ case "$parm" in
+ '[' -n '' ']'
+ url=http://localhost/
+ '[' 0 -gt 0 ']'
+ '[' -z http://localhost/ ']'
+ detectDE
+ unset GREP_OPTIONS
+ '[' -n '' ']'
+ '[' x = x ']'
+ '[' x = xtrue ']'
+ '[' x '!=' x ']'
+ '[' x '!=' x ']'
++ dbus-send --print-reply --dest=org.freedesktop.DBus /org/freedesktop/DBus org.freedesktop.DBus.GetNameOwner string:org.gnome.SessionManager
+ xprop -root _DT_SAVE_MODE
+ grep ' = \"xfce4\"$'
+ DE=xfce
+ '[' xxfce = x ']'
+ '[' xxfce = x ']'
+ '[' xxfce = xgnome ']'
+ '[' xxfce = x ']'
+ DEBUG 2 'Selected DE xfce'
+ '[' -z '' ']'
+ return 0
+ '[' x/usr/bin/xbrowser = x ']'
+ case "$DE" in
+ open_xfce http://localhost/
+ exo-open http://localhost/
+ '[' 0 -eq 0 ']'
+ exit_success
+ '[' 0 -gt 0 ']'
+ exit 0
Comment 14 Andrey Cherepanov 2013-04-29 10:45:55 MSK
На Simply Linux не воспроизводится. Ни через явный вызов, ни через xchat.
Comment 15 Michael Shigorin 2014-05-24 21:06:50 MSK
Помечу тут как дубликат, там меньше шуму [пока].

*** This bug has been marked as a duplicate of bug 28728 ***
Comment 16 Michael Shigorin 2014-05-24 23:36:34 MSK
(В ответ на комментарий №14)
> На Simply Linux не воспроизводится. Ни через явный вызов, ни через xchat.
Это потому, что там определяется Xfce и соответственно не используется глючная функция open_generic().  Отправил 1.1.0-alt10.