[devel] repocop bashisms check

[devel] repocop bashisms check

[Igor Vlasenko]

Уважаемые коллеги,
в repocop добавлена проверка на башизмы с помощью
утилиты checkbashisms. Пока проверялись только скрипты из
/bin /sbin /usr/bin /usr/sbin и /etc/rc.d/init.d.
Обругано 335 пакетов. С одной стороны, это не слишком много,
с другой сторны, там много системообразующих пакетов.
Результаты доступны на 
http://repocop.altlinux.org/pub/repocop/reports/txt/by-test/checkbashisms.txt
завтра будут доступны и на prometeus.

Сообщения выдаются с уровнем info, так как 
у нас полиси на эту тему нет.

-- 

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

Re: [devel] [JT] repocop bashisms check

[Ildar Mulyukov]

On 19.08.2010 17:32:08, Igor Vlasenko wrote:
> Уважаемые коллеги,
> в repocop добавлена проверка на башизмы

"Респект и уважуха!"
Теперь мы знаем всех оборотней в лицо! :-)

Как интересно, что в этот список попал даже пакет bash :-)

> с помощью утилиты checkbashisms.
Спасибо Виталию Липатову.

С уважением,
	Ильдар.
-- 
Ildar  Mulyukov,  free SW designer/programmer/packager
=========================================
email: ildar@altlinux.ru
Jabber: ildar.mulyukov@gmail.com
ICQ: 4334029
ALT Linux Sisyphus http://www.sisyphus.ru
=========================================

Re: [devel] repocop bashisms check

[Alexey I. Froloff]

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

On Thu, Aug 19, 2010 at 07:59:28PM +0400, Alexey Gladkov wrote:
> > Уважаемые коллеги,
> > в repocop добавлена проверка на башизмы с помощью
> > утилиты checkbashisms.
> http://www.opengroup.org/onlinepubs/009695399/utilities/type.html
Эта утилита гавно:

$LEADIN . qr'time\s' =>           q<time>,
$LEADIN . qr'ulimit(\s|\Z)' =>    q<ulimit>,
 - Желающие могут посмотреть в стандарт.

$LEADIN . qr'kill\s+-[^sl]\w*' => q<kill -[0-9] or -[A-Z]>,
$LEADIN . qr'trap\s+["\']?.*["\']?\s+.*[1-9]' => q<trap with signal numbers>,
 - Исправлено в 2.10.66.

А ещё она протухла на четыре месяца.

-- 
Regards,    --
Sir Raorn.   --- http://thousandsofhate.blogspot.com/

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

Re: [devel] repocop bashisms check

[Igor Vlasenko]

On Thu, Aug 19, 2010 at 04:38:51PM +0400, Alexey I. Froloff wrote:
> > > в repocop добавлена проверка на башизмы с помощью
> > > утилиты checkbashisms.
> Эта утилита гавно:
[..]
> А ещё она протухла на четыре месяца.

Это как раз легко исправить,
а вот с примерами выше сложнее.

нам всем требуется доброволец,
который бы написал вдобавок к
http://www.altlinux.org/SecurePackagingPolicy
что-то вроде Portable sh scripts policy
c рекомендуемым статусом, а заодно бы и
допилил checkbashisms до консенсуса.
тогда в таком виде ее и rpm-build можно
будет вставить, проверять %post/un скрипты.

-- 

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

Re: [devel] repocop bashisms check

[Andrey Rahmatullin]

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

On Thu, Aug 19, 2010 at 04:05:06PM +0300, Igor Vlasenko wrote:
> допилил checkbashisms до консенсуса.
Вешайте баги на devscripts.

-- 
WBR, wRAR (ALT Linux Team)
Powered by the ALT Linux fortune(6):

> А как вы называете систему, которая делает не то, что вам хочется или то,
> но не так? :))
Системы, которые делают не то, что мне хочется, бэкапятся на
сервер патологоанатомического отделения.
На предмет выяснения -- не являются ли они летальными мутантами.
		-- jaa in community@

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

