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

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

[Евгений Ростовцев]

Привет!

> > -%py_dependencies() %(%{expand: %%__python_deps_internal %{*}})
> > +%py_dependencies() BuildRequires: %(%{expand: %%__python_deps_internal %{*}})
[...]
> А почему это изменение семантики называется "Fix"?

Потому что семантику я поменял так, чтобы она соответствовала своему
же описанию:

> >  # 	что будет преобразовано в
> >  # 	BuildRequires: pythonX.Y(module1) pythonX.Y(module2) pythonX.Y(module3)

Без этого "Fix" вместо
BuildRequires: pythonX.Y(...)...
мы получаем:
pythonX.Y(...)...

И сборка убивается ещё на стадии установки пакетов в хэшер.

В результате чего мейнтейнеру придёт в голову написать нечто,
противоречащее задокументированному в rpm-build-python, т.е. писать:
BuildRequires: %py_dependencies qwe

А не как задумано, т.е.:
%py_dependencies qwe

-- 
REAL aka Евгений Ростовцев, программист ЦНИТ КемГУ

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

[Timur Batyrshin]

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

On Wed, 3 Feb 2010 14:40:53 +0700
Евгений Ростовцев wrote:

> В результате чего мейнтейнеру придёт в голову написать нечто,
> противоречащее задокументированному в rpm-build-python, т.е. писать:
> BuildRequires: %py_dependencies qwe
> 
> А не как задумано, т.е.:
> %py_dependencies qwe

Может я, конечно, не в теме, и не туда смотрю, но за пару строк до твоего изменения
как раз и написано, что использовать нужно
BuildRequires: %py_dependencies qwe

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

Да и в документации в каталоге doc/ это же написано.

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

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

[Dmitry V. Levin]

[-- 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 --]