[devel] mailutils

[devel] mailutils

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

Привет!

http://git.altlinux.org/people/real/packages/mailutils.git

Могу сам собрать (ACL есть), а можете у меня смержить. Какой вариант
предпочтительней?

На всякий случай добавил объявление top_builddir:

From: Eugeny A. Rostovtsev <real@altlinux.org>
Date: Sun, 7 Feb 2010 20:03:09 +0000 (+0000)
Subject: mailutils/Makefile.am: add top_builddir definition
X-Git-Tag: 2.1-alt0.20091105.4~2
X-Git-Url: http://git.altlinux.org/people/real/packages/mailutils.git?p=mailutils.git;a=commitdiff_plain;h=491f84b765a1d2e6a25544ac84e6b2d73c989dfc

mailutils/Makefile.am: add top_builddir definition
---

diff --git a/mailutils/Makefile.am b/mailutils/Makefile.am
index 5cabed2..7c8461a 100644
--- a/mailutils/Makefile.am
+++ b/mailutils/Makefile.am
@@ -18,6 +18,8 @@
 ##   Foundation, Inc.  51 Franklin Street, Fifth Floor, Boston, MA
 ##   02110-1301 USA

+top_builddir = @top_builddir@
+
 ACLOCAL_AMFLAGS = -I m4 -I am

 if MU_COND_PYTHON


> Есть подозрение, что этот модуль - noarch.

Подозрение не подтвердилось.

Один из .la пришлось заменить на .so в конфиге.

From: Eugeny A. Rostovtsev <real@altlinux.org>
Date: Sun, 7 Feb 2010 20:05:04 +0000 (+0000)
Subject: mailutils/configure.ac: fix libmu_py (.la -> .so)
X-Git-Tag: 2.1-alt0.20091105.4~1
X-Git-Url: http://git.altlinux.org/people/real/packages/mailutils.git?p=mailutils.git;a=commitdiff_plain;h=27908f33a0ed7a2e0ab4e23b37851e7e2375321d

mailutils/configure.ac: fix libmu_py (.la -> .so)
---

diff --git a/mailutils/configure.ac b/mailutils/configure.ac
index e6151ce..c890204 100644
--- a/mailutils/configure.ac
+++ b/mailutils/configure.ac
@@ -37,6 +37,8 @@ AC_SUBST(VI_REVISION, 2)
 AC_SUBST(VI_AGE,      1)

 dnl Library paths
+top_builddir=@TOP@
+AC_SUBST(top_builddir)
 AC_SUBST(MU_LIB_AUTH,'${top_builddir}/libmu_auth/libmu_auth.la')
 AC_SUBST(MU_LIB_MAILUTILS,'${top_builddir}/mailbox/libmailutils.la')
 AC_SUBST(MU_LIB_SQL,'${top_builddir}/sql/libsql.la')
@@ -1135,7 +1137,7 @@ if test "$status_python" = yes; then

     if test "$status_python" = yes; then
       AC_DEFINE(WITH_PYTHON,1,[Enable Python support])
