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

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

[Dmitry V. Levin]

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

On Thu, Oct 18, 2012 at 09:04:51AM +0000, Pavel Shilovsky wrote:
> Update of /people/piastry/packages/rpm-build-python.git
[...]
> commit dfbf2f35ec7cf6ac55b6579f813d03c2078795a5
> Author: Pavel Shilovsky <piastry@etersoft.ru>
> Date:   Thu Oct 18 12:57:28 2012 +0400
> 
>     Process *.py_tmpl files properly
> 
[...]
> --- a/rpm-build-python/python.req.py
> +++ b/rpm-build-python/python.req.py
> @@ -231,6 +231,8 @@ else :
>                  def must_fail() :
>                      if ext == '.py' :
>                          return True
> +                    if ext == '.py_tmpl':
> +                        return False
>                      if len(lines) > 0 and lines[0].startswith('#!') :
>                          return True
>                  try :

Насколько я понимаю, суть этого изменения в том, что ошибки поиска
зависимостей в python-файлах с именами, заканчивающимися на .py_tmpl,
не будут считаться ошибками.  Просьба пояснить, зачем понадобилось такое
изменение, и почему оно правильное.


-- 
ldv

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

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

[Pavel Shilovsky]

30 октября 2012 г., 11:50 пользователь Dmitry V. Levin
<ldv@altlinux.org> написал:
> On Thu, Oct 18, 2012 at 09:04:51AM +0000, Pavel Shilovsky wrote:
>> Update of /people/piastry/packages/rpm-build-python.git
> [...]
>> commit dfbf2f35ec7cf6ac55b6579f813d03c2078795a5
>> Author: Pavel Shilovsky <piastry@etersoft.ru>
>> Date:   Thu Oct 18 12:57:28 2012 +0400
>>
>>     Process *.py_tmpl files properly
>>
> [...]
>> --- a/rpm-build-python/python.req.py
>> +++ b/rpm-build-python/python.req.py
>> @@ -231,6 +231,8 @@ else :
>>                  def must_fail() :
>>                      if ext == '.py' :
>>                          return True
>> +                    if ext == '.py_tmpl':
>> +                        return False
>>                      if len(lines) > 0 and lines[0].startswith('#!') :
>>                          return True
>>                  try :
>
> Насколько я понимаю, суть этого изменения в том, что ошибки поиска
> зависимостей в python-файлах с именами, заканчивающимися на .py_tmpl,
> не будут считаться ошибками.  Просьба пояснить, зачем понадобилось такое
> изменение, и почему оно правильное.

