Bug 10392

Summary: tuning.xsl: contains invalid QName
Product: Sisyphus Reporter: kirill <kirill>
Component: alt-docs-xsl-commonAssignee: Artem Zolochevskiy <azol>
Status: CLOSED FIXED QA Contact: qa-sisyphus
Severity: blocker    
Priority: P4 CC: azol, boyarsh, cas, eostapets, george, mike
Version: unstable   
Hardware: all   
OS: Linux   
Bug Depends on:    
Bug Blocks: 9199    

Description kirill 2006-12-11 12:52:27 MSK
> > docs-wifi_short-kirill-050321-alt4.1
> >     /usr/src/RPM/BUILD/wifi_short.kirill.050321
> >     compilation error: file
file:///usr/share/xml/alt-docs-xsl/common/tuning.xsl line 184
> >     element with-param
> >     The value 'replace-with ' of the attribute 'name' is not a valid QName.

Я правильно понял, что следует удалить пробелы после "replace-with " ?

Вследствие этой ошибки не собирается вся документация в DocBook.
Comment 1 Vitaly A. Ostanin 2006-12-13 18:03:21 MSK
Да, пробел там лишний, исправил в cvs docs. Кстати, в cvs это строка 187, а не 184.

В связи с малым размером пакетов могу обновить все alt-docs-xsl хором. Разбивать
их для независимого обновления нет смысла, поскольку их должна заменить DCBEN:
http://vyt.vzljot.ru/dcben/browser/dcben/trunk

Есть возражения?
Comment 2 kirill 2006-12-13 18:38:26 MSK
Возражений нет, ждем обновления.
Есть вопрос: в какой примерно перспективе можно будет переходить на 
DCBEN? разговор об этом был уже давно. В связи с этим надо решить, есть 
ли смысл импортировать alt-docs-xsl-* из CVS на git.alt.

(In reply to comment #1)
> Да, пробел там лишний, исправил в cvs docs. Кстати, в cvs это строка 187, а не
184.
> 
> В связи с малым размером пакетов могу обновить все alt-docs-xsl хором. Разбивать
> их для независимого обновления нет смысла, поскольку их должна заменить DCBEN:
> http://vyt.vzljot.ru/dcben/browser/dcben/trunk
> 
> Есть возражения?



Comment 3 Vitaly A. Ostanin 2006-12-14 18:24:03 MSK
Залил в incoming alt-docs-xsl-0.4-alt1.src.rpm

На dcben можно будет переходить после его адаптации к стандартной упаковке Ruby
(аналогично rbot). Сейчас он заточен под Rubygems, но можно пользоваться и
просто развёрнутым деревом с модулями.

http://www.vzljot.ru/vyt/dcben/
http://www.vzljot.ru/vyt/dcben/docs/

Когда это будет, зависит от заинтересованных. Думаю, что в следующем году :)

Импортировать стили из cvs docs в git.alt не вижу смысла - проект маленький,
активно не развивается.
Comment 4 Vitaly A. Ostanin 2006-12-14 18:26:33 MSK
Кстати, я не смог залить релиз на
http://docs.altlinux.ru/releases/xsl/current/

Видимо, сайт docs.altlinux.ru опять куда-то переехал с lrn.ru. Ну и
перенаправление с docs.altlinux.ru на heap.altlinux.ru кажется странным. Старая
документация уже не нужна?
Comment 5 Michael Shigorin 2006-12-17 22:50:30 MSK
Вит, давай-ка я старую заархивирую на altlinux.org.ua.  Где забрать?
Comment 6 Vitaly A. Ostanin 2006-12-18 14:35:01 MSK
Заархивировать я и сам могу, проблема в смене Public ID стилей. Этот Public по
хорошему меняться не должен, хотя и допускаю, что никому не нужен, кроме самих
стилей.

А факт перенаправления (замещения) docs.a.r на heap.a.r желательно мотивировать.

