Re: [devel] [SCM] packages/rpm-build-python: heads/master

["Dmitry V. Levin" <ldv@altlinux.org>]

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

On Wed, Feb 03, 2010 at 02:40:53PM +0700, Евгений Ростовцев wrote:
> Привет!
> 
> > > -%py_dependencies() %(%{expand: %%__python_deps_internal %{*}})
> > > +%py_dependencies() BuildRequires: %(%{expand: %%__python_deps_internal %{*}})
> [...]
> > А почему это изменение семантики называется "Fix"?
> 
> Потому что семантику я поменял так, чтобы она соответствовала своему
> же описанию:

Описание выглядит следующим образом:

# py_dependencies <имя_модуля1> [<имя_модуля2> ...]
#
#       Формирует список модулей питон для использования, например,
#       в тегах BuildRequires или Obsoletes, н-р, так:
#       BuildRequires: %py_dependencies module1 module2 module3
#       что будет преобразовано в
#       BuildRequires: pythonX.Y(module1) pythonX.Y(module2) pythonX.Y(module3)
#
%py_dependencies() %(%{expand: %%__python_deps_internal %{*}})

И в doc/RpmMacros.txt:

    py_dependencies <имя_модуля1> [ <имя_модуля2> ... ]        
        
        Генерирует список модулей для указания в директивах BuildRequires и др,
        расширяется в pythonX.Y(<имя_модуля1>) [ pythonX.Y(<имя_модуля1>) ... ]

Так что вы поменяли семантику, сломав обратную совместимость и сделав
макрос несоответствующим документации.

Можете изготовить новый макрос, например, %py_buildrequires, если хотите.


-- 
ldv

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