Re: [devel] [git update] packages/katrin: heads/master

Re: [devel] [git update] packages/katrin: heads/master

[Slava Semushin]

2007/7/11, Denis Klimov <zver / altlinux.org>:
[...]
> --- a/katrin.spec
> +++ b/katrin.spec
[...]
> -install -pD -m 644 src/init.d/%name %buildroot/etc/init.d/%name
> +install -pD -m 744 src/init.d/%name %buildroot/%_initdir/%name
[...]

Макрос %_initdir уже начинается с косого слэша, так что можно его
безболезнено убрать:

-install -pD -m 744 src/init.d/%name %buildroot/%_initdir/%name
+install -pD -m 744 src/init.d/%name %buildroot%_initdir/%name

>  %files
>  %_bindir/*
>  %_libdir/%name
>  %config(noreplace) %_sysconfdir/%name
>  %_datadir/%name-%version
> +%_initdir/%name
>  #%_man1dir/*
[...]

1. А зачем нужно версионировать каталог в /usr/share? Это же даже не
документация? Предлагаю написать просто: %_datadir/%name

2. Обычно скрипты для управления демонами заканчиваются на d, что
обозначает что это демон. Может и вам также сделать?

3. #%_man1dir -- так не комментируют. См.
http://www.freesource.info/wiki/AltLinux/Sisyphus/devel/TypicalPackagingErrors/StrikeOutMacro

[...]
> +* Fri Jul 06 2007 Denis Klimov <zver@altlinux.ru> 0.4.10-alt2
> +- change /etc/init.d to %_initdir
[...]

Если вы не хотите, чтобы макрос раскрылся, то его следует записать как
%%_initdir

[...]
> --- a/src/bz/libkatrin-bz-business.c
> +++ b/src/bz/libkatrin-bz-business.c
> @@ -75,6 +75,8 @@ double getCost(struct traffInfo traff) {
>                                 inet_aton(traff.dst_addr,&addr_ip);
>                         //Check of conditions
>                         //Check net area
> +//                     if (strstr(traff.src_addr,"192.168.203")!=NULL)
> +//                             err("========AHTUNG=======");

Несмотря на то, что С99 позволяет использовать комментарии в стиле
С++, в очень многих проектах это не рекомендуется. Я бы тоже не
советовал, как минимум, не все компиляторы вас поймут.

[...]
> +               if (pfl == NULL)
> +                       debug("pfl == NULL");
> +               else
> +                       debug("pfl != NULL");

Я бы попробовал упростить с использованием тернарной операции. Как-нибудь так:

debug("pfl %c= NULL", (pfl == NULL) ? '=' : '!' );

Хотя это менее читабельно получится...

[...]
> --- a/src/db/libkatrin-db-mysql.c
> +++ b/src/db/libkatrin-db-mysql.c
> @@ -294,7 +294,7 @@ if (num_rows > 0)
>                          else
>                          {
>                          (*p).next = (struct filterslist *) malloc(sizeof(struct filterslist));
> -//                        p = (*p).next;
> +                        p = (*p).next;
>                          }
>                  (*p).next = NULL;
>                  (*p).filterid = atoi(row[0]);

Гм. *впал в while(0)*

Отройте для себя оператор стрелки:
http://www.intuit.ru/department/pl/c/14/3.html и пишите p->next вместо
(*p).next


P.S. Изучать язык Си надо на более простых программах, нежели биллинг. IMHO.


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

Re: [devel] [git update] packages/katrin: heads/master

[Denis Klimov]

On Wed, 11 Jul 2007 18:16:49 +0700 Slava Semushin wrote:

> Макрос %_initdir уже начинается с косого слэша, так что можно его
> безболезнено убрать:
> 
> -install -pD -m 744 src/init.d/%name %buildroot/%_initdir/%name
> +install -pD -m 744 src/init.d/%name %buildroot%_initdir/%name

я считаю что со слэшем более читабельно.

> 
> >  %files
> >  %_bindir/*
> >  %_libdir/%name
> >  %config(noreplace) %_sysconfdir/%name
> >  %_datadir/%name-%version
> > +%_initdir/%name
> >  #%_man1dir/*
> [...]
> 
> 1. А зачем нужно версионировать каталог в /usr/share? Это же даже не
> документация? Предлагаю написать просто: %_datadir/%name
> 
В эту директорию ложится пока схемы базы mysql, которые иногда изменяются от версии к версии.

> 2. Обычно скрипты для управления демонами заканчиваются на d, что
> обозначает что это демон. Может и вам также сделать?
верно, так и сделаю.

--
Denis Klimov 
zver

Re: [devel] [git update] packages/katrin: heads/master

[Victor Forsyuk]

On Thu, Jul 12, 2007 at 11:31:54AM +0600, Denis Klimov <kliden@km.ru> wrote:
> > 
> > >  %files
> > >  %_bindir/*
> > >  %_libdir/%name
> > >  %config(noreplace) %_sysconfdir/%name
> > >  %_datadir/%name-%version
> > > +%_initdir/%name
> > >  #%_man1dir/*
> > [...]
> > 
> > 1. А зачем нужно версионировать каталог в /usr/share? Это же даже не
> > документация? Предлагаю написать просто: %_datadir/%name
> > 
> В эту директорию ложится пока схемы базы mysql, которые иногда изменяются от версии к версии.

Допустим. Но, к примеру, man-страница тоже может измениться от версии к
версии. Вы же не пакуете ее с указанием номера версии в имени. Исполняемый
файл так точно изменяется от версии к версии. Вы же не пакуете его с
указанием номера версии в имени.

Версионирование имен имеет смысл только если специально предпринимаются меры
для сосуществования в системе двух пакетов разных версий - для разруливания
конфликта по файлам.

Re: [devel] [git update] packages/katrin: heads/master

[Denis Klimov]

On Thu, 12 Jul 2007 18:02:55 +0300 Victor Forsyuk wrote:

> Версионирование имен имеет смысл только если специально предпринимаются меры
> для сосуществования в системе двух пакетов разных версий - для разруливания
> конфликта по файлам.

Убедили.
--
Denis Klimov 
zver

Re: [devel] [git update] packages/katrin: heads/master

[Dmitry V. Levin]

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

On Wed, Jul 11, 2007 at 06:16:49PM +0700, Slava Semushin wrote:
> 2007/7/11, Denis Klimov <zver / altlinux.org>:
[...]
> P.S. Изучать язык Си надо на более простых программах, нежели биллинг. IMHO.

Не соглашусь.  Изучать язык (не важно, C или не C) всё-таки лучше на
какой-нибудь реальной задаче.  Конечно, первые несколько реализаций
придётся выбросить...


-- 
ldv

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

Re: [devel] [git update] packages/katrin: heads/master

[Денис Смирнов]

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

On Sun, Jul 15, 2007 at 02:27:55AM +0400, Dmitry V. Levin wrote:

>> P.S. Изучать язык Си надо на более простых программах, нежели биллинг. IMHO.
DVL> Не соглашусь.  Изучать язык (не важно, C или не C) всё-таки лучше на
DVL> какой-нибудь реальной задаче.  Конечно, первые несколько реализаций
DVL> придётся выбросить...

Вопрос в том, стоит ли брать в качестве такой реальной задачи столь
крупную, если заведомо собираться выкидывать код.

У Линуса, конечно, получилось поучиться программированию на linux, но
боюсь это на редкость большая удача.

-- 
С уважением, Денис

http://freesource.info
----------------------------------------------------------------------------
<Voins> посади дебианщика за alt так он сразу локали сломает! :)))

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

Re: [devel] [git update] packages/katrin: heads/master

[Ivan Fedorov]

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

Денис Смирнов пишет:
> On Sun, Jul 15, 2007 at 02:27:55AM +0400, Dmitry V. Levin wrote:
> 
>>> P.S. Изучать язык Си надо на более простых программах, нежели биллинг. IMHO.
> DVL> Не соглашусь.  Изучать язык (не важно, C или не C) всё-таки лучше на
> DVL> какой-нибудь реальной задаче.  Конечно, первые несколько реализаций
> DVL> придётся выбросить...
> 
> Вопрос в том, стоит ли брать в качестве такой реальной задачи столь
> крупную, если заведомо собираться выкидывать код.
> 
> У Линуса, конечно, получилось поучиться программированию на linux, но
> боюсь это на редкость большая удача.

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


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 252 bytes --]