Ну и неплохо бы сообщить, собирается ли теперь документация, и можно ли
закрывать баг.
Comment 7 kirill 2006-12-18 17:15:18 MSK
(In reply to comment #4)
> Кстати, я не смог залить релиз на
> http://docs.altlinux.ru/releases/xsl/current/
> 
> Видимо, сайт docs.altlinux.ru опять куда-то переехал с lrn.ru. Ну и
> перенаправление с docs.altlinux.ru на heap.altlinux.ru кажется странным. Старая
> документация уже не нужна?

Нет, сайт остался на lrn, но там происходили какие-то изменения, в результате
которых сами каталоги с публичными стилями переехали тоже на lrn.

Проверьте, на что у Вас указывает хост docs в ssh/config: скорее всего 
там cvs.altlinux.ru, а нужно теперь lrn.ru (для публикации там нужен аккаунт на
lrn, принадлежащий к группе webmaster). 

У меня make public отработал, так что новая версия стилей опубликована.
Comment 8 kirill 2006-12-18 17:26:41 MSK
(In reply to comment #6)
> Заархивировать я и сам могу, проблема в смене Public ID стилей. Этот Public по
> хорошему меняться не должен, хотя и допускаю, что никому не нужен, кроме самих
> стилей.

Public ID стилей не менялись; остаются теми же самыми.

> А факт перенаправления (замещения) docs.a.r на heap.a.r желательно мотивировать.

Перенаправление происходит на двух страницах: 
главная -- поскольку на docs.altlinux.ru вообще не было сведений о ПРОЕКТЕ как
таковом, а на heap.altlinux нечто подобное имеется.
Руководство начинающего мантейнера -- поскольку на heap.altlinux опубликована
обновленная и исправленная версия.

Вся остальная старая документация висит по старым адресам на docs, собственно, 
документация там только в трех каталогах: admin, alt и SOHO. 

С другой стороны, большая часть этой документации серьезно устарела. 
Большая часть документации из старого cvs/docs давно перенесена в Кучу 
в виде отдельных модулей (документов) и развивается там, и, соответственно, 
публикуется на heap.altlinux.

> 
> Ну и неплохо бы сообщить, собирается ли теперь документация, и можно ли
> закрывать баг.

Спасибо за оперативное исправление, но тут, к сожалению, ответ отрицательный, 
теперь документация не собирается с такой диагностикой: 

file:///usr/share/xml/alt-docs-xsl/common/collect_images.xsl line 36 element
variable
redefinition of global variable images-database
compilation error: file
file:///usr/share/xml/alt-docs-xsl/common/collect_images.xsl line 40 element
template
xsl:template: error duplicate name 'get-full-base'
Comment 9 Vitaly A. Ostanin 2006-12-18 18:18:11 MSK
В ~/.ssh/config docs указывает на lrn.ru

Пришлите мне testcase (желательно маленький), please, на личную почту.
Comment 10 Vitaly A. Ostanin 2006-12-19 17:06:59 MSK
Это уже ошибка в rpm-build-docs, а точнее, в файле
/usr/share/xml/alt-docs-xsl/common/alttune.xsl

1. rpm-build-docs с этим файлом должен требовать alt-docs-xsl-common, раз уж
alttune.xsl ставится в этот каталог.

2. alttune.xsl должен паковаться в alt-docs-xsl-common, если уж рулит тюнингом.

3. Если alttune.xsl не относится в alt-docs-xsl-common, то он должен лежать в
другом каталог и самостоятельно регистрироваться в XML Catalogs. Кстати, не
стоит использовать PUBLIC ID, которых нет:
http://docs.altlinux.ru/releases/xsl/current/common/alttune.xsl

4. alttune.xsl неправильно рулит тюнингом - include на collect_images.xsl и
tuning.xsl приводит к дублированию кода, т.к. tuning.xsl сам включает
collect_images.xsl. Не знаю назначения alttune.xsl, возможно, стоит использовать
import вместо include, но получится стиль, почти идентичный tuning.xsl из
alt-docs-xsl-common.

Перевесить (не умею) или заведёте новую багу?
Comment 11 kirill 2006-12-20 11:37:35 MSK
(In reply to comment #10)
> Это уже ошибка в rpm-build-docs, а точнее, в файле
> /usr/share/xml/alt-docs-xsl/common/alttune.xsl

<skipped/>

> Перевесить (не умею) или заведёте новую багу?

Большое спасибо за расследование! Думаю, я лучше заведу новую, а эту можно 
закрывать.

Comment 12 kirill 2006-12-20 11:49:33 MSK
Повесил на rpm-build-docs, #10459
Comment 13 Vitaly A. Ostanin 2006-12-20 12:43:07 MSK
Ошибка с QName исправлена, следующая ошибка относится к rpm-build-docs.