Данное изменение требуется для корректной сборки python-module-migrate
(http://git.altlinux.org/people/piastry/packages/?p=python-module-migrate.git;a=summary),
который отказывается собираться без данного патча, пишет:

python.req: ERROR:
/home/piastry/tmp/python-module-migrate-buildroot/usr/lib/python2.7/site-packages/migrate/versioning/templates/manage/default.py_tmpl:
invalid syntax (line 5)
Traceback (most recent call last):
  File "/usr/lib/rpm/python.req.py", line 237, in <module>
    lis = parser.suite('\n'.join(lines)+'\n').tolist(line_info=1)
  File "<string>", line 5
    _vars = locals().copy()
          ^
SyntaxError: invalid syntax
find-requires: ERROR: /usr/lib/rpm/python.req failed

что собственно говорит о том, что py_tmpl не соответствуют синтаксису
python в полной мере - поэтому падать с ошибкой в данном случае не
имеет смысла.

-- 
Best regards,
Pavel Shilovsky.

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

[thecrux]

On Tue, Oct 30, 2012 at 12:32:30PM +0400, Pavel Shilovsky wrote:
> 30 октября 2012 г., 11:50 пользователь Dmitry V. Levin
...
> > Насколько я понимаю, суть этого изменения в том, что ошибки поиска
> > зависимостей в python-файлах с именами, заканчивающимися на .py_tmpl,
> > не будут считаться ошибками.  Просьба пояснить, зачем понадобилось такое
> > изменение, и почему оно правильное.
> 
> Данное изменение требуется для корректной сборки python-module-migrate
> (http://git.altlinux.org/people/piastry/packages/?p=python-module-migrate.git;a=summary),
> который отказывается собираться без данного патча, пишет:
> 
> python.req: ERROR:
> /home/piastry/tmp/python-module-migrate-buildroot/usr/lib/python2.7/site-packages/migrate/versioning/templates/manage/default.py_tmpl:
> invalid syntax (line 5)
> Traceback (most recent call last):
>   File "/usr/lib/rpm/python.req.py", line 237, in <module>
>     lis = parser.suite('\n'.join(lines)+'\n').tolist(line_info=1)
>   File "<string>", line 5
>     _vars = locals().copy()
>           ^
> SyntaxError: invalid syntax
> find-requires: ERROR: /usr/lib/rpm/python.req failed
> 
> что собственно говорит о том, что py_tmpl не соответствуют синтаксису
> python в полной мере - поэтому падать с ошибкой в данном случае не
> имеет смысла.

AFAIR, для подобных целей есть макрос %add_findreq_skiplist

Укажите в спеке шаблон пути, который надо исключить из проверки, например:

%add_findreq_skiplist %python_sitelibdir_noarch/migrate/versioning/templates/*

-- 
Vladimir Lettiev aka crux ✉ theCrux@gmail.com

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

[Dmitry V. Levin]

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

On Tue, Oct 30, 2012 at 12:32:30PM +0400, Pavel Shilovsky wrote:
> 30 октября 2012 г., 11:50 пользователь Dmitry V. Levin
> <ldv@altlinux.org> написал:
> > On Thu, Oct 18, 2012 at 09:04:51AM +0000, Pavel Shilovsky wrote:
> >> Update of /people/piastry/packages/rpm-build-python.git
> > [...]
> >> commit dfbf2f35ec7cf6ac55b6579f813d03c2078795a5
> >> Author: Pavel Shilovsky <piastry@etersoft.ru>
> >> Date:   Thu Oct 18 12:57:28 2012 +0400
> >>
> >>     Process *.py_tmpl files properly
> >>
> > [...]
> >> --- a/rpm-build-python/python.req.py
> >> +++ b/rpm-build-python/python.req.py
> >> @@ -231,6 +231,8 @@ else :
> >>                  def must_fail() :
> >>                      if ext == '.py' :
> >>                          return True
> >> +                    if ext == '.py_tmpl':
> >> +                        return False
> >>                      if len(lines) > 0 and lines[0].startswith('#!') :
> >>                          return True
> >>                  try :
> >
> > Насколько я понимаю, суть этого изменения в том, что ошибки поиска
> > зависимостей в python-файлах с именами, заканчивающимися на .py_tmpl,
> > не будут считаться ошибками.  Просьба пояснить, зачем понадобилось такое
> > изменение, и почему оно правильное.
> 
> Данное изменение требуется для корректной сборки python-module-migrate
> (http://git.altlinux.org/people/piastry/packages/?p=python-module-migrate.git;a=summary),
> который отказывается собираться без данного патча, пишет:
> 
> python.req: ERROR:
> /home/piastry/tmp/python-module-migrate-buildroot/usr/lib/python2.7/site-packages/migrate/versioning/templates/manage/default.py_tmpl:
> invalid syntax (line 5)
> Traceback (most recent call last):
>   File "/usr/lib/rpm/python.req.py", line 237, in <module>
>     lis = parser.suite('\n'.join(lines)+'\n').tolist(line_info=1)
>   File "<string>", line 5
>     _vars = locals().copy()
>           ^
> SyntaxError: invalid syntax
> find-requires: ERROR: /usr/lib/rpm/python.req failed
> 
> что собственно говорит о том, что py_tmpl не соответствуют синтаксису
> python в полной мере - поэтому падать с ошибкой в данном случае не
> имеет смысла.

Скажите, пожалуйста, это общее свойство всех python-файлов с именами,
заканчивающимися на  *.py_tmpl, или это специфика пакета
python-module-migrate?


-- 
ldv

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

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

[Pavel Shilovsky]

30 октября 2012 г., 14:34 пользователь Dmitry V. Levin
<ldv@altlinux.org> написал:
> On Tue, Oct 30, 2012 at 12:32:30PM +0400, Pavel Shilovsky wrote:
>> 30 октября 2012 г., 11:50 пользователь Dmitry V. Levin
>> <ldv@altlinux.org> написал:
>> > On Thu, Oct 18, 2012 at 09:04:51AM +0000, Pavel Shilovsky wrote:
>> >> Update of /people/piastry/packages/rpm-build-python.git
>> > [...]
>> >> commit dfbf2f35ec7cf6ac55b6579f813d03c2078795a5
>> >> Author: Pavel Shilovsky <piastry@etersoft.ru>
>> >> Date:   Thu Oct 18 12:57:28 2012 +0400
>> >>
>> >>     Process *.py_tmpl files properly
>> >>
>> > [...]
>> >> --- a/rpm-build-python/python.req.py
>> >> +++ b/rpm-build-python/python.req.py
>> >> @@ -231,6 +231,8 @@ else :
>> >>                  def must_fail() :
>> >>                      if ext == '.py' :
>> >>                          return True
>> >> +                    if ext == '.py_tmpl':
>> >> +                        return False
>> >>                      if len(lines) > 0 and lines[0].startswith('#!') :
>> >>                          return True
>> >>                  try :
>> >
>> > Насколько я понимаю, суть этого изменения в том, что ошибки поиска
>> > зависимостей в python-файлах с именами, заканчивающимися на .py_tmpl,
>> > не будут считаться ошибками.  Просьба пояснить, зачем понадобилось такое
>> > изменение, и почему оно правильное.
>>
>> Данное изменение требуется для корректной сборки python-module-migrate
>> (http://git.altlinux.org/people/piastry/packages/?p=python-module-migrate.git;a=summary),
>> который отказывается собираться без данного патча, пишет:
>>
>> python.req: ERROR:
>> /home/piastry/tmp/python-module-migrate-buildroot/usr/lib/python2.7/site-packages/migrate/versioning/templates/manage/default.py_tmpl:
>> invalid syntax (line 5)
>> Traceback (most recent call last):
>>   File "/usr/lib/rpm/python.req.py", line 237, in <module>
>>     lis = parser.suite('\n'.join(lines)+'\n').tolist(line_info=1)
>>   File "<string>", line 5
>>     _vars = locals().copy()
>>           ^
>> SyntaxError: invalid syntax
>> find-requires: ERROR: /usr/lib/rpm/python.req failed
>>
>> что собственно говорит о том, что py_tmpl не соответствуют синтаксису
>> python в полной мере - поэтому падать с ошибкой в данном случае не
>> имеет смысла.
>
> Скажите, пожалуйста, это общее свойство всех python-файлов с именами,
> заканчивающимися на  *.py_tmpl, или это специфика пакета
> python-module-migrate?

Так, выяснил, что *.py_tmpl это произвольное расширение, выбранное в
migrate для tempita шаблонов. Поэтому, правильным будет в этом случае
воспользоваться способом, предложенным @thecrux, и исправить спек в
python-module-migrate.

Всем спасибо!

-- 
Best regards,
Pavel Shilovsky.