From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <legion@altlinux.ru>
X-Spam-Checker-Version: SpamAssassin 3.2.5 (2008-06-10) on sa.int.altlinux.org
X-Spam-Level: 
X-Spam-Status: No, score=-2.6 required=5.0 tests=BAYES_00 autolearn=ham
	version=3.2.5
Message-ID: <50466CDC.7060302@altlinux.ru>
Date: Wed, 05 Sep 2012 01:04:28 +0400
From: Alexey Gladkov <legion@altlinux.ru>
User-Agent: Mozilla/5.0 (X11; Linux x86_64;
	rv:15.0) Gecko/20120829 Thunderbird/15.0
MIME-Version: 1.0
To: devel@lists.altlinux.org
References: <20120904181911.GA18971@dad.imath.kiev.ua>
	<5046500B.9050804@altlinux.ru>
	<20120904195624.GA20079@dad.imath.kiev.ua>
	<50465F88.40403@altlinux.ru>
	<20120904201308.GA20384@dad.imath.kiev.ua>
	<504663FC.5050302@altlinux.ru>
	<20120904203933.GA20609@dad.imath.kiev.ua>
In-Reply-To: <20120904203933.GA20609@dad.imath.kiev.ua>
X-Enigmail-Version: 1.5a1pre
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Subject: Re: [devel] I: Repocop NMU phase 2.
X-BeenThere: devel@lists.altlinux.org
X-Mailman-Version: 2.1.12
Precedence: list
Reply-To: ALT Linux Team development discussions <devel@lists.altlinux.org>
List-Id: ALT Linux Team development discussions <devel.lists.altlinux.org>
List-Unsubscribe: <https://lists.altlinux.org/mailman/options/devel>,
	<mailto:devel-request@lists.altlinux.org?subject=unsubscribe>
List-Archive: <http://lists.altlinux.org/pipermail/devel>
List-Post: <mailto:devel@lists.altlinux.org>
List-Help: <mailto:devel-request@lists.altlinux.org?subject=help>
List-Subscribe: <https://lists.altlinux.org/mailman/listinfo/devel>,
	<mailto:devel-request@lists.altlinux.org?subject=subscribe>
X-List-Received-Date: Tue, 04 Sep 2012 21:04:52 -0000
Archived-At: <http://lore.altlinux.org/devel/50466CDC.7060302@altlinux.ru/>
List-Archive: <http://lore.altlinux.org/devel/>
List-Post: <mailto:devel@altlinux.org>

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