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

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

[Slava Semushin]

2007/5/16, Vladimir V. Kamarzin <vvk / altlinux.org>:
[...]
> @@ -0,0 +1,6 @@
> +#!/bin/sh
> +if [ "$1" == "kmemsize" ]; then
> +       cat /proc/user_beancounters | grep $1 | awk -F\  '{print $3}'
> +else
> +       cat /proc/user_beancounters | grep $@ | awk -F\  '{print $2}'
> +fi

'==' это башизм. Для пущей переносимости лучше использовать '='.
Также не лишним, на мой взгляд, был бы замена cat file | grep на grep
file -- таким образом создаётся на один процесс меньше.


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

Re: [devel] [git update] [JT] packages/zabbix: heads/master

[Hihin Ruslan]

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

Здравствуйте Slava Semushin
  В сообщении от 17 мая 2007 Slava Semushin написал(a):
 > '==' это башизм.
Не надо так ругаться :)
Это не только башизм, но и сишизм и пхпишизм :)

-- 
  А ещё говорят так  (fortune):
 
________________________________________________________________________
С уважением Хихин Руслан

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

Re: [devel] [git update] [JT] packages/zabbix: heads/master

[Mikhail Gusarov]

Twas brillig at 08:28:34 17.05.2007 UTC+04 when Hihin Ruslan did gyre and gimble:

 >> '==' это башизм.

 HR> Не надо так ругаться :) Это не только башизм, но и сишизм и пхпишизм :)

bashism - это уже устоявшийся термин: конструкция bash, не входящая в синтаксис
sh стандарта POSIX/SUS.

-- 
JID: dottedmag@jabber.dottedmag.net

Re: [devel] [git update] [JT] packages/zabbix: heads/master

[Mikhail Gusarov]

Twas brillig at 11:32:03 17.05.2007 UTC+07 when Mikhail Gusarov did gyre and gimble:

 MG> bashism - это уже устоявшийся термин: конструкция bash, не входящая в
 MG> синтаксис sh стандарта POSIX/SUS.

Вот, например, bashism-ы, проверяемые debian-овым lintian-ом:

http://lintian.debian.org/reports/Tpossible-bashism-in-maintainer-script.html

-- 
JID: dottedmag@jabber.dottedmag.net

Re: [devel] [git update] [JT] packages/zabbix: heads/master

[Slava Semushin]

2007/5/17, Mikhail Gusarov <dottedmag / dottedmag.net>:
>  MG> bashism - это уже устоявшийся термин: конструкция bash, не входящая в
>  MG> синтаксис sh стандарта POSIX/SUS.
>
> Вот, например, bashism-ы, проверяемые debian-овым lintian-ом:
>
> http://lintian.debian.org/reports/Tpossible-bashism-in-maintainer-script.html

Спасибо за ссылку. Сейчас добавлю к
http://freesource.info/wiki/devel/PortabilityNotes

Конкретно, про '==' есть здесь:
http://wiki.netbsd.se/index.php/The_pkgsrc_portability_guide#Other_non-standard_extensions

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


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

Re: [devel] [git update] [JT] packages/zabbix: heads/master

[Hihin Ruslan]

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

Здравствуйте Mikhail Gusarov
  В сообщении от 17 мая 2007 Mikhail Gusarov написал(a):
 
 > Вот, например, bashism-ы, проверяемые debian-овым lintian-ом:

 > http://lintian.debian.org/reports/Tpossible-bashism-in-maintainer-sc
 >ript.html

Может и устаявшееся, но режет ухо, достаточно поменять одну букву ...
лучше уж так : bash`изм (башь`изм) или bashism

-- 
  А ещё говорят так  (fortune):
 
________________________________________________________________________
С уважением Хихин Руслан

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

Re: [devel] [git update] [JT] packages/zabbix: heads/master

[Alexey Gladkov]

Mikhail Gusarov пишет:
> Вот, например, bashism-ы, проверяемые debian-овым lintian-ом:

Очень интересный этот lintian.

-- 
Rgrds, legion