Re: [devel] repocop bashisms check

[Alexey Tourbin]

On Thu, Aug 19, 2010 at 04:05:06PM +0300, Igor Vlasenko wrote:
> > А ещё она протухла на четыре месяца.
> 
> Это как раз легко исправить,
> а вот с примерами выше сложнее.
> 
> нам всем требуется доброволец,
> который бы написал вдобавок к
> http://www.altlinux.org/SecurePackagingPolicy
> что-то вроде Portable sh scripts policy

Думаю, нет смысла стремиться к портабельности шелл-скриптов.
Не в последнюю очередь, потому что не доказано, что /bin/ash
работает быстрее (или лучше), чем /bin/sh.  И что вcя эта херня
вообще хоть что-то даёт.  Это просто такие пуристы-онанисты,
навроде защитников химкинского леса.

> c рекомендуемым статусом, а заодно бы и
> допилил checkbashisms до консенсуса.
> тогда в таком виде ее и rpm-build можно
> будет вставить, проверять %post/un скрипты.

Re: [devel] repocop bashisms check

[Igor Vlasenko]

On Thu, Aug 19, 2010 at 05:17:27PM +0400, Alexey Tourbin wrote:
> Думаю, нет смысла стремиться к портабельности шелл-скриптов.
> Не в последнюю очередь, потому что не доказано, что /bin/ash
> работает быстрее (или лучше), чем /bin/sh.  И что вcя эта херня
> вообще хоть что-то даёт.  

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

-- 

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

Re: [devel] repocop bashisms check

[Dmitry V. Levin]

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

On Thu, Aug 19, 2010 at 05:17:27PM +0400, Alexey Tourbin wrote:
> Думаю, нет смысла стремиться к портабельности шелл-скриптов.
> Не в последнюю очередь, потому что не доказано, что /bin/ash
> работает быстрее (или лучше), чем /bin/sh.

/bin/ash в среднем работает быстрее, чем /bin/sh, это наблюдаемый факт.

Впрочем, к переносимости шелл-скриптов это отношения не имеет.


-- 
ldv

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

Re: [devel] repocop bashisms check

[Alexey Gladkov]

19.08.2010 15:32, Igor Vlasenko wrote:
> Уважаемые коллеги,
> в repocop добавлена проверка на башизмы с помощью
> утилиты checkbashisms.

$ checkbashisms /usr/bin/system-report
possible bashism in /usr/bin/system-report line 559 (type):
if   type udevtrigger >/dev/null 2>&1; then
possible bashism in /usr/bin/system-report line 561 (type):
elif type udevadm >/dev/null 2>&1; then

http://www.opengroup.org/onlinepubs/009695399/utilities/type.html

-- 
Rgrds, legion

Re: [devel] repocop bashisms check

[Alexey Tourbin]

On Thu, Aug 19, 2010 at 06:19:39PM +0400, Dmitry V. Levin wrote:
> On Thu, Aug 19, 2010 at 05:17:27PM +0400, Alexey Tourbin wrote:
> > Думаю, нет смысла стремиться к портабельности шелл-скриптов.
> > Не в последнюю очередь, потому что не доказано, что /bin/ash
> > работает быстрее (или лучше), чем /bin/sh.
> 
> /bin/ash в среднем работает быстрее, чем /bin/sh, это наблюдаемый факт.

Что-то помню про /usr/bin/gear ты писал что нифига не быстрее.
Но для пуристов-онанистов это какая разница.  Им нужна питательная среда.
Если бы башизмов не было, их стоило бы придумать.

> Впрочем, к переносимости шелл-скриптов это отношения не имеет.
> 
> 
> -- 
> ldv

Re: [devel] repocop bashisms check

[Aleksey Novodvorsky]

19 августа 2010 г. 17:17 пользователь Alexey Tourbin <at@altlinux.ru> написал:
> навроде защитников химкинского леса.

Прошу без политики.

Rgrds, Алексей