25368 2011-04-03 08:55:59 +0400 Добавить в lua.pc INSTALL_CMOD и др. 2011-05-04 21:36:39 +0400 1 1 4 Development Sisyphus liblua5-devel unstable all Linux CLOSED FIXED http://git.altlinux.org/people/evg/packages/?p=lua5.git;a=blobdiff;f=etc/lua.pc;h=c3e9750e0ab89b7c1e335af5b2899867ec1547c4;hp=5d4fe94dc38ec072dd703e6a76a45cccb13f1cad;hb=520ce4fd483966a57bd9672460b5edecfb1e0bf8;hpb=e18b106b76ddc8607c3f13efc3f1cb72a5536290 P3 normal --- 1 evg vseleznv ildar real.altlinux.org viy qa-sisyphus oldest_to_newest 119813 0 4860 evg 2011-04-03 08:55:59 +0400 Created attachment 4860 PC-файл из другого дистрибутива Столкнулся с LUA-программами, при сборке спрашивающими у lua.pc , куда им ставить динамически подгружаемые .so-файлы (а именно, запрашивается переменная INSTALL_CMOD). Однако наш аскетичный lua.pc подобной информации не содержит. Анализ актуальных lua*.pc на основных rpm-based дистрибутивах показал, что все (!) они предоставляют гораздо больше информации, чем наш. Хотелось бы иметь возможность собирать lua-пакеты, запрашивающие такую информацию, без кувалды. Для этого нужно (как я понял) на этапе сборки lua внести в pc-файл дополнительную build-time информацию. Не думаю, что это изменение способно что-то сломать. Для примера прилагаю pc-файл из другого дистрибутива. 119853 1 ildar 2011-04-04 08:12:52 +0400 Кроме этого, прошу явно провайдить соответствующие папки. На данный момент это %_libdir/lua5 и %_datadir/lua5 Обоснование: для того же, для чего это делают в других пакетах: чтобы пакеты, ставящие соотв. модули, были уверены, что ставятся в правильное место (т.е. неявная зависимость на владельца соотв. папок) 119856 2 real.altlinux.org 2011-04-04 09:44:08 +0400 Я не понял. В pkgconfig-файле сказано: INSTALL_CMOD= ${prefix}/lib/lua/${V} А Ильдар говорит про %_libdir/lua5. Так куда именно ставить нужно? Или сейчас это (%_libdir/lua5) предполагается как симлинк оформить? Если так, то в чём смысл что-то исправлять в других пакетах? 119857 3 ildar 2011-04-04 09:59:13 +0400 (В ответ на комментарий №2) > Я не понял. В pkgconfig-файле сказано: > INSTALL_CMOD= ${prefix}/lib/lua/${V} Читайте внимательно: (комментарий №0) > Для примера прилагаю pc-файл из другого дистрибутива. Суть бага в том, чтобы положить в пакет обновлённый lua.pc, в котором указан правильный текущий INSTALL_CMOD= Сейчас это ${prefix}/lib/lua5 119858 4 real.altlinux.org 2011-04-04 10:25:07 +0400 "Читайте внимательно: (комментарий №0)" Может быть, имелся в виду №1? ;) В общем, ждём реакции ldv@ или at@... 120990 5 evg 2011-05-02 19:18:22 +0400 ping? 121000 6 ildar 2011-05-03 08:22:34 +0400 (В ответ на комментарий №5) > ping? 1. у нас нет достаточно заинтересованного мэйнтейнера lua 2. ldv@ в рассылке намекнул, что готов посмотреть готовый патч в GIT-е вопрошающего Если я что-то неправильно написал, прошу меня поправить. 121038 7 evg 2011-05-03 21:00:33 +0400 Патч по сути бага. Следующим коммитом - сборка с запрошенными явными Provides:. Прошу посмотреть/покритиковать/смержить/дать ACL/etc 121039 8 evg 2011-05-03 21:08:17 +0400 (У меня работает, что хотелось собирается с данным .pc без вопросов). 121050 9 ldv 2011-05-04 02:38:51 +0400 (In reply to comment #0) > Created an attachment (id=4860) [details] > PC-файл из другого дистрибутива Из какого? > Столкнулся с LUA-программами, при сборке спрашивающими у lua.pc , куда им > ставить динамически подгружаемые .so-файлы (а именно, запрашивается переменная > INSTALL_CMOD). Однако наш аскетичный lua.pc подобной информации не содержит. > > Анализ актуальных lua*.pc на основных rpm-based дистрибутивах показал, что все > (!) они предоставляют гораздо больше информации, чем наш. Я сейчас посмотрел федорный пакет. Выходит, что они удаляют эту информацию пачем http://pkgs.fedoraproject.org/gitweb/?p=lua.git;a=blob;f=lua-5.1.4-autotoolize.patch Они что, враги себе, или эта информация на самом деле не нужна? > Хотелось бы иметь > возможность собирать lua-пакеты, запрашивающие такую информацию, без кувалды. > Для этого нужно (как я понял) на этапе сборки lua внести в pc-файл > дополнительную build-time информацию. Не думаю, что это изменение способно > что-то сломать. > > Для примера прилагаю pc-файл из другого дистрибутива. > +INSTALL_BIN=${prefix}/bin > +INSTALL_INC=${prefix}/include > +INSTALL_LIB=${prefix}/lib > +INSTALL_MAN=${prefix}/share/man/man1 Я полагаю, что все вышеперечисленное не нужно, а INSTALL_LIB к тому же еще и неправильно. В debian определяют только INSTALL_LMOD и INSTALL_CMOD, предлагаю и нам ограничиться этими переменными. 121052 10 evg 2011-05-04 04:05:11 +0400 (В ответ на комментарий №9) > (In reply to comment #0) > > Created an attachment (id=4860) [details] [details] > > PC-файл из другого дистрибутива > Из какого? Такие lua.pc я вижу в свежих сборках lua-5.1.4 в PCLinuxOS, Mandriba, OpenSuSe. > > Столкнулся с LUA-программами, при сборке спрашивающими у lua.pc , куда им > > ставить динамически подгружаемые .so-файлы (а именно, запрашивается переменная > > INSTALL_CMOD). Однако наш аскетичный lua.pc подобной информации не содержит. > > > > Анализ актуальных lua*.pc на основных rpm-based дистрибутивах показал, что все > > (!) они предоставляют гораздо больше информации, чем наш. > Я сейчас посмотрел федорный пакет. Выходит, что они удаляют эту информацию > пачем > http://pkgs.fedoraproject.org/gitweb/?p=lua.git;a=blob;f=lua-5.1.4-autotoolize.patch > Они что, враги себе, или эта информация на самом деле не нужна? Подобные спеки я вижу в FC и PLD. Думаю, сборку каждого пакета, хотящего больше от lua.pc, им пришлось запатчить. Стоит ли и нам так делать? > > Хотелось бы иметь > > возможность собирать lua-пакеты, запрашивающие такую информацию, без кувалды. > > Для этого нужно (как я понял) на этапе сборки lua внести в pc-файл > > дополнительную build-time информацию. Не думаю, что это изменение способно > > что-то сломать. > > Для примера прилагаю pc-файл из другого дистрибутива. > > +INSTALL_BIN=${prefix}/bin > > +INSTALL_INC=${prefix}/include > > +INSTALL_LIB=${prefix}/lib > > +INSTALL_MAN=${prefix}/share/man/man1 > Я полагаю, что все вышеперечисленное не нужно, а INSTALL_LIB к тому же еще и > неправильно. Кому? Почему? > В debian определяют только INSTALL_LMOD и INSTALL_CMOD, предлагаю и нам > ограничиться этими переменными. Я добавил больше информации, чем нужно было сейчас мне, ориентируясь на сборки в других дистрибутивах (не FC/PLD), чтобы потом лишний раз маинтайнеру не вставать. Но если он хочет - мне пока этих двух переменных хватит. 121053 11 ldv 2011-05-04 05:09:35 +0400 (In reply to comment #10) > (В ответ на комментарий №9) > > > +INSTALL_BIN=${prefix}/bin > > > +INSTALL_INC=${prefix}/include > > > +INSTALL_LIB=${prefix}/lib > > > +INSTALL_MAN=${prefix}/share/man/man1 > > Я полагаю, что все вышеперечисленное не нужно, а INSTALL_LIB к тому же еще и > > неправильно. > > Кому? Почему? Все вышеперечисленное -- это стандартные каталоги ОС, и не lua.pc определять, где они находятся. INSTALL_LIB неправильный потому, что %_libdir не всегда совпадает с %_prefix/lib. > > В debian определяют только INSTALL_LMOD и INSTALL_CMOD, предлагаю и нам > > ограничиться этими переменными. > > Я добавил больше информации, чем нужно было сейчас мне, ориентируясь на сборки > в других дистрибутивах (не FC/PLD), чтобы потом лишний раз маинтайнеру не > вставать. Но если он хочет - мне пока этих двух переменных хватит. Я думаю, что добавлять lua-специфичные переменные для не-lua-специфичных каталогов не стоит. 121054 12 evg 2011-05-04 05:58:41 +0400 Понятно. Делаю другой патч. 121069 13 evg 2011-05-04 17:42:19 +0400 Новый патч. Ляп с libdir починен в спеке (следующий коммит). 121076 14 ldv 2011-05-04 18:27:40 +0400 (In reply to comment #13) > Новый патч. Ляп с libdir починен в спеке (следующий коммит). Fine with me. 121080 15 evg 2011-05-04 19:35:44 +0400 merge/acl/что-то ещё? 121086 16 repository-robot 2011-05-04 21:17:44 +0400 lua5-5.1.4-alt5 -> sisyphus: * Tue May 03 2011 Terechkov Evgenii <evg@altlinux> 5.1.4-alt5 - Add more info in lua.pc (ALT#25368) 121094 17 evg 2011-05-04 21:36:39 +0400 Ok 4860 2011-04-03 08:55:59 +0400 2011-04-03 08:55:59 +0400 PC-файл из другого дистрибутива lua.pc application/octet-stream 614 evg IyBsdWEucGMgLS0gcGtnLWNvbmZpZyBkYXRhIGZvciBMdWEKCiMgdmFycyBmcm9tIGluc3RhbGwg TWFrZWZpbGUKCiMgZ3JlcCAnXlY9JyAuLi9NYWtlZmlsZQpWPSA1LjEKIyBncmVwICdeUj0nIC4u L01ha2VmaWxlClI9IDUuMS40CgojIGdyZXAgJ15JTlNUQUxMXy4qPScgLi4vTWFrZWZpbGUgfCBz ZWQgJ3MvSU5TVEFMTF9UT1AvcHJlZml4LycKcHJlZml4PSAvdXNyCklOU1RBTExfQklOPSAke3By ZWZpeH0vYmluCklOU1RBTExfSU5DPSAke3ByZWZpeH0vaW5jbHVkZQpJTlNUQUxMX0xJQj0gJHtw cmVmaXh9L2xpYgpJTlNUQUxMX01BTj0gJHtwcmVmaXh9L21hbi9tYW4xCklOU1RBTExfTE1PRD0g JHtwcmVmaXh9L3NoYXJlL2x1YS8ke1Z9CklOU1RBTExfQ01PRD0gJHtwcmVmaXh9L2xpYi9sdWEv JHtWfQoKIyBjYW5vbmljYWwgdmFycwpleGVjX3ByZWZpeD0ke3ByZWZpeH0KaW5jbHVkZWRpcj0k e3ByZWZpeH0vaW5jbHVkZQoKTmFtZTogTHVhCkRlc2NyaXB0aW9uOiBBbiBFeHRlbnNpYmxlIEV4 dGVuc2lvbiBMYW5ndWFnZQpWZXJzaW9uOiAke1J9ClJlcXVpcmVzOiAKTGliczogLWxsdWEgLWxt CkNmbGFnczogLUkke2luY2x1ZGVkaXJ9CgojIChlbmQgb2YgbHVhLnBjKQo=