* Re: [devel] [SCM] packages/rpm-build-python: heads/master
@ 2012-10-30 7:50 ` Dmitry V. Levin
2012-10-30 8:32 ` Pavel Shilovsky
0 siblings, 1 reply; 5+ messages in thread
From: Dmitry V. Levin @ 2012-10-30 7:50 UTC (permalink / raw)
To: ALT Devel discussion list
[-- 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 --]
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [devel] [SCM] packages/rpm-build-python: heads/master
2012-10-30 7:50 ` [devel] [SCM] packages/rpm-build-python: heads/master Dmitry V. Levin
@ 2012-10-30 8:32 ` Pavel Shilovsky
2012-10-30 9:41 ` thecrux
2012-10-30 10:34 ` Dmitry V. Levin
0 siblings, 2 replies; 5+ messages in thread
From: Pavel Shilovsky @ 2012-10-30 8:32 UTC (permalink / raw)
To: ALT Linux Team development discussions
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.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [devel] [SCM] packages/rpm-build-python: heads/master
2012-10-30 8:32 ` Pavel Shilovsky
@ 2012-10-30 9:41 ` thecrux
2012-10-30 10:34 ` Dmitry V. Levin
1 sibling, 0 replies; 5+ messages in thread
From: thecrux @ 2012-10-30 9:41 UTC (permalink / raw)
To: Pavel Shilovsky; +Cc: ALT Linux Team development discussions
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
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [devel] [SCM] packages/rpm-build-python: heads/master
2012-10-30 8:32 ` Pavel Shilovsky
2012-10-30 9:41 ` thecrux
@ 2012-10-30 10:34 ` Dmitry V. Levin
2012-10-30 11:16 ` Pavel Shilovsky
1 sibling, 1 reply; 5+ messages in thread
From: Dmitry V. Levin @ 2012-10-30 10:34 UTC (permalink / raw)
To: ALT Linux Team development discussions
[-- 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 --]
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [devel] [SCM] packages/rpm-build-python: heads/master
2012-10-30 10:34 ` Dmitry V. Levin
@ 2012-10-30 11:16 ` Pavel Shilovsky
0 siblings, 0 replies; 5+ messages in thread
From: Pavel Shilovsky @ 2012-10-30 11:16 UTC (permalink / raw)
To: ALT Linux Team development discussions
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.
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2012-10-30 11:16 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-10-30 7:50 ` [devel] [SCM] packages/rpm-build-python: heads/master Dmitry V. Levin
2012-10-30 8:32 ` Pavel Shilovsky
2012-10-30 9:41 ` thecrux
2012-10-30 10:34 ` Dmitry V. Levin
2012-10-30 11:16 ` Pavel Shilovsky
ALT Linux Team development discussions
This inbox may be cloned and mirrored by anyone:
git clone --mirror http://lore.altlinux.org/devel/0 devel/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 devel devel/ http://lore.altlinux.org/devel \
devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru
public-inbox-index devel
Example config snippet for mirrors.
Newsgroup available over NNTP:
nntp://lore.altlinux.org/org.altlinux.lists.devel
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git