[devel] Что за фигня у incoming'a с проверкой макросов в спеках?

[devel] Что за фигня у incoming'a с проверкой макросов в спеках?

[Илья Евсеев]

  Всем привет.
На выходных coreteam внёс очередное историческое улучшение
в процедуру проверки присылаемых в Сизиф пакетов, после чего QA Robot
не смог пересобрать где-то с десяток пакетов, числящихся за мной.

БОльшая часть ошибок касалась макроса %url, который до некоторых пор
я писал как придётся - и %Url, и %URL.
Придирки по мелочам неприятны, но терпимы.

Однако когда новый проверяльщик приё*ывается к комментариям - это уже перебор!
Например, в netcat.spec есть такой фрагмент:
--------
#  Sources are taken and packaged by following command:                                                                       
#     CVSROOT=anoncvs@anoncvs2.de.openbsd.org:/cvs cvs get src/usr.bin/nc                                                     
#     tar czf netcat-$(date +%Y%m%d).tar.gz src/                                                                              
Source: %name-20050411.tar.gz
--------
Теперь при пересборке выдаётся ругань на неопределённые макросы %Y, %m и %d. Каково?

Или в mdadm.spec:
#ghost %config(noreplace,missingok) %noverify /etc/mdadm.conf.before-udev
Жил себе до лучших времён кусок, никого не трогал - и вот на тебе, фасыст, гранату!

В связи с этим у меня просьба-вопрос:
нельзя ли добавить такой макрос, который будет включать/отключать
поиск неопределённых макросов внутри комментариев?

С уважением,
Илья Евсеев

[devel] Re: Что за фигня у incoming'a с проверкой макросов в спеках?

[Michael Shigorin]

On Wed, Oct 12, 2005 at 04:07:24PM +0400, Илья Евсеев wrote:
> нельзя ли добавить такой макрос, который будет
> включать/отключать поиск неопределённых макросов внутри
> комментариев?

Дима уже сказал, что можно, но затруднительно.  Макрофабрика rpm
не в курсе комментариев...

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

Re: [devel] Re: Что за фигня у incoming'a с проверкой макросов в спеках?

[Sergey V Turchin]

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

On Wednesday 12 October 2005 16:18, Michael Shigorin wrote:

[...]

> Дима уже сказал, что можно, но затруднительно.
Я че-то пропустил то письмо :-(

> Макрофабрика rpm не в курсе комментариев...
Убрать их можно перед проверкой временно созданного spec-файла

-- 
Regards, Sergey, ALT Linux Team, http://www.altlinux.ru
http://stinkfoot.org:11371/pks/lookup?op=get&search=0x1C2A3F08

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

Re: [devel] Что за фигня у incoming'a с проверкой макросов в спеках?

[Dmitry V. Levin]

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

On Wed, Oct 12, 2005 at 04:07:24PM +0400, Илья Евсеев wrote:
> На выходных coreteam внёс очередное историческое улучшение
> в процедуру проверки присылаемых в Сизиф пакетов, после чего QA Robot
> не смог пересобрать где-то с десяток пакетов, числящихся за мной.
> 
> БОльшая часть ошибок касалась макроса %url, который до некоторых пор
> я писал как придётся - и %Url, и %URL.

Ещё одна ошибка - использование макроса %url до указания тэга Url.

> Придирки по мелочам неприятны, но терпимы.
> 
> Однако когда новый проверяльщик приё*ывается к комментариям - это уже перебор!
> Например, в netcat.spec есть такой фрагмент:
> --------
> #  Sources are taken and packaged by following command:
> #     CVSROOT=anoncvs@anoncvs2.de.openbsd.org:/cvs cvs get src/usr.bin/nc
> #     tar czf netcat-$(date +%Y%m%d).tar.gz src/
> Source: %name-20050411.tar.gz
> --------
> Теперь при пересборке выдаётся ругань на неопределённые макросы %Y, %m и %d. Каково?

Неприятно. :)

> Или в mdadm.spec:
> #ghost %config(noreplace,missingok) %noverify /etc/mdadm.conf.before-udev
> Жил себе до лучших времён кусок, никого не трогал - и вот на тебе, фасыст, гранату!

Там вроде другое:
error: Macro %pre_udev_conf not found
error: line 86: #[ -e %pre_udev_conf ] || exit 0

> В связи с этим у меня просьба-вопрос:
> нельзя ли добавить такой макрос, который будет включать/отключать
> поиск неопределённых макросов внутри комментариев?

Реализовать это непросто.  Препроцессор, раскрывающий макросы, ещё не знает
о том, что бывают на свете комментарии.

Мне удалось его обучить, что бывают секции, в которых нераскрытые макросы
допустимы, как раньше.

Подождите до следующей тестовой пересборки, может я чего-нибудь придумаю.


-- 
ldv

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

Re: [devel] Что за фигня у incoming'a с проверкой макросов в спеках?

[Sergey V Turchin]

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

On Wednesday 12 October 2005 16:49, Dmitry V. Levin wrote:

[...]

