Bug 14368

Summary: Typo in sed expression in /usr/share/jabber/component/j2j
Product: Sisyphus Reporter: Nikolay A. Fetisov <naf>
Component: jabber-j2jAssignee: Sergey Bolshakov <sbolshakov>
Status: CLOSED NOTABUG QA Contact: qa-sisyphus
Severity: major    
Priority: P2    
Version: unstable   
Hardware: all   
OS: Linux   

Description Nikolay A. Fetisov 2008-02-09 13:04:41 MSK
В 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 Sergey Bolshakov 2008-02-09 15:12:35 MSK
я не выпускал релиз пакета 1.1.8-alt1.2, файл /etc/jabber-j2j/j2j.conf не 
принадлежит пакету jabber-j2j. в актуальном 1.1.8-alt2 и предыдущем alt1
обсуждаемый символ не является опечаткой.
Comment 2 Nikolay A. Fetisov 2008-02-09 16:33:08 MSK
Извиняюсь, 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 Sergey Bolshakov 2008-02-09 17:14:31 MSK
fixed in 1.1.8-alt3