Re: [devel] Предложение по изменению SysVinit

Re: [devel] Предложение по изменению SysVinit

[Slava Semushin]

21.04.08, Dmitrij Maslennikov<maslennikovdm / gmail.com> написал(а):
[...]
> http://git.etersoft.ru/people/rlz/packages/SysVinit.git/

Чисто по спеку:

- почему релиз -alt3, а не -alt2?

- +Summary: Utilites from SysVinit
  +%description utils
  +Some utilites from SysVinit

  IMHO слишком уж кратко.

- в секции %files:
  + не указывайте расширение для манов, т.к. оно может быть разным
(RPM сам выбирает способ сжатия)
  + используйте маскросы %_man1dir/%_man5dir/%_man8dir

- Вот это

  +%files utils
  +%defattr(700,root,root)
  +/sbin/bootlogd
  +
  +%defattr(-,root,root,-)
  +/sbin/pidof
  +/sbin/killall5

   можно переписать как

   %files
   %attr(700,root,root) /sbin/bootlogd
   /sbin/pidof
   /sbin/killall5

- IMHO описание в %changelog слишком туманно. Если вы "распилили"
пакет, то неплохо указать на какие подпакеты. У вас же фактически
просто был выделено пара программ в под-пакет, т.е. я бы написал
что-то вроде "Moved some non-specific SysVinit utilities to
SysVinit-utils subpackage."

  +* Mon Apr 21 2008 Dmitry M. Maslennikov <rlz at etersoft.ru> 2.86-alt3
  +- Split package

   P.S. Поправьте меня, если я не прав.

-- 
+ Slava Semushin | slava.semushin @ gmail.com
+ ALT Linux Team | php-coder @ altlinux.ru

Re: [devel] Предложение по изменению SysVinit

[Dmitrij Maslennikov]

21.04.08, Slava Semushin<slava.semushin@gmail.com> написал(а):
>  - почему релиз -alt3, а не -alt2?
Опечатался

>  - +Summary: Utilites from SysVinit
>   +%description utils
>   +Some utilites from SysVinit
>
>   IMHO слишком уж кратко.
>
Попробовал написать побольше

>  - в секции %files:
>   + не указывайте расширение для манов, т.к. оно может быть разным
>  (RPM сам выбирает способ сжатия)
>   + используйте маскросы %_man1dir/%_man5dir/%_man8dir
Пофиксил.

>
>  - Вот это
>
>   +%files utils
>   +%defattr(700,root,root)
>   +/sbin/bootlogd
>   +
>   +%defattr(-,root,root,-)
>   +/sbin/pidof
>   +/sbin/killall5
>
>    можно переписать как
>
>    %files
>    %attr(700,root,root) /sbin/bootlogd
>    /sbin/pidof
>    /sbin/killall5
Пофиксил
>
>  - IMHO описание в %changelog слишком туманно. Если вы "распилили"
>  пакет, то неплохо указать на какие подпакеты. У вас же фактически
>  просто был выделено пара программ в под-пакет, т.е. я бы написал
>  что-то вроде "Moved some non-specific SysVinit utilities to
>  SysVinit-utils subpackage."
Пофиксил

>    P.S. Поправьте меня, если я не прав.
Да нет, просто я не мантейню пакетов, практически, вот и опыта не
хватает. Я больше разработчик.

Re: [devel] Предложение по изменению SysVinit

[Slava Semushin]

21.04.08, Dmitrij Maslennikov<maslennikovdm / gmail.com> написал(а):
[...]
> Пофиксил.
[...]

Не увидел. Забыли git-push?


-- 
+ Slava Semushin | slava.semushin @ gmail.com
+ ALT Linux Team | php-coder @ altlinux.ru

Re: [devel] Предложение по изменению SysVinit

[Dmitrij Maslennikov]

21.04.08, Slava Semushin<slava.semushin@gmail.com> написал(а):
>  Не увидел. Забыли git-push?
Да, уже сделал.

Re: [devel] Предложение по изменению SysVinit

[Serhii Hlodin]

понеділок, 21-кві-2008, Dmitrij Maslennikov написав:
> После внесения изменений планирую активно заняться поддержкой LSB стандарта
> касающегося init-скриптов.

IMHO:

1. Для начала нужно допилить repocop-unittest-init-lsb. Обязательно должно 
проверяться между BEGIN INIT и END INIT:
  - Provides
  - Default-Start
  - Default-Stop
  - Required-Start на наличие $network, если в Default-Start есть 3-5 уровни 
загрузки. Отсутствие $network должно давать ошибку.
  - Required-Start на отсутствие $network, если в Default-Start есть 2-й 
уровень загрузки. Наличие $network должно давать ошибку
  - Соответственно, в зависимости от значений в Default-Start, Default-Stop 
должен иметь остальные уровни. 0 и 6 -- обязательные.
2. В repocop-unittest-init-condrestart нужно заменить проверку condrestart на 
try-restart. Condrestart можно оставить опционально (как ALT-специфичное 
расширение).
3. Исправить все пакеты на которые будет ругаться repocop.

Тогда можно будет и upstart пробовать.

-- 
С уважением,
               С.В. Глодин

Re: [devel] Предложение по изменению SysVinit

[Serhii Hlodin]

понеділок, 21-кві-2008, Dmitrij Maslennikov написав:
> 21.04.08, Serhii Hlodin <hlodin@altlinux.ru> написал(а):
> > 1. Для начала нужно допилить repocop-unittest-init-lsb. Обязательно
> > должно проверяться между BEGIN INIT и END INIT:
>
> Как это относиться к upstart и sysvinit-utils?

