* [newbies] проблемы с find_requires @ 2010-11-17 15:37 Anton Chernyshov 2010-11-17 15:46 ` Alexey I. Froloff 2010-11-17 15:49 ` Андрей Черепанов 0 siblings, 2 replies; 11+ messages in thread From: Anton Chernyshov @ 2010-11-17 15:37 UTC (permalink / raw) To: devel@ where you _can_ ask Всем привет! Собрал новую версию preload. В результате после сборки sisyphus_check ругался на вот такое: /.out/preload-0.6.4-alt1.x86_64.rpm: forbidden requires: /etc/rc.d/init.d(daemon) /etc/rc.d/init.d(killproc) sisyphus_check: check-deps ERROR: package dependencies violation Проблему сборки я "решил" добавлением в начало спека: %add_findreq_skiplist /etc/rc.d/init.d/preload Но у меня очень серьезные подозрения, что так делать неправильно. Просьба к более опытным майнтейнерам, если не трудно, прокомментируйте, пожалуйста. Спек-файл можно посмотреть здесь - http://git.altlinux.org/people/ach/packages/?p=preload.git;a=blob;f=preload.spec;h=2a3ac2c4414e62bd88be9caecd44f810b5cb0785;hb=3eb081a807ef65b3e5b749b0e99343deea259f57 -- С уважением, Чернышов Антон ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [newbies] проблемы с find_requires 2010-11-17 15:37 [newbies] проблемы с find_requires Anton Chernyshov @ 2010-11-17 15:46 ` Alexey I. Froloff 2010-11-17 15:49 ` Андрей Черепанов 1 sibling, 0 replies; 11+ messages in thread From: Alexey I. Froloff @ 2010-11-17 15:46 UTC (permalink / raw) To: devel-newbies [-- Attachment #1: Type: text/plain, Size: 416 bytes --] On Wed, Nov 17, 2010 at 06:37:05PM +0300, Anton Chernyshov wrote: > Проблему сборки я "решил" добавлением в начало спека: Не делайте так никогда. > Но у меня очень серьезные подозрения, что так делать неправильно. > Просьба к более опытным майнтейнерам, если не трудно, > прокомментируйте, пожалуйста. Вы зря выкинули все патчи. -- Regards, -- Sir Raorn. --- http://thousandsofhate.blogspot.com/ [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 836 bytes --] ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [newbies] проблемы с find_requires 2010-11-17 15:37 [newbies] проблемы с find_requires Anton Chernyshov 2010-11-17 15:46 ` Alexey I. Froloff @ 2010-11-17 15:49 ` Андрей Черепанов 2010-11-17 15:52 ` Alexey I. Froloff ` (2 more replies) 1 sibling, 3 replies; 11+ messages in thread From: Андрей Черепанов @ 2010-11-17 15:49 UTC (permalink / raw) To: devel@ where you _can_ ask 17 ноября 2010 Anton Chernyshov написал: > Всем привет! > > Собрал новую версию preload. В результате после сборки sisyphus_check > ругался на вот такое: > > /.out/preload-0.6.4-alt1.x86_64.rpm: forbidden requires: > /etc/rc.d/init.d(daemon) /etc/rc.d/init.d(killproc) > sisyphus_check: check-deps ERROR: package dependencies violation > > Проблему сборки я "решил" добавлением в начало спека: > > %add_findreq_skiplist /etc/rc.d/init.d/preload > > Но у меня очень серьезные подозрения, что так делать неправильно. > Просьба к более опытным майнтейнерам, если не трудно, > прокомментируйте, пожалуйста. > > Спек-файл можно посмотреть здесь - > http://git.altlinux.org/people/ach/packages/?p=preload.git;a=blob;f=preload > .spec;h=2a3ac2c4414e62bd88be9caecd44f810b5cb0785;hb=3eb081a807ef65b3e5b749b > 0e99343deea259f57 Антон, придётся публично разьяснять (хотя ты мог бы и меня, как ментора спросить): Пакет сложный, зря ты за него взялся. Весь спек-файл ужасный. 1. Для начала запусти buildreq для получения сборочных зависимостей. А то glib2 явно прибит, хотя вытягивается glib2-devel 2. Ты уверен, что лицензия такая? 3. Зачем явно писал # install pdf documentation install -D --mode=0644 %SOURCE1 %buildroot/%_defaultdocdir/%name-%version/%name.pdf # install other upstream documentation for i in README AUTHORS ChangeLog TODO THANKS NEWS ; do cp $i %buildroot/%_defaultdocdir/%name-%version/ done ? В %files проще было прописать: %doc %SOURCE1 README AUTHORS ChangeLog TODO THANKS NEWS -- Андрей Черепанов ALT Linux cas@altlinux.ru ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [newbies] проблемы с find_requires 2010-11-17 15:49 ` Андрей Черепанов @ 2010-11-17 15:52 ` Alexey I. Froloff 2010-11-17 16:05 ` Dmitry V. Levin 2010-11-17 16:09 ` Anton Chernyshov 2 siblings, 0 replies; 11+ messages in thread From: Alexey I. Froloff @ 2010-11-17 15:52 UTC (permalink / raw) To: devel-newbies [-- Attachment #1: Type: text/plain, Size: 375 bytes --] On Wed, Nov 17, 2010 at 06:49:52PM +0300, Андрей Черепанов wrote: > Антон, придётся публично разьяснять (хотя ты мог бы и меня, как ментора > спросить): 4. Зачем нужно было добавлять в спек эти чудовищные переводы Summary и Description? 5. Почему и отчего издохли попуга^Wстарые патчи? -- Regards, -- Sir Raorn. --- http://thousandsofhate.blogspot.com/ [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 836 bytes --] ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [newbies] проблемы с find_requires 2010-11-17 15:49 ` Андрей Черепанов 2010-11-17 15:52 ` Alexey I. Froloff @ 2010-11-17 16:05 ` Dmitry V. Levin 2010-11-17 16:16 ` Anton Chernyshov 2010-11-17 16:09 ` Anton Chernyshov 2 siblings, 1 reply; 11+ messages in thread From: Dmitry V. Levin @ 2010-11-17 16:05 UTC (permalink / raw) To: devel@ where you _can_ ask [-- Attachment #1: Type: text/plain, Size: 505 bytes --] On Wed, Nov 17, 2010 at 06:49:52PM +0300, Андрей Черепанов wrote: > Пакет сложный, зря ты за него взялся. Весь спек-файл ужасный. Не весь, я нашел одно нормальное место: %makeinstall_std :) Антон, вы совершенно зря переименовали каталог preload в preload-0.6.4; откройте для себя gear-import(1). Что касается вашего вопроса про preload.init.in, то вы совершенно зря выкинули preload.alt.preload.init.patch; возможно, другие патчи, которые вы выкинули, тоже достойны внимания. -- ldv [-- Attachment #2: Type: application/pgp-signature, Size: 198 bytes --] ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [newbies] проблемы с find_requires 2010-11-17 16:05 ` Dmitry V. Levin @ 2010-11-17 16:16 ` Anton Chernyshov 0 siblings, 0 replies; 11+ messages in thread From: Anton Chernyshov @ 2010-11-17 16:16 UTC (permalink / raw) To: devel@ where you _can_ ask 17 ноября 2010 г. 19:05 пользователь Dmitry V. Levin <ldv@altlinux.org> написал: > Не весь, я нашел одно нормальное место: %makeinstall_std :) Ну значит не все так плохо :). > Антон, вы совершенно зря переименовали каталог preload в preload-0.6.4; > откройте для себя gear-import(1). Я не переименовывал - он так в тарболе с исходниками теперь есть. А переименовывать каталог из апстрима я не решился :). > Что касается вашего вопроса про preload.init.in, то вы совершенно зря > выкинули preload.alt.preload.init.patch; возможно, другие патчи, которые > вы выкинули, тоже достойны внимания. Ладно, буду ковырять патчи. Кстати, за строчки 64 и 65 меня тоже просьба не бить, права я там тоже уже исправил. -- С уважением, Чернышов Антон ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [newbies] проблемы с find_requires 2010-11-17 15:49 ` Андрей Черепанов 2010-11-17 15:52 ` Alexey I. Froloff 2010-11-17 16:05 ` Dmitry V. Levin @ 2010-11-17 16:09 ` Anton Chernyshov 2010-11-17 16:39 ` Mikhail Efremov 2 siblings, 1 reply; 11+ messages in thread From: Anton Chernyshov @ 2010-11-17 16:09 UTC (permalink / raw) To: devel@ where you _can_ ask 17 ноября 2010 г. 18:49 пользователь Андрей Черепанов <cas@altlinux.ru> написал: > Пакет сложный, зря ты за него взялся. Весь спек-файл ужасный. Это "сырой" вариант, который я выложил для решения проблемы с find_requires. Кстати, это же и к вопросу о переводах Summary и Description. Сборку я не пускал, потому пока все шито белыми нитками. > 1. Для начала запусти buildreq для получения сборочных зависимостей. А то > glib2 явно прибит, хотя вытягивается glib2-devel Я это убрал уже. Запушивать не стал еще раз, потому что исправление мелкое и вносить много чего придется еще. > 2. Ты уверен, что лицензия такая? В COPYING написано, что да. > 3. Зачем явно писал ... > ? > В %files проще было прописать: > %doc %SOURCE1 README AUTHORS ChangeLog TODO THANKS NEWS Такой вариант не проехал, потому пришлось через заднее крыльцо все запихивать. > Антон, придётся публично разьяснять (хотя ты мог бы и меня, как ментора > спросить): Моя вина. -- С уважением, Чернышов Антон ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [newbies] проблемы с find_requires 2010-11-17 16:09 ` Anton Chernyshov @ 2010-11-17 16:39 ` Mikhail Efremov 2010-11-25 9:45 ` [newbies] проблемы с инитскриптом Michael Shigorin 0 siblings, 1 reply; 11+ messages in thread From: Mikhail Efremov @ 2010-11-17 16:39 UTC (permalink / raw) To: devel-newbies On Wed, 17 Nov 2010 19:09:37 +0300 Anton Chernyshov wrote: > > 2. Ты уверен, что лицензия такая? > > В COPYING написано, что да. Там просто текст лицензии. Загляните в шапку любого исходника, это явно %gpl2plus. -- WBR, Mikhail Efremov ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [newbies] проблемы с инитскриптом 2010-11-17 16:39 ` Mikhail Efremov @ 2010-11-25 9:45 ` Michael Shigorin 2010-11-25 10:26 ` Anton Chernyshov 2010-11-25 16:47 ` Dmitry V. Levin 0 siblings, 2 replies; 11+ messages in thread From: Michael Shigorin @ 2010-11-25 9:45 UTC (permalink / raw) To: devel@ where you _can_ ask On Wed, Nov 17, 2010 at 06:49:52PM +0300, Андрей Черепанов wrote: > > Собрал новую версию preload. В результате после сборки > > sisyphus_check ругался на вот такое: > > > > /.out/preload-0.6.4-alt1.x86_64.rpm: forbidden requires: > > /etc/rc.d/init.d(daemon) /etc/rc.d/init.d(killproc) > > sisyphus_check: check-deps ERROR: package dependencies violation Во времена 2.3 произошло изменение в обработке инитксриптов -- с функций redhat-style произошёл переход на свои, созданные с использованием дебиановского start-stop-daemon. К сожалению, не припомню, где это должно быть документировано на вики. > > Проблему сборки я "решил" добавлением в начало спека: > > %add_findreq_skiplist /etc/rc.d/init.d/preload > > Но у меня очень серьезные подозрения, что так делать неправильно. Уже хорошо, что понимаете. > # install pdf documentation > install -D --mode=0644 %SOURCE1 > %buildroot/%_defaultdocdir/%name-%version/%name.pdf Скорее в качестве цели просто ".", чтоб потом подобрать из %doc (который заодно отметит файл как документацию -- см. rpm -qd). On Wed, Nov 17, 2010 at 06:52:57PM +0300, Alexey I. Froloff wrote: > 4. Зачем нужно было добавлять в спек эти чудовищные переводы > Summary и Description? Ты вот столько с ними воюешь, а хоть бы задокументировал rationale вместе с указанием на specspo на вики. Ссылку-то подсунуть быстрее, экономичней и эффективней ;-) > 5. Почему и отчего издохли попуга^Wстарые патчи? Очевидно, поотваливались и не было желания/возможности вникать. Некоторые пакеты старые патчи якорят так, что получается даже больше вреда -- можно было бы выкинуть chroot patch и рекомендовать размещение в отдельном контейнере, так нет, бьёмся и обламываемся. On Wed, Nov 17, 2010 at 07:05:37PM +0300, Dmitry V. Levin wrote: > Что касается вашего вопроса про preload.init.in, то вы > совершенно зря выкинули preload.alt.preload.init.patch; BTW обычно альтовский инитскрипт проще написать с нуля по мотивам апстримного (if any) и /etc/init.d/template, чем оформлять патчем. On Wed, Nov 17, 2010 at 07:09:37PM +0300, Anton Chernyshov wrote: > Это "сырой" вариант, который я выложил для решения проблемы с > find_requires. Это уже не со сборкой, а с проверкой соответствия policy в части инитсктиптов. Работать-то оно и с этим инитскриптом может, см. тж. /etc/init.d/functions-compat и $WITHOUT_RC_COMPAT. > > 1. Для начала запусти buildreq для получения сборочных > > зависимостей. А то glib2 явно прибит, хотя вытягивается > > glib2-devel > Я это убрал уже. Запушивать не стал еще раз, потому что исправление > мелкое и вносить много чего придется еще. На практике пушить стоит всё, в чём уже уверен. :) > > В %files проще было прописать: > > %doc %SOURCE1 README AUTHORS ChangeLog TODO THANKS NEWS > Такой вариант не проехал, потому пришлось через заднее крыльцо все запихивать. А как ругалось? Не уверен, что раскрывшийся %SOURCE1 тут сгодится, я в таких случаях обычно делаю: %install cp -a %SOURCEx . %files %doc that_source_name.txt On Wed, Nov 17, 2010 at 07:39:52PM +0300, Mikhail Efremov wrote: > > > 2. Ты уверен, что лицензия такая? > > В COPYING написано, что да. > Там просто текст лицензии. Загляните в шапку любого исходника, > это явно %gpl2plus. До сих пор этот макрос определён отдельно и требует BuildRequires: rpm-build-licenses -- ---- WBR, Michael Shigorin <mike@altlinux.ru> ------ Linux.Kiev http://www.linux.kiev.ua/ ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [newbies] проблемы с инитскриптом 2010-11-25 9:45 ` [newbies] проблемы с инитскриптом Michael Shigorin @ 2010-11-25 10:26 ` Anton Chernyshov 2010-11-25 16:47 ` Dmitry V. Levin 1 sibling, 0 replies; 11+ messages in thread From: Anton Chernyshov @ 2010-11-25 10:26 UTC (permalink / raw) To: devel@ where you _can_ ask Да, честно говоря, у меня сейчас появились сомнения в том, что смогу поддерживать preload. Жуткие сейчас проблемы с наличием свободного времени. Если уж брать на себя поддержку такого пакета - надо быть твердо уверенным в том, что сможешь его тянуть. А брать и потом говорить "нет" не айс. По поводу замечаний, спасибо. Если найду время, поковыряю его все-таки. На пару вопрос отвечу. 25 ноября 2010 г. 12:45 пользователь Michael Shigorin <mike@osdn.org.ua> написал: > On Wed, Nov 17, 2010 at 06:52:57PM +0300, Alexey I. Froloff wrote: >> 5. Почему и отчего издохли попуга^Wстарые патчи? > > Очевидно, поотваливались и не было желания/возможности вникать. > Некоторые пакеты старые патчи якорят так, что получается даже > больше вреда -- можно было бы выкинуть chroot patch и > рекомендовать размещение в отдельном контейнере, так нет, > бьёмся и обламываемся. Все (!) патчи на исходники (которые патчат .c и .h файлы), судя по diff брались из cvs preload. Т.е. на сегодняшний день они все уже добрались до релиза. А патчи на init-скрипты надо ковырять. Или, действительно, брать и писать с нуля. -- С уважением, Чернышов Антон ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [newbies] проблемы с инитскриптом 2010-11-25 9:45 ` [newbies] проблемы с инитскриптом Michael Shigorin 2010-11-25 10:26 ` Anton Chernyshov @ 2010-11-25 16:47 ` Dmitry V. Levin 1 sibling, 0 replies; 11+ messages in thread From: Dmitry V. Levin @ 2010-11-25 16:47 UTC (permalink / raw) To: devel@ where you _can_ ask [-- Attachment #1: Type: text/plain, Size: 881 bytes --] On Thu, Nov 25, 2010 at 11:45:00AM +0200, Michael Shigorin wrote: > On Wed, Nov 17, 2010 at 06:49:52PM +0300, Андрей Черепанов wrote: > > > Собрал новую версию preload. В результате после сборки > > > sisyphus_check ругался на вот такое: > > > > > > /.out/preload-0.6.4-alt1.x86_64.rpm: forbidden requires: > > > /etc/rc.d/init.d(daemon) /etc/rc.d/init.d(killproc) > > > sisyphus_check: check-deps ERROR: package dependencies violation > > Во времена 2.3 произошло изменение в обработке инитксриптов -- > с функций redhat-style произошёл переход на свои, созданные с > использованием дебиановского start-stop-daemon. К сожалению, > не припомню, где это должно быть документировано на вики. Миша, в предыдущей сборке preload уже был готовый патч на эту тему. Просто при обновлении этот патч, видимо, куда-то затерялся вместе с остальными патчами. -- ldv [-- Attachment #2: Type: application/pgp-signature, Size: 198 bytes --] ^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2010-11-25 16:47 UTC | newest] Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2010-11-17 15:37 [newbies] проблемы с find_requires Anton Chernyshov 2010-11-17 15:46 ` Alexey I. Froloff 2010-11-17 15:49 ` Андрей Черепанов 2010-11-17 15:52 ` Alexey I. Froloff 2010-11-17 16:05 ` Dmitry V. Levin 2010-11-17 16:16 ` Anton Chernyshov 2010-11-17 16:09 ` Anton Chernyshov 2010-11-17 16:39 ` Mikhail Efremov 2010-11-25 9:45 ` [newbies] проблемы с инитскриптом Michael Shigorin 2010-11-25 10:26 ` Anton Chernyshov 2010-11-25 16:47 ` Dmitry V. Levin
devel@ where you _can_ ask This inbox may be cloned and mirrored by anyone: git clone --mirror http://lore.altlinux.org/devel-newbies/0 devel-newbies/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-newbies devel-newbies/ http://lore.altlinux.org/devel-newbies \ devel-newbies@lists.altlinux.org devel-newbies@lists.altlinux.ru devel-newbies@lists.altlinux.com public-inbox-index devel-newbies Example config snippet for mirrors. Newsgroup available over NNTP: nntp://lore.altlinux.org/org.altlinux.lists.devel-newbies AGPL code for this site: git clone https://public-inbox.org/public-inbox.git