> Реализовать это непросто.
Тогда может в sisyphus_check проверять spec с убранными 
комментариями? Или он еще не распаковывает пакеты для проверки?

-- 
Regards, Sergey, ALT Linux Team, http://www.altlinux.ru
http://stinkfoot.org:11371/pks/lookup?op=get&search=0x1C2A3F08

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

Re: [devel] Что за фигня у incoming'a с проверкой макросов в спеках?

[Dmitry V. Levin]

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

On Wed, Oct 12, 2005 at 05:25:07PM +0400, Sergey V Turchin wrote:
> On Wednesday 12 October 2005 16:49, Dmitry V. Levin wrote:
> 
> [...]
> 
> > Реализовать это непросто.
> Тогда может в sisyphus_check проверять spec с убранными 
> комментариями? Или он еще не распаковывает пакеты для проверки?

sisyphus_check обрабатывает только заголовок пакета, не залезая внутрь.


-- 
ldv

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

Re: [devel] Что за фигня у incoming'a с проверкой макросов в спеках?

[Alexey Gladkov]

Sergey V Turchin пишет:
> Тогда может в sisyphus_check проверять spec с убранными 
> комментариями? Или он еще не распаковывает пакеты для проверки?

sisyphus_check не распаковывает пакеты.

Сделать проверку макросов в sisyphus_check неполучится.

-- 
Rgrds, legion

Re: [devel] Что за фигня у incoming'a с проверкой макросов в спеках?

[Igor Zubkov]

В сообщении от Среда, 12-Окт-2005 16:55 Alexey Gladkov написал(a):
> Sergey V Turchin пишет:
> > Тогда может в sisyphus_check проверять spec с убранными
> > комментариями? Или он еще не распаковывает пакеты для проверки?
>
> sisyphus_check не распаковывает пакеты.

rpmlint распаковывает.

> Сделать проверку макросов в sisyphus_check неполучится.

Может rpmlint напрячь? Я бы такое потянул.

-- 
Now playing: 

Re: [devel] Что за фигня у incoming'a с проверкой макросов в спеках?

[Alexey Gladkov]

Igor Zubkov пишет:
> rpmlint распаковывает.

Я знаю.

> Может rpmlint напрячь? Я бы такое потянул.

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

-- 
Rgrds, legion

Re: [devel] Что за фигня у incoming'a с проверкой макросов в спеках?

[Dmitry V. Levin]

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

On Wed, Oct 12, 2005 at 04:49:47PM +0400, Dmitry V. Levin wrote:
> On Wed, Oct 12, 2005 at 04:07:24PM +0400, Илья Евсеев wrote:
[...]
> > В связи с этим у меня просьба-вопрос:
> > нельзя ли добавить такой макрос, который будет включать/отключать
> > поиск неопределённых макросов внутри комментариев?
> 
> Реализовать это непросто.  Препроцессор, раскрывающий макросы, ещё не знает
> о том, что бывают на свете комментарии.
> 
> Мне удалось его обучить, что бывают секции, в которых нераскрытые макросы
> допустимы, как раньше.
> 
> Подождите до следующей тестовой пересборки, может я чего-нибудь придумаю.

* Thu Oct 13 2005 Dmitry V. Levin <ldv@altlinux> 4.0.4-alt51
- rpmio/macro.c, build/parseSpec:
  + When %_allow_undefined_macros is set to true value,
    pass undefined macros but issue warning messages.
  + When processing section where comments should be skipped,
    pass undefined macros within comments but issue warning messages.
  + When processing %prep, %build, %install and %clean sections,
    pass undefined macros and issue warning messages.


-- 
ldv

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

Re: [devel] Что за фигня у incoming'a с проверкой макросов в спеках?

[Kirill A. Shutemov]

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

On 22:10 Wed 12 Oct, Alexey Gladkov wrote:
> Igor Zubkov пишет:
> > rpmlint распаковывает.
> 
> Я знаю.
> 
> > Может rpmlint напрячь? Я бы такое потянул.
> 
> Напрячь-то можно ... можно и sisyphus_check научить извлекать спек, но
> толку от этого немного будет. Полученный спек проверять на
> неправильные макросы нельзя.
> 
Из-за /etc/rpm/macros.d/* в builddeps?

-- 
Kirill A. Shutemov				Belarus, Minsk
E-mail:	k.shutemov (AT) sam-solutions.net
JID: kas (AT) altlinux.org
ICQ: 152302675

Для начала просто попробуйте осознать, что красота интерфейса и его
удобство друг с другом абсолютно не связаны.
		-- mithraen in community@

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

Re: [devel] Что за фигня у incoming'a с проверкой макросов в спеках?

[Sergey V Turchin]

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

On Wednesday 12 October 2005 17:48, Dmitry V. Levin wrote:

[...]

> sisyphus_check обрабатывает только заголовок пакета, не залезая
> внутрь.
А может придумать hasher_sisyphus_check? Там пакет распакованный.

-- 
Regards, Sergey, ALT Linux Team, http://www.altlinux.ru
http://stinkfoot.org:11371/pks/lookup?op=get&search=0x1C2A3F08

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