Это относится к LSB.

upstart и initng всёравно имеют собственные реализации.

>   - Required-Start на наличие $network, если в Default-Start есть 3-5
> уровни загрузки. Отсутствие $network должно давать ошибку.

> Это еще почему? А если сервису сеть не нужна?

Тормознулся. Бывает. :-)

> Неправда, его можно уже сейчас пробовать. Он вообще ничего особенного не
> требует.

Уже увидел.

-- 
С уважением,
               С.В. Глодин

Re: [devel] Предложение по изменению SysVinit

[Dmitrij Maslennikov]

21.04.08, Serhii Hlodin<hlodin@altlinux.ru> написал(а):
>  upstart и initng всёравно имеют собственные реализации.
Ну реализацию upstaart
>
>
>  >   - Required-Start на наличие $network, если в Default-Start есть 3-5
>  > уровни загрузки. Отсутствие $network должно давать ошибку.
>
>  > Это еще почему? А если сервису сеть не нужна?
>
>
> Тормознулся. Бывает. :-)
>
>
>  > Неправда, его можно уже сейчас пробовать. Он вообще ничего особенного не
>  > требует.
>
>
> Уже увидел.
>
>
>  --
>  С уважением,
>                С.В. Глодин
>
>  _______________________________________________
>  Devel mailing list
>  Devel@lists.altlinux.org
>  https://lists.altlinux.org/mailman/listinfo/devel

Re: [devel] Предложение по изменению SysVinit

[Dmitrij Maslennikov]

21.04.08, Serhii Hlodin<hlodin@altlinux.ru> написал(а):
>  upstart и initng всёравно имеют собственные реализации.
Ну реализацию upstart можно и поменять. В Ubuntu они сейчас запускают
скрипты так же как и у нас. Но никто не мешает использовать
собственную реализацию запускалки сервисов.

[devel] JFYI re LSB

[Michael Shigorin]

On Mon, Apr 21, 2008 at 04:07:17PM +0300, Serhii Hlodin wrote:
> > > 1. Для начала нужно допилить repocop-unittest-init-lsb.
> > Как это относиться к upstart и sysvinit-utils?
> Это относится к LSB.

LSB -- скорее муха на окне, чем рабочий и полезный стандарт...
поэтому на данном этапе нет смысла ни для каких целей указывать
его в качестве основания для блокирования любой другой работы.

Вывод, сделанный из множественных ситуаций, включая эту:
http://www.linux.kiev.ua/ru/news/archive/details/view/1448/
и общение с одним из участников проекта разработки.

Возможно, через несколько лет это досадное положение
уже изменится.

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

Re: [devel] Предложение по изменению SysVinit

[Igor Vlasenko]

On Mon, Apr 21, 2008 at 02:41:49PM +0300, Serhii Hlodin wrote:
> понеділок, 21-кві-2008, Dmitrij Maslennikov написав:

Сделал 
repocop-unittest-init-lsb add hlodin

В принципе не обязательно делать все в виде одного пакета,
тем более одного теста. Единственная деталь - сообщения должны быть
уровня "info"

> > После внесения изменений планирую активно заняться поддержкой LSB стандарта
> > касающегося init-скриптов.
> 
> IMHO:
> 
> 1. Для начала нужно допилить repocop-unittest-init-lsb. Обязательно должно 
> проверяться между BEGIN INIT и END INIT:
>   - Provides
>   - Default-Start
>   - Default-Stop
>   - Required-Start на наличие $network, если в Default-Start есть 3-5 уровни 
> загрузки. Отсутствие $network должно давать ошибку.
>   - Required-Start на отсутствие $network, если в Default-Start есть 2-й 
> уровень загрузки. Наличие $network должно давать ошибку
>   - Соответственно, в зависимости от значений в Default-Start, Default-Stop 
> должен иметь остальные уровни. 0 и 6 -- обязательные.
> 2. В repocop-unittest-init-condrestart нужно заменить проверку condrestart на 
> try-restart. Condrestart можно оставить опционально (как ALT-специфичное 
> расширение).
> 3. Исправить все пакеты на которые будет ругаться repocop.
> 
> Тогда можно будет и upstart пробовать.
> 
> -- 
> С уважением,
>                С.В. Глодин
> 
> _______________________________________________
> Devel mailing list
> Devel@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/devel
-- 

Dr. Igor Vlasenko
--------------------
Topology Department
Institute of Math
Kiev, Ukraine

Re: [devel] Предложение по изменению SysVinit

[Dmitry V. Levin]

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

On Mon, Apr 21, 2008 at 02:43:02PM +0400, Dmitrij Maslennikov wrote:
> Я собрал upstart для AltLinux. Но не могу выложить его из-за того, что пакет
> конфликтует с SysVinit, который в свою очередь содержит несколько полезных
> утилит. Чтобы сохранить утилиты и предоставить мозможность поставить upstart
> необходимо разбить пакет на две части, например, как это сделал я у себя в
> 
> http://git.etersoft.ru/people/rlz/packages/SysVinit.git/

Дмитрий, вы, конечно, вольны выкладывать (промежуточные) результаты своей
работы куда угодно, но если бы вы запушили изменения на git.alt, то я (и
некоторые другие) благодаря системе уведомлений могли бы узнать и
посмотреть эти изменения, не дожидаясь вашего анонса.

> Прошу мантейнера проверить и принять мои правки или предложить свой вариант.

todo++ :)


-- 
ldv

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