ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: Alexey Gladkov <legion@altlinux.ru>
To: devel@lists.altlinux.org
Subject: Re: [devel] I: Repocop NMU phase 2.
Date: Wed, 05 Sep 2012 01:04:28 +0400
Message-ID: <50466CDC.7060302@altlinux.ru> (raw)
In-Reply-To: <20120904203933.GA20609@dad.imath.kiev.ua>

05.09.2012 00:39, Igor Vlasenko wrote:
> В тесте используется утилита /usr/bin/checkbashisms из
> одноименного пакета checkbashisms (часть Debian'овских
> devscripts).
> Репокоп просто запускает эту утилиту и выводит, ругалась она или нет.

На самом деле эта проверка бессмыслена вообще т.к. у нас нет
возможности указать правильный шелл в системных скриптах. Указание
/bin/bash в них будет ошибкой. Поэтому /bin/sh это в нашем случае
просто правильный баш.

Кроме того, миграции на posix shell у нас не планируется.

> checkbashisms тоже не без греха. Если что, лучше всего сразу вешать
> баги в debian на devscripts.

Это не совсем верно т.к. в debian свои полиси и взгляды на то что
правильно и не правильно. Например в debian используется dash как sh,
но это не мешает checkbashisms ругаться на то, что поддерживается этим
шеллом:

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

формально её нет в стандарте, но это не башизм.

-- 
Rgrds, legion



  reply	other threads:[~2012-09-04 21:04 UTC|newest]

Thread overview: 51+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-09-04 18:19 Igor Vlasenko
2012-09-04 18:57 ` Dmitry V. Levin
2012-09-04 19:01 ` Alexey Gladkov
2012-09-04 19:13   ` Igor Vlasenko
2012-09-04 19:31     ` Alexey Gladkov
2012-09-04 19:46       ` Igor Vlasenko
2012-09-04 19:56   ` Igor Vlasenko
2012-09-04 20:07     ` Alexey Gladkov
2012-09-04 20:13       ` Igor Vlasenko
2012-09-04 20:26         ` Alexey Gladkov
2012-09-04 20:39           ` Igor Vlasenko
2012-09-04 21:04             ` Alexey Gladkov [this message]
2012-09-04 21:15               ` Led
2012-09-04 21:28                 ` [devel] posix shell Alexey Gladkov
2012-09-05  4:58                   ` Ildar Mulyukov
2012-09-05  9:45                     ` Alexey Gladkov
2012-09-05 10:13                       ` [devel] [JT] " Michael Shigorin
2012-09-05 10:33                         ` Alexey Gladkov
2012-09-05 15:59                       ` [devel] " Igor Vlasenko
2012-09-05 16:11                         ` Led
2012-09-05 16:38                           ` Igor Vlasenko
2012-09-05 17:15                             ` Led
2012-09-05 20:49                               ` Michael Shigorin
2012-09-04 21:31                 ` [devel] I: Repocop NMU phase 2 Igor Vlasenko
2012-09-04 21:29               ` Igor Vlasenko
2012-09-04 21:42                 ` Alexey Gladkov
2012-09-04 21:52                   ` Igor Vlasenko
2012-09-04 22:07                   ` Michael Shigorin
2012-09-05 15:57                     ` Igor Vlasenko
2012-09-05 15:57                   ` Igor Vlasenko
2012-09-04 21:46             ` Alexey Gladkov
2012-09-04 20:12 ` Yuri N. Sedunov
2012-09-04 20:17   ` Igor Vlasenko
2012-09-05  2:41 ` REAL
2012-09-06 12:17 ` Igor Vlasenko
2012-09-06 12:26   ` Valery V. Inozemtsev
2012-09-06 12:30     ` Dmitry V. Levin
2012-09-06 12:31       ` Michael Shigorin
2012-09-06 12:36         ` REAL
2012-09-06 12:37         ` Dmitry V. Levin
2012-09-06 14:30           ` Igor Vlasenko
2012-09-06 17:21             ` Kirill Maslinsky
2012-09-06 17:45               ` Igor Vlasenko
2012-09-10 17:05             ` Igor Vlasenko
2012-09-10 19:52               ` Dmitry V. Levin
2012-09-11  9:00                 ` Igor Vlasenko
2012-09-10 20:32       ` Dmitry V. Levin
2012-09-11  9:08         ` Igor Vlasenko
2012-09-11 10:04           ` Dmitry V. Levin
2012-09-06 12:30     ` Igor Vlasenko
2012-09-06 19:02   ` Sergey Bolshakov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=50466CDC.7060302@altlinux.ru \
    --to=legion@altlinux.ru \
    --cc=devel@lists.altlinux.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

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