* [devel] repocop bashisms check
@ 2010-08-19 11:32 Igor Vlasenko
2010-08-19 11:40 ` [devel] [JT] " Ildar Mulyukov
2010-08-19 15:59 ` [devel] " Alexey Gladkov
0 siblings, 2 replies; 11+ messages in thread
From: Igor Vlasenko @ 2010-08-19 11:32 UTC (permalink / raw)
To: devel
Уважаемые коллеги,
в 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
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [devel] [JT] repocop bashisms check
2010-08-19 11:32 [devel] repocop bashisms check Igor Vlasenko
@ 2010-08-19 11:40 ` Ildar Mulyukov
2010-08-19 15:59 ` [devel] " Alexey Gladkov
1 sibling, 0 replies; 11+ messages in thread
From: Ildar Mulyukov @ 2010-08-19 11:40 UTC (permalink / raw)
To: devel
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
=========================================
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [devel] repocop bashisms check
2010-08-19 11:32 [devel] repocop bashisms check Igor Vlasenko
2010-08-19 11:40 ` [devel] [JT] " Ildar Mulyukov
@ 2010-08-19 15:59 ` Alexey Gladkov
2010-08-19 12:38 ` Alexey I. Froloff
1 sibling, 1 reply; 11+ messages in thread
From: Alexey Gladkov @ 2010-08-19 15:59 UTC (permalink / raw)
To: devel
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
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [devel] repocop bashisms check
2010-08-19 15:59 ` [devel] " Alexey Gladkov
@ 2010-08-19 12:38 ` Alexey I. Froloff
2010-08-19 13:05 ` Igor Vlasenko
0 siblings, 1 reply; 11+ messages in thread
From: Alexey I. Froloff @ 2010-08-19 12:38 UTC (permalink / raw)
To: ALT Devel discussion list
[-- 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 --]
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [devel] repocop bashisms check
2010-08-19 12:38 ` Alexey I. Froloff
@ 2010-08-19 13:05 ` Igor Vlasenko
2010-08-19 13:12 ` Andrey Rahmatullin
2010-08-19 13:17 ` Alexey Tourbin
0 siblings, 2 replies; 11+ messages in thread
From: Igor Vlasenko @ 2010-08-19 13:05 UTC (permalink / raw)
To: ALT Linux Team development discussions
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
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [devel] repocop bashisms check
2010-08-19 13:05 ` Igor Vlasenko
@ 2010-08-19 13:12 ` Andrey Rahmatullin
2010-08-19 13:17 ` Alexey Tourbin
1 sibling, 0 replies; 11+ messages in thread
From: Andrey Rahmatullin @ 2010-08-19 13:12 UTC (permalink / raw)
To: devel
[-- 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 --]
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [devel] repocop bashisms check
2010-08-19 13:05 ` Igor Vlasenko
2010-08-19 13:12 ` Andrey Rahmatullin
@ 2010-08-19 13:17 ` Alexey Tourbin
2010-08-19 13:33 ` Igor Vlasenko
` (2 more replies)
1 sibling, 3 replies; 11+ messages in thread
From: Alexey Tourbin @ 2010-08-19 13:17 UTC (permalink / raw)
To: ALT Linux Team development discussions
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 скрипты.
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [devel] repocop bashisms check
2010-08-19 13:17 ` Alexey Tourbin
@ 2010-08-19 13:33 ` Igor Vlasenko
2010-08-19 14:19 ` Dmitry V. Levin
2010-08-19 18:40 ` Aleksey Novodvorsky
2 siblings, 0 replies; 11+ messages in thread
From: Igor Vlasenko @ 2010-08-19 13:33 UTC (permalink / raw)
To: ALT Linux Team development discussions
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
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [devel] repocop bashisms check
2010-08-19 13:17 ` Alexey Tourbin
2010-08-19 13:33 ` Igor Vlasenko
@ 2010-08-19 14:19 ` Dmitry V. Levin
2010-08-19 18:37 ` Alexey Tourbin
2010-08-19 18:40 ` Aleksey Novodvorsky
2 siblings, 1 reply; 11+ messages in thread
From: Dmitry V. Levin @ 2010-08-19 14:19 UTC (permalink / raw)
To: ALT Linux Team development discussions
[-- 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 --]
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [devel] repocop bashisms check
2010-08-19 14:19 ` Dmitry V. Levin
@ 2010-08-19 18:37 ` Alexey Tourbin
0 siblings, 0 replies; 11+ messages in thread
From: Alexey Tourbin @ 2010-08-19 18:37 UTC (permalink / raw)
To: ALT Linux Team development discussions
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
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [devel] repocop bashisms check
2010-08-19 13:17 ` Alexey Tourbin
2010-08-19 13:33 ` Igor Vlasenko
2010-08-19 14:19 ` Dmitry V. Levin
@ 2010-08-19 18:40 ` Aleksey Novodvorsky
2 siblings, 0 replies; 11+ messages in thread
From: Aleksey Novodvorsky @ 2010-08-19 18:40 UTC (permalink / raw)
To: ALT Linux Team development discussions
19 августа 2010 г. 17:17 пользователь Alexey Tourbin <at@altlinux.ru> написал:
> навроде защитников химкинского леса.
Прошу без политики.
Rgrds, Алексей
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2010-08-19 18:40 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-08-19 11:32 [devel] repocop bashisms check Igor Vlasenko
2010-08-19 11:40 ` [devel] [JT] " Ildar Mulyukov
2010-08-19 15:59 ` [devel] " Alexey Gladkov
2010-08-19 12:38 ` Alexey I. Froloff
2010-08-19 13:05 ` Igor Vlasenko
2010-08-19 13:12 ` Andrey Rahmatullin
2010-08-19 13:17 ` Alexey Tourbin
2010-08-19 13:33 ` Igor Vlasenko
2010-08-19 14:19 ` Dmitry V. Levin
2010-08-19 18:37 ` Alexey Tourbin
2010-08-19 18:40 ` Aleksey Novodvorsky
ALT Linux Team development discussions
This inbox may be cloned and mirrored by anyone:
git clone --mirror http://lore.altlinux.org/devel/0 devel/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 devel devel/ http://lore.altlinux.org/devel \
devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru
public-inbox-index devel
Example config snippet for mirrors.
Newsgroup available over NNTP:
nntp://lore.altlinux.org/org.altlinux.lists.devel
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git