Bug 14368 - Typo in sed expression in /usr/share/jabber/component/j2j
: Typo in sed expression in /usr/share/jabber/component/j2j
Status: CLOSED NOTABUG
: Sisyphus
(All bugs in Sisyphus/jabber-j2j)
: unstable
: all Linux
: P2 major
Assigned To:
:
:
:
:
:
  Show dependency tree
 
Reported: 2008-02-09 13:04 by
Modified: 2008-02-09 17:14 (History)


Attachments


Note

You need to log in before you can comment on or make changes to this bug.


Description From 2008-02-09 13:04:41
В 1.1.8-alt1.2, в /usr/share/jabber/component/j2j есть опечатка:
в строке 10, позиция 62 стоит лишний символ 'u':

    --host) sed -n '/^[[:blank:]]\+JID=/ s/^[[:blank:]]\+JID=u"\([^\"]\+\).
\+$/\1/p' < $CONFIG_FILE

Должно быть 

    --host) sed -n '/^[[:blank:]]\+JID=/ s/^[[:blank:]]\+JID="\([^\"]\+\).
\+$/\1/p' < $CONFIG_FILE


Как итог, скрипт не возвращает имя хоста при вызове с ключом --host.
Steps to Reproduce:
# grep JID /etc/jabber-j2j/j2j.conf
    JID="j2j.tests.oits.ru"
# /usr/share/jabber/component/j2j --host
#
------- Comment #1 From 2008-02-09 15:12:35 -------
я не выпускал релиз пакета 1.1.8-alt1.2, файл /etc/jabber-j2j/j2j.conf не 
принадлежит пакету jabber-j2j. в актуальном 1.1.8-alt2 и предыдущем alt1
обсуждаемый символ не является опечаткой.
------- Comment #2 From 2008-02-09 16:33:08 -------
Извиняюсь, alt1.2 - это моя сборка alt1 под branch. Перемещение конфигурации 
в /etc/jabber-j2j - оттуда же.


По 'JID=u"' - согласен, в рамках конфигурации по-умолчанию - тоже Вы правы.

Есть только несколько но:
- JID, конечно, может содержать юникод - вопрос только в том, насколько это 
оправдано в данном месте, и насколько такой JID воспримут в своей конфигурации 
jabber-сервера;
- с JID, заданным простой строкой, j2j тоже вполне работает;
- люди, не знакомые с Python, скорее всего 'u' выкинут - поскольку это 
единственная таким образом записанная строка в файле. И у них всё будет 
работать - до следующего вызова /usr/share/jabber/component/j2j.

Хорошо бы всё-таки подправить выражение для sed, на  JID=u\?"

И, может быть, добавив комментарий к этому символу непосредственно в файл 
конфигурации.
------- Comment #3 From 2008-02-09 17:14:31 -------
fixed in 1.1.8-alt3