From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Huth Subject: Re: [kvm-unit-tests PATCH] Use /bin/env in shebang to make scripts more portable Date: Fri, 17 Mar 2017 09:04:17 +0100 Message-ID: <9bcf5946-dd02-c312-0b00-3dcb09568fda@redhat.com> References: <20170315102550.GA44201@Sergeys-MacBook-Pro-2.local> <29e0ed97-d18a-acf8-2483-5c53db9ca053@redhat.com> <20170317051122.GA61717@Sergeys-MacBook-Pro-2.local> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: kvm@vger.kernel.org To: Sergey Bronnikov Return-path: Received: from mx1.redhat.com ([209.132.183.28]:50604 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751159AbdCQIOi (ORCPT ); Fri, 17 Mar 2017 04:14:38 -0400 In-Reply-To: <20170317051122.GA61717@Sergeys-MacBook-Pro-2.local> Sender: kvm-owner@vger.kernel.org List-ID: On 17.03.2017 06:11, Sergey Bronnikov wrote: > On 08:38 Thu 16 Mar , Thomas Huth wrote: >> On 15.03.2017 11:25, Sergey Bronnikov wrote: >>> Some operating systems installs bash executable file to other directory >>> than /bin. So it is better to use env utility to find bash. >> >> Which operating systems do you have in mind here? > > OpenBSD Ok, then I'd say this patch simply does not make sense. We're talking about kvm-unit-tests here - and KVM is pretty much Linux-only as far as I know. Every Linux distro that I've seen so far provides /bin/bash, so I don't think we've got to change this. Thomas