[newbies] проблемы с find_requires

[newbies] проблемы с find_requires

[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=3eb081a807ef65b3e5b749b0e99343deea259f57

-- 
С уважением,
Чернышов Антон

Re: [newbies] проблемы с find_requires

[Alexey I. Froloff]

[-- 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 --]

Re: [newbies] проблемы с find_requires

[Андрей Черепанов]

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

Re: [newbies] проблемы с find_requires

[Alexey I. Froloff]

[-- 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 --]

Re: [newbies] проблемы с find_requires

[Dmitry V. Levin]

[-- 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 --]

Re: [newbies] проблемы с find_requires

[Anton Chernyshov]

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

Такой вариант не проехал, потому пришлось через заднее крыльцо все запихивать.

> Антон, придётся публично разьяснять (хотя ты мог бы и меня, как ментора
> спросить):

Моя вина.

-- 
С уважением,
Чернышов Антон

Re: [newbies] проблемы с find_requires

[Anton Chernyshov]

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 меня тоже просьба не бить, права я там тоже
уже исправил.

-- 
С уважением,
Чернышов Антон

Re: [newbies] проблемы с find_requires

[Mikhail Efremov]

On Wed, 17 Nov 2010 19:09:37 +0300 Anton Chernyshov wrote:
> > 2. Ты уверен, что лицензия такая?
> 
> В COPYING написано, что да.

Там просто текст лицензии. Загляните в шапку любого исходника, это явно
%gpl2plus.

-- 
WBR, Mikhail Efremov

Re: [newbies] проблемы с инитскриптом

[Michael Shigorin]

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/

Re: [newbies] проблемы с инитскриптом

[Anton Chernyshov]

Да, честно говоря, у меня сейчас появились сомнения в том, что смогу
поддерживать 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-скрипты надо ковырять. Или,
действительно, брать и писать с нуля.

-- 
С уважением,
Чернышов Антон

Re: [newbies] проблемы с инитскриптом

[Dmitry V. Levin]

[-- 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 --]