-      MU_LIB_PY='${top_builddir}/python/libmu_py/libmu_py.la'
+      MU_LIB_PY='${top_builddir}/python/libmu_py/.libs/libmu_py.so'
       AC_SUBST([PYTHON_SITE_DIR])
       AC_SUBST([PYTHON_EXEC_DIR])
       AC_ARG_WITH([pythondir],

И ещё это было косячное использование distutils

Там ещё aclocal дурил, пришлось подчищать.

From: Eugeny A. Rostovtsev <real@altlinux.org>
Date: Sun, 7 Feb 2010 20:11:52 +0000 (+0000)
Subject: 2.1-alt0.20091105.4
X-Git-Tag: 2.1-alt0.20091105.4^0
X-Git-Url: http://git.altlinux.org/people/real/packages/mailutils.git?p=mailutils.git;a=commitdiff_plain;h=0952b0f210796fe815b2a4125a25327edd0b16af

2.1-alt0.20091105.4

- Fixed get_python_lib using
- configure: .la war
---

diff --git a/mailutils.spec b/mailutils.spec
index 36c4e2f..b0268cf 100644
--- a/mailutils.spec
+++ b/mailutils.spec
@@ -7,7 +7,7 @@ Name: mailutils
 %if %snapshot
 %define snapshotdate 20091105
 Version: %baseversion
-Release: alt0.%snapshotdate.3
+Release: alt0.%snapshotdate.4
 %define srcdir %name-%snapshotdate
 %else
 Version: %baseversion
@@ -38,6 +38,7 @@ Provides: /bin/mail

 Requires: libmailutils = %{version}-%{release}, guile >= 1.8, libreadline
 BuildRequires: rpm-build-licenses
+BuildPreReq: /usr/bin/cvs

 # Automatically added by buildreq on Thu Mar 12 2009
 BuildRequires: emacs-X11 flex gcc-c++ guile18-devel libgmp-devel
libldap-devel libltdl-devel libncurses-devel libpam-devel
libreadline-devel libwrap-devel python-devel
@@ -209,15 +210,16 @@ Python bindings for GNU Mailutils.
 %endif

 %patch1 -p0
+sed -i "s|@TOP@|$PWD|" configure.ac

 gzip ChangeLog

 %build

-#autoreconf -f -i -s
+%autoreconf
+sed -i 's|get_python_lib[^)]*|get_python_lib(1|' aclocal.m4

 %set_verify_elf_method unresolved=relaxed
-%undefine __libtoolize

 CFLAGS="-ltinfo"
 %configure --enable-mh-utils --with-mh-bindir=%_libexecdir/mu-mh
@@ -356,6 +358,10 @@ rm -f $RPM_BUILD_ROOT%python_sitelibdir/mailutils/c_api.la
 %python_sitelibdir/mailutils/*.py*

 %changelog
+* Sun Feb 07 2010 Eugeny A. Rostovtsev (REAL) <real at altlinux.org>
2.1-alt0.20091105.4
+- Fixed get_python_lib using
+- configure: .la war
+
 * Sun Jan 31 2010 Sergey Y. Afonin <asy@altlinux.ru> 2.1-alt0.20091105.3
 - created symlinks (for better exchangeability with the mailx package):
   Mail -> mail

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

Re: [devel] mailutils

[Dmitry V. Levin]

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

On Mon, Feb 08, 2010 at 03:41:27AM +0700, Евгений Ростовцев wrote:
[...]
> X-Git-Url: http://git.altlinux.org/people/real/packages/mailutils.git?p=mailutils.git;a=commitdiff_plain;h=27908f33a0ed7a2e0ab4e23b37851e7e2375321d
> 
> mailutils/configure.ac: fix libmu_py (.la -> .so)
> ---
> 
> diff --git a/mailutils/configure.ac b/mailutils/configure.ac
> index e6151ce..c890204 100644
> --- a/mailutils/configure.ac
> +++ b/mailutils/configure.ac
> @@ -37,6 +37,8 @@ AC_SUBST(VI_REVISION, 2)
>  AC_SUBST(VI_AGE,      1)
> 
>  dnl Library paths
> +top_builddir=@TOP@
> +AC_SUBST(top_builddir)

Апстрим это не возьмёт, потому что сочтёт за грязный хак.

>  AC_SUBST(MU_LIB_AUTH,'${top_builddir}/libmu_auth/libmu_auth.la')
>  AC_SUBST(MU_LIB_MAILUTILS,'${top_builddir}/mailbox/libmailutils.la')
>  AC_SUBST(MU_LIB_SQL,'${top_builddir}/sql/libsql.la')
> @@ -1135,7 +1137,7 @@ if test "$status_python" = yes; then
> 
>      if test "$status_python" = yes; then
>        AC_DEFINE(WITH_PYTHON,1,[Enable Python support])
> -      MU_LIB_PY='${top_builddir}/python/libmu_py/libmu_py.la'
> +      MU_LIB_PY='${top_builddir}/python/libmu_py/.libs/libmu_py.so'

И это апстрим не возьмёт, по причине непереносимости.
Вообще странно, когда и почему это сломалось.

> +sed -i 's|get_python_lib[^)]*|get_python_lib(1|' aclocal.m4

И это апстрим не возьмёт, у него вообще такого файла aclocal.m4 нет в
репозитории.


-- 
ldv

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

Re: [devel] mailutils

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

Привет!

>>  dnl Library paths
>> +top_builddir=@TOP@
>> +AC_SUBST(top_builddir)
>
> Апстрим это не возьмёт, потому что сочтёт за грязный хак.

Подскажете, как сделать нормально, если на данный момент это считать
грязным хаком. Или после остальных правок это
+top_builddir=@TOP@
+AC_SUBST(top_builddir)
уже неактуально?

>> +      MU_LIB_PY='${top_builddir}/python/libmu_py/.libs/libmu_py.so'
>
> И это апстрим не возьмёт, по причине непереносимости.

А оно надо - переносимость? Если всё же надо, то как сделать лучше в этом месте?

>> +sed -i 's|get_python_lib[^)]*|get_python_lib(1|' aclocal.m4
>
> И это апстрим не возьмёт, у него вообще такого файла aclocal.m4 нет в
> репозитории.

А вот тут не уверен. aclocal.m4 в этом пакете генерируется %autoreconf.

PS. просто мой вариант работает, но улучшить его я был бы рад.

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

Re: [devel] mailutils

[Dmitry V. Levin]

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

On Mon, Feb 08, 2010 at 04:40:58AM +0700, Евгений Ростовцев wrote:
> Привет!
> 
> >>  dnl Library paths
> >> +top_builddir=@TOP@
> >> +AC_SUBST(top_builddir)
> >
> > Апстрим это не возьмёт, потому что сочтёт за грязный хак.
> 
> Подскажете, как сделать нормально,

Для того, чтобы подсказать, мне придётся сперва хотя бы посмотреть в
пакет.  Не получается посмотреть всё сразу.

> если на данный момент это считать
> грязным хаком. Или после остальных правок это
> +top_builddir=@TOP@
> +AC_SUBST(top_builddir)
> уже неактуально?

Это хак грязный по формальному определению: @TOP@ нигде не определён, и на
него нельзя ссылаться.

Ну и, конечно, он не нужен, поскольку переменная top_builddir определяется
автоматически, см. info autoconf-2.60 'Preset Output Variables'

> >> +      MU_LIB_PY='${top_builddir}/python/libmu_py/.libs/libmu_py.so'
> >
> > И это апстрим не возьмёт, по причине непереносимости.
> 
> А оно надо - переносимость? Если всё же надо, то как сделать лучше в этом месте?

Исправления хорошо бы заапстримить, а для этого они должны быть
переносимыми.  Скажите, отчего оно сломалось в этом месте, и почему
оно сломалось в этом месте только на x86-64?

> >> +sed -i 's|get_python_lib[^)]*|get_python_lib(1|' aclocal.m4
> >
> > И это апстрим не возьмёт, у него вообще такого файла aclocal.m4 нет в
> > репозитории.
> 
> А вот тут не уверен. aclocal.m4 в этом пакете генерируется %autoreconf.

Это значит, что в aclocal.m4 макрос, который вы исправляется (и который,
весьма вероятно, нуждается в исправлении), определён в каком-то другом
файле, возможно, даже не в пакете mailutils.

> PS. просто мой вариант работает, но улучшить его я был бы рад.

Конечно.


-- 
ldv

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

Re: [devel] mailutils

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

Привет!

>> если на данный момент это считать
>> грязным хаком. Или после остальных правок это
>> +top_builddir=@TOP@
>> +AC_SUBST(top_builddir)
>> уже неактуально?
>
> Это хак грязный по формальному определению: @TOP@ нигде не определён, и на
> него нельзя ссылаться.
>
> Ну и, конечно, он не нужен, поскольку переменная top_builddir определяется
> автоматически, см. info autoconf-2.60 'Preset Output Variables'

Понял.

> Исправления хорошо бы заапстримить, а для этого они должны быть
> переносимыми.  Скажите, отчего оно сломалось в этом месте, и почему
> оно сломалось в этом месте только на x86-64?

Не могу сказать. Пакет-то вообще не мой, я его только с новым питоном
пересобрал.

>> А вот тут не уверен. aclocal.m4 в этом пакете генерируется %autoreconf.
>
> Это значит, что в aclocal.m4 макрос, который вы исправляется (и который,
> весьма вероятно, нуждается в исправлении), определён в каком-то другом
> файле, возможно, даже не в пакете mailutils.

Да, точно, не в пакете. Тогда где искать-то? :)

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

Re: [devel] mailutils

[Sergey Y. Afonin]

On Monday 08 February 2010, Евгений Ростовцев wrote:

> > Исправления хорошо бы заапстримить, а для этого они должны
> > быть переносимыми.  Скажите, отчего оно сломалось в этом месте,
> > и почему оно сломалось в этом месте только на x86-64?
> 
> Не могу сказать. Пакет-то вообще не мой, я его только с новым
> питоном пересобрал.
 
А мне, честно говоря, в этом пакете не нужен (по крайней мере пока),
Питон. Собирал потому, что собиралось и, может, кому-то надо. В
принципе, могу отключить. С другой стороны, оно повод разобраться...

-- 
С уважением, Сергей Афонин
asy@altlinux.ru

Re: [devel] mailutils

[Sergey Y. Afonin]

On Monday 08 February 2010, Dmitry V. Levin wrote:

> > > И это апстрим не возьмёт, по причине непереносимости.
> > 
> > А оно надо - переносимость? Если всё же надо, то как сделать лучше в этом месте?
> 
> Исправления хорошо бы заапстримить, а для этого они должны быть
> переносимыми.  Скажите, отчего оно сломалось в этом месте, и почему
> оно сломалось в этом месте только на x86-64?
 
А что произошло с Питоном со времени 

24 ноября 2009 Eugeny A. Rostovtsev (REAL) <real at altlinux.org> 2.1-alt0.20091105.2
    * Rebuilt with python 2.6

?

И вижу ещё, что после было:

5 января 2010 Dmitry V. Levin <ldv at altlinux.org> 2:2.65-alt1

    * Updated to v2.65-19-g6c11abd (closes: #21744).
    * autoreconf: Added gtkdocize support.

Плюс немного подросла версия gcc...


-- 
С уважением, Сергей Афонин
asy@altlinux.ru

Re: [devel] mailutils

[Dmitry V. Levin]

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

On Mon, Feb 08, 2010 at 05:39:52AM +0700, Евгений Ростовцев wrote:
> >> А вот тут не уверен. aclocal.m4 в этом пакете генерируется %autoreconf.
> >
> > Это значит, что в aclocal.m4 макрос, который вы исправляется (и который,
> > весьма вероятно, нуждается в исправлении), определён в каком-то другом
> > файле, возможно, даже не в пакете mailutils.
> 
> Да, точно, не в пакете. Тогда где искать-то? :)

Макрос, который вы исправляете, называется AM_PATH_PYTHON и определён он,
соответственно, в automake.  Стало быть, и фиксить надо automake.
Видимо, это придётся сделать мне.


-- 
ldv

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

Re: [devel] mailutils

[Dmitry V. Levin]

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

On Mon, Feb 08, 2010 at 12:21:28AM +0300, Dmitry V. Levin wrote:
> On Mon, Feb 08, 2010 at 03:41:27AM +0700, Евгений Ростовцев wrote:
[...]
> > +sed -i 's|get_python_lib[^)]*|get_python_lib(1|' aclocal.m4
> 
> И это апстрим не возьмёт, у него вообще такого файла aclocal.m4 нет в
> репозитории.

В общем, я не выдержал и заглянул в mailutils.  Там, оказывается, не один
модуль python, а целых два: mailutils и libmu_py, причём первый -- чистый
noarch, а второй -- вполне себе python extension module на C.
Так что их просто нужно установить в разные каталоги, и соответствующие
записи в Makefile.am именно это и призваны сделать.

Судя по содержимому config.log, эти каталоги во время сборки определяются
правильно:
exec_prefix='/usr'
prefix='/usr'
pyexecdir='${exec_prefix}/lib64/python2.6/site-packages'
pythondir='${prefix}/lib/python2.6/site-packages'

Вывод: просто в mailutils.spec надо было распилить python-module-mailutils
и поправить пути:

--- a/mailutils.spec
+++ b/mailutils.spec
@@ -192,14 +192,25 @@ Group: System/Libraries
 The libraries for integration with Python
 
 %package -n python-module-mailutils
-Summary: GNU Mailutils: Python modules.
-Copyright: %gpl3plus
-Requires: libmailutils = %{version}-%{release}
-Requires: python >= 2.5
+Summary: A Python interface to Mailutils framework
+License: %gpl3plus
 Group: Networking/Mail
+BuildArch: noarch
+Requires: python-module-mailutils-api = %version-%release
 
 %description -n python-module-mailutils
-Python bindings for GNU Mailutils.
+This package contains Python bindings for GNU Mailutils.
+
+%package -n python-module-mailutils-api
+Summary: A Python interface to Mailutils framework, arch specific part
+License: %gpl3plus
+Group: Networking/Mail
+Requires: libmailutils = %version-%release
+Conflicts: python-module-mailutils < %version-%release
+
+%description -n python-module-mailutils-api
+This package contains architecture specific part of the
+python-module-mailutils.
 
 %prep
 %if %snapshot
@@ -351,9 +362,12 @@ rm -f $RPM_BUILD_ROOT%python_sitelibdir/mailutils/c_api.la
 %_libdir/libmu_py.so.*
 
 %files -n python-module-mailutils
-%dir %python_sitelibdir/mailutils
-%python_sitelibdir/mailutils/c_api.so
-%python_sitelibdir/mailutils/*.py*
+%dir %_target_libdir_noarch/python%__python_version/site-packages/mailutils/
+%_target_libdir_noarch/python%__python_version/site-packages/mailutils/*.py*
+
+%files -n python-module-mailutils-api
+%dir %python_sitelibdir/mailutils/
+%dir %python_sitelibdir/mailutils/*.so
 
 %changelog
 * Sun Jan 31 2010 Sergey Y. Afonin <asy@altlinux.ru> 2.1-alt0.20091105.3


P.S. mailutils не собирается с --nprocs=N при N>1, просьба заменить
%make_build на %make, оставить об этом комментарий и пожаловаться апстриму.


-- 
ldv

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

Re: [devel] mailutils

[Yuri N. Sedunov]

В Втр, 16/02/2010 в 04:02 +0300, Dmitry V. Levin пишет:
<skip>
> -%python_sitelibdir/mailutils/c_api.so
> -%python_sitelibdir/mailutils/*.py*
> +%dir %_target_libdir_noarch/python%__python_version/site-packages/mailutils/
> +%_target_libdir_noarch/python%__python_version/site-packages/mailutils/*.py*

А можно переопределить макрос %python_sitelibdir_noarch как 
%_target_libdir_noarch/python%__python_version/site-packages
чтоб не городить этот огород?

-- 
Yuri N. Sedunov

Re: [devel] mailutils

[Dmitry V. Levin]

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

On Tue, Feb 16, 2010 at 04:02:44AM +0300, Dmitry V. Levin wrote:
> +%files -n python-module-mailutils-api
> +%dir %python_sitelibdir/mailutils/
> +%dir %python_sitelibdir/mailutils/*.so

В последней строке %dir избыточный.


-- 
ldv

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

Re: [devel] %python_sitelibdir_noarch

[Dmitry V. Levin]

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

On Tue, Feb 16, 2010 at 04:38:53AM +0300, Yuri N. Sedunov wrote:
> В Втр, 16/02/2010 в 04:02 +0300, Dmitry V. Levin пишет:
> <skip>
> > -%python_sitelibdir/mailutils/c_api.so
> > -%python_sitelibdir/mailutils/*.py*
> > +%dir %_target_libdir_noarch/python%__python_version/site-packages/mailutils/
> > +%_target_libdir_noarch/python%__python_version/site-packages/mailutils/*.py*
> 
> А можно переопределить макрос %python_sitelibdir_noarch как 
> %_target_libdir_noarch/python%__python_version/site-packages
> чтоб не городить этот огород?

Отчего же не определить новый макрос %python_sitelibdir_noarch?
Можно, наверное.


-- 
ldv

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

Re: [devel] %python_sitelibdir_noarch

[Yuri N. Sedunov]

В Втр, 16/02/2010 в 05:33 +0300, Dmitry V. Levin пишет:
> On Tue, Feb 16, 2010 at 04:38:53AM +0300, Yuri N. Sedunov wrote:
> > В Втр, 16/02/2010 в 04:02 +0300, Dmitry V. Levin пишет:
> > <skip>
> > > -%python_sitelibdir/mailutils/c_api.so
> > > -%python_sitelibdir/mailutils/*.py*
> > > +%dir %_target_libdir_noarch/python%__python_version/site-packages/mailutils/
> > > +%_target_libdir_noarch/python%__python_version/site-packages/mailutils/*.py*
> > 
> > А можно переопределить макрос %python_sitelibdir_noarch как 
> > %_target_libdir_noarch/python%__python_version/site-packages
> > чтоб не городить этот огород?
> 
> Отчего же не определить новый макрос %python_sitelibdir_noarch?
> Можно, наверное.

Он определен, но через _libexecdir, что вроде бы одно и тоже, 
что и _target_libdir_noarch
$ rpm --showrc|grep sitelibdir_noarch
-14: python_sitelibdir_noarch	%_libexecdir/python%
__python_version/site-packages

-- 
Yuri N. Sedunov

Re: [devel] mailutils

[Dmitry V. Levin]

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

On Tue, Feb 16, 2010 at 04:02:44AM +0300, Dmitry V. Levin wrote:
[...]
> Вывод: просто в mailutils.spec надо было распилить python-module-mailutils
> и поправить пути:

Альтернативный вариант исправления -- перенести всё в arch specific.
Выглядит проще, но зато не upstreamable.

--- a/mailutils.spec
+++ b/mailutils.spec
@@ -39,6 +39,9 @@ Provides: /bin/mail
 Requires: libmailutils = %{version}-%{release}, guile >= 1.8, libreadline
 BuildRequires: rpm-build-licenses
 
+# due to autopoint
+BuildRequires: cvs
+
 # Automatically added by buildreq on Thu Mar 12 2009
 BuildRequires: emacs-X11 flex gcc-c++ guile18-devel libgmp-devel libldap-devel libltdl-devel libncurses-devel libpam-devel libreadline-devel libwrap-devel python-devel
 
@@ -192,14 +195,13 @@ Group: System/Libraries
 The libraries for integration with Python
 
 %package -n python-module-mailutils
-Summary: GNU Mailutils: Python modules.
-Copyright: %gpl3plus
-Requires: libmailutils = %{version}-%{release}
-Requires: python >= 2.5
+Summary: A Python interface to Mailutils framework
+License: %gpl3plus
 Group: Networking/Mail
+Requires: libmailutils = %version-%release
 
 %description -n python-module-mailutils
-Python bindings for GNU Mailutils.
+This package contains Python bindings for GNU Mailutils.
 
 %prep
 %if %snapshot
@@ -213,24 +215,20 @@ Python bindings for GNU Mailutils.
 gzip ChangeLog
 
 %build
-
-#autoreconf -f -i -s
-
+%autoreconf
 %set_verify_elf_method unresolved=relaxed
-%undefine __libtoolize
 
-CFLAGS="-ltinfo"
+LDFLAGS="-ltinfo"
 %configure --enable-mh-utils --with-mh-bindir=%_libexecdir/mu-mh
 
-%make_build
+# SMP-incompatible build
+%make
 
 %check
-
 %make check
 
 %install
-
-make install DESTDIR=%buildroot
+%makeinstall_std
 
 install -d $RPM_BUILD_ROOT/bin
 ln -s ..%_bindir/mail "$RPM_BUILD_ROOT/bin/mail"
@@ -351,9 +349,7 @@ rm -f $RPM_BUILD_ROOT%python_sitelibdir/mailutils/c_api.la
 %_libdir/libmu_py.so.*
 
 %files -n python-module-mailutils
-%dir %python_sitelibdir/mailutils
-%python_sitelibdir/mailutils/c_api.so
-%python_sitelibdir/mailutils/*.py*
+%python_sitelibdir/mailutils/
 
 %changelog
 * Sun Jan 31 2010 Sergey Y. Afonin <asy@altlinux.ru> 2.1-alt0.20091105.3
--- a/mailutils/python/mailutils/Makefile.am
+++ b/mailutils/python/mailutils/Makefile.am
@@ -17,8 +17,8 @@
 ##   Foundation, Inc.  51 Franklin Street, Fifth Floor, Boston, MA
 ##   02110-1301 USA
 
-pythonsitedir=@PYTHON_SITE_DIR@
-pythonsite_PYTHON=\
+pythonexecdir=@PYTHON_SITE_DIR@
+pythonexec_PYTHON=\
  __init__.py \
  error.py \
  address.py \


-- 
ldv

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

Re: [devel] %python_sitelibdir_noarch

[Dmitry V. Levin]

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

On Tue, Feb 16, 2010 at 05:46:31AM +0300, Yuri N. Sedunov wrote:
> В Втр, 16/02/2010 в 05:33 +0300, Dmitry V. Levin пишет:
> > On Tue, Feb 16, 2010 at 04:38:53AM +0300, Yuri N. Sedunov wrote:
> > > В Втр, 16/02/2010 в 04:02 +0300, Dmitry V. Levin пишет:
> > > <skip>
> > > > -%python_sitelibdir/mailutils/c_api.so
> > > > -%python_sitelibdir/mailutils/*.py*
> > > > +%dir %_target_libdir_noarch/python%__python_version/site-packages/mailutils/
> > > > +%_target_libdir_noarch/python%__python_version/site-packages/mailutils/*.py*
> > > 
> > > А можно переопределить макрос %python_sitelibdir_noarch как 
> > > %_target_libdir_noarch/python%__python_version/site-packages
> > > чтоб не городить этот огород?
> > 
> > Отчего же не определить новый макрос %python_sitelibdir_noarch?
> > Можно, наверное.
> 
> Он определен, но через _libexecdir, что вроде бы одно и тоже, 
> что и _target_libdir_noarch
> $ rpm --showrc|grep sitelibdir_noarch
> -14: python_sitelibdir_noarch	%_libexecdir/python%
> __python_version/site-packages

А, это совсем свежий макрос.  Ну да, %_libexecdir тут совсем не к месту,
должен быть %_target_libdir_noarch.

Rationale: В некоторых пакетах %_libexecdir переопределён в /usr/libexec,
однако значение %python_sitelibdir_noarch не должно из-за этого меняться.


-- 
ldv

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

Re: [devel] %python_sitelibdir_noarch

[Yuri N. Sedunov]

В Втр, 16/02/2010 в 05:51 +0300, Dmitry V. Levin пишет:
> On Tue, Feb 16, 2010 at 05:46:31AM +0300, Yuri N. Sedunov wrote:
> > В Втр, 16/02/2010 в 05:33 +0300, Dmitry V. Levin пишет:
> > > On Tue, Feb 16, 2010 at 04:38:53AM +0300, Yuri N. Sedunov wrote:
> > > > В Втр, 16/02/2010 в 04:02 +0300, Dmitry V. Levin пишет:
> > > > <skip>
> > > > > -%python_sitelibdir/mailutils/c_api.so
> > > > > -%python_sitelibdir/mailutils/*.py*
> > > > > +%dir %_target_libdir_noarch/python%__python_version/site-packages/mailutils/
> > > > > +%_target_libdir_noarch/python%__python_version/site-packages/mailutils/*.py*
> > > > 
> > > > А можно переопределить макрос %python_sitelibdir_noarch как 
> > > > %_target_libdir_noarch/python%__python_version/site-packages
> > > > чтоб не городить этот огород?
> > > 
> > > Отчего же не определить новый макрос %python_sitelibdir_noarch?
> > > Можно, наверное.
> > 
> > Он определен, но через _libexecdir, что вроде бы одно и тоже, 
> > что и _target_libdir_noarch
> > $ rpm --showrc|grep sitelibdir_noarch
> > -14: python_sitelibdir_noarch	%_libexecdir/python%
> > __python_version/site-packages
> 
> А, это совсем свежий макрос.  Ну да, %_libexecdir тут совсем не к месту,
> должен быть %_target_libdir_noarch.
> 
> Rationale: В некоторых пакетах %_libexecdir переопределён в /usr/libexec,
> однако значение %python_sitelibdir_noarch не должно из-за этого меняться.

Да, я на это уже напоролся.

-- 
Yuri N. Sedunov

Re: [devel] %python_sitelibdir_noarch

[Dmitry V. Levin]

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

On Tue, Feb 16, 2010 at 05:58:16AM +0300, Yuri N. Sedunov wrote:
> В Втр, 16/02/2010 в 05:51 +0300, Dmitry V. Levin пишет:
> > On Tue, Feb 16, 2010 at 05:46:31AM +0300, Yuri N. Sedunov wrote:
> > > В Втр, 16/02/2010 в 05:33 +0300, Dmitry V. Levin пишет:
> > > > On Tue, Feb 16, 2010 at 04:38:53AM +0300, Yuri N. Sedunov wrote:
> > > > > В Втр, 16/02/2010 в 04:02 +0300, Dmitry V. Levin пишет:
> > > > > <skip>
> > > > > > -%python_sitelibdir/mailutils/c_api.so
> > > > > > -%python_sitelibdir/mailutils/*.py*
> > > > > > +%dir %_target_libdir_noarch/python%__python_version/site-packages/mailutils/
> > > > > > +%_target_libdir_noarch/python%__python_version/site-packages/mailutils/*.py*
> > > > > 
> > > > > А можно переопределить макрос %python_sitelibdir_noarch как 
> > > > > %_target_libdir_noarch/python%__python_version/site-packages
> > > > > чтоб не городить этот огород?
> > > > 
> > > > Отчего же не определить новый макрос %python_sitelibdir_noarch?
> > > > Можно, наверное.
> > > 
> > > Он определен, но через _libexecdir, что вроде бы одно и тоже, 
> > > что и _target_libdir_noarch
> > > $ rpm --showrc|grep sitelibdir_noarch
> > > -14: python_sitelibdir_noarch	%_libexecdir/python%
> > > __python_version/site-packages
> > 
> > А, это совсем свежий макрос.  Ну да, %_libexecdir тут совсем не к месту,
> > должен быть %_target_libdir_noarch.
> > 
> > Rationale: В некоторых пакетах %_libexecdir переопределён в /usr/libexec,
> > однако значение %python_sitelibdir_noarch не должно из-за этого меняться.
> 
> Да, я на это уже напоролся.

OK, fixed in rpm-build-python-0.34.4-alt8 (task #20230).


-- 
ldv

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

Re: [devel] mailutils

[Dmitry V. Levin]

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

On Tue, Feb 16, 2010 at 05:47:33AM +0300, Dmitry V. Levin wrote:
> On Tue, Feb 16, 2010 at 04:02:44AM +0300, Dmitry V. Levin wrote:
> [...]
> > Вывод: просто в mailutils.spec надо было распилить python-module-mailutils
> > и поправить пути:
> 
> Альтернативный вариант исправления -- перенести всё в arch specific.
> Выглядит проще, но зато не upstreamable.

Впрочем, перенести файлы можно и более простым способом.


-- 
ldv

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

Re: [devel] mailutils

[Sergey Y. Afonin]

On Tuesday 16 February 2010, Dmitry V. Levin wrote:

> > > +sed -i 's|get_python_lib[^)]*|get_python_lib(1|' aclocal.m4
> > 
> > И это апстрим не возьмёт, у него вообще такого файла aclocal.m4
> > нет в репозитории.
> 
> В общем, я не выдержал и заглянул в mailutils.  Там, оказывается,
> не один 

Всем спасибо большое. К сожалению, сам в этом поучаствовать не
смог - что-то то одно, то другое, а Питон, как я уже писал, штука
для меня несколько сбоку... Попробую выходные всё это осмыслить и
что-нибудь сделать c пакетом.

-- 
С уважением, Сергей Афонин
asy@altlinux.ru

Re: [devel] mailutils

[Dmitry V. Levin]

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

On Tue, Feb 16, 2010 at 02:45:58PM +0400, Sergey Y. Afonin wrote:
> On Tuesday 16 February 2010, Dmitry V. Levin wrote:
> 
> > > > +sed -i 's|get_python_lib[^)]*|get_python_lib(1|' aclocal.m4
> > > 
> > > И это апстрим не возьмёт, у него вообще такого файла aclocal.m4
> > > нет в репозитории.
> > 
> > В общем, я не выдержал и заглянул в mailutils.  Там, оказывается,
> > не один 
> 
> Всем спасибо большое. К сожалению, сам в этом поучаствовать не
> смог - что-то то одно, то другое, а Питон, как я уже писал, штука
> для меня несколько сбоку... Попробую выходные всё это осмыслить и
> что-нибудь сделать c пакетом.

В общем, я запушил необходимые изменения в свой git (письмо вам пришло),
а дальше вы уже смотрите сами.


-- 
ldv

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