]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
tox.ini: Avoid using bash where unnecessary
authorYAMAMOTO Takashi <yamamoto@valinux.co.jp>
Fri, 17 Oct 2014 03:30:38 +0000 (12:30 +0900)
committerYAMAMOTO Takashi <yamamoto@valinux.co.jp>
Tue, 21 Oct 2014 00:29:12 +0000 (09:29 +0900)
Switch to sh, which is hopefully more ubiquitously available than bash.

A recent change (commit 085a35d657cf0fa41a402f2af66c4beaa0f60db2)
introduced bash dependency for "tox -e pep8".  It broke my environment,
where bash is not available.  This change aims to restore it.
As far as I undestand, the change in question doesn't actually need
the specific shell dialect.  So switch to sh, which is expected to be
available for any POSIX-like systems, would improve the situation.

Closes-Bug: #1382327
Change-Id: Ie023d2265d9949a3e3721f03e865e4acfb460de2

tox.ini

diff --git a/tox.ini b/tox.ini
index 853ae1fe6bf7e74d1bd3be1501388acc2dab9e4d..454f29f8c76d825598fcfb95f387d10fff8e14ec 100644 (file)
--- a/tox.ini
+++ b/tox.ini
@@ -43,8 +43,8 @@ downloadcache = ~/cache/pip
 commands =
   flake8
   neutron-db-manage check_migration
-  bash -c "find neutron -type f -regex '.*\.pot?' -print0|xargs -0 -n 1 msgfmt --check-format -o /dev/null"
-whitelist_externals = bash
+  sh -c "find neutron -type f -regex '.*\.pot?' -print0|xargs -0 -n 1 msgfmt --check-format -o /dev/null"
+whitelist_externals = sh
 
 [testenv:i18n]
 commands = python ./tools/check_i18n.py ./neutron ./tools/i18n_cfg.py