Re: [devel] rpm макросы для пакета mk-configure

[Aleksey Cheusov <vle@gmx.net>]

25.05.2020, 22:49, "Andrey Savchenko" <bircoph@altlinux.org>:
> On Mon, 25 May 2020 21:59:49 +0300 Aleksey Cheusov wrote:
>>>  Нужен review и разрешение на публикацию в Сизифе.
>
> Вот замеченные проблемы. Большинство из них есть в логах сборки, так

Обновил.
См. http://git.altlinux.org/tasks/252281/

> что полезно их грепать на предмет "(warning|error):".
>
> 1) mk-configure-0.34.2-alt4:
>
> 1.1) При сборке пакета игнорируются CFLAGS, например, так:

Да, это только тесты.

> 2) libmaa-1.4.7-alt2:
>
> 2.1) Нет зависимости на макрос.
> Зависимости на макросы нужно делать через BuildRequires(pre):
> BuildRequires(pre): rpm-macros-mk-configure

Я сделал так, как сделано с cmake, где cmake Requires rpm-macro-cmake и этого вроде достаточно.
"BuildRequires: rpm-macro-mk-configure" действительно исправляет ворнинги, но мне непонятно,
зачем они. Зависимость добавил -- бессмысленные ворнинги пропали :-)

> 2.2) Есть установленные, но не упакованные файлы:

Fixed

> С /usr/share/doc/libmaa какой-то косяк вышел и там дублируются
> файлы документации, уже разложенные в другие места:

Fixed

> 3) runawk-1.6.1-alt2:
>
> 3.1) С макросами аналогично 2.1)

Fixed

> 3.2) Безопаснее использовать mkstemp вместо tempnam:
> /usr/src/RPM/BUILD/runawk-1.6.1/runawk/runawk.c:168: warning: the use of `tempnam' is dangerous, better use `mkstemp'

С точки зрения security здесь все в порядке. Так что это предупреждение я сознательно игнорирую.