grub-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 2/2] autogen.sh: Detect python


From: Daniel Kiper
Subject: Re: [PATCH 2/2] autogen.sh: Detect python
Date: Thu, 26 Aug 2021 16:37:50 +0200
User-agent: NeoMutt/20170113 (1.7.2)

On Wed, Aug 18, 2021 at 09:38:11AM +0200, Petr Vorel wrote:
> > Hi Daniel,
>
> > sorry for longer time to reply (vacation).
>
> > > On Fri, Aug 06, 2021 at 08:45:08AM +0200, Petr Vorel wrote:
> > > > It help to avoid error on distros which has only python3 binary:
> > > > ./autogen.sh: line 20: python: command not found
>
> > > > Using bash builtin 'command -v' to avoid requiring which as extra
> > > > dependency (usable on containers).
> > command -v is supported in other common shells (busybox sh and dash),
> > I'll add it to the commit message.
> > IMHO it's non-POSIX extension, but because it's support we use it in LTP 
> > shell
> > API, where we expect very minimal shell tools (i.e. no bash, no core utils).
> Actually "command -v" [1] and "type" [2] are POSIX.

OK, then ignore my comment WRT adding bash to the requirements
into the INSTALL file.

Daniel



reply via email to

[Prev in Thread] Current Thread [Next in Thread]