linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Re: [PATCH] scripts/dev-needs: depend on bash in shebang
       [not found] <20220314221703.30910-1-dmueller@suse.de>
@ 2022-03-14 22:30 ` Saravana Kannan
  0 siblings, 0 replies; only message in thread
From: Saravana Kannan @ 2022-03-14 22:30 UTC (permalink / raw)
  To: Dirk Müller, LKML, Greg Kroah-Hartman

Adding LKML

On Mon, Mar 14, 2022 at 3:17 PM Dirk Müller <dmueller@suse.de> wrote:
>
> This script uses bash specific syntax. make it more specific by
> declaring it to be used via bash rather than /bin/sh which could
> be non-bash.
>
> Signed-off-by: Dirk Müller <dmueller@suse.de>

Technically this is targeted towards toybox because that's what I deal
with everyday while I work on Android devices. But toybox seems to
follow bash conventions. Would this shebang be considered valid in
that instance?

Btw, I've been meaning to add (for more than year) the following to
the top of the script:
shopt -s expand_aliases

Geert pointed out that the script doesn't work with some setups
because this flag isn't default enabled. I also remember this being a
deprecated option, but until someone wants to rewrite this script in a
way that it'll work on toybox (this is what Android uses) with zero
setup and also not depend on the aliases, we might as well set the
flag to match with reality.

Wanna pull that shopt into your patch?

-Saravana

> ---
>  scripts/dev-needs.sh | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/scripts/dev-needs.sh b/scripts/dev-needs.sh
> index 454cc304fb44..c324f0737f6b 100755
> --- a/scripts/dev-needs.sh
> +++ b/scripts/dev-needs.sh
> @@ -1,4 +1,4 @@
> -#! /bin/sh
> +#!/bin/bash
>  # SPDX-License-Identifier: GPL-2.0
>  # Copyright (c) 2020, Google LLC. All rights reserved.
>  # Author: Saravana Kannan <saravanak@google.com>
> --
> 2.35.1
>

On Mon, Mar 14, 2022 at 3:17 PM Dirk Müller <dmueller@suse.de> wrote:
>
> This script uses bash specific syntax. make it more specific by
> declaring it to be used via bash rather than /bin/sh which could
> be non-bash.
>
> Signed-off-by: Dirk Müller <dmueller@suse.de>
> ---
>  scripts/dev-needs.sh | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/scripts/dev-needs.sh b/scripts/dev-needs.sh
> index 454cc304fb44..c324f0737f6b 100755
> --- a/scripts/dev-needs.sh
> +++ b/scripts/dev-needs.sh
> @@ -1,4 +1,4 @@
> -#! /bin/sh
> +#!/bin/bash
>  # SPDX-License-Identifier: GPL-2.0
>  # Copyright (c) 2020, Google LLC. All rights reserved.
>  # Author: Saravana Kannan <saravanak@google.com>
> --
> 2.35.1
>

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2022-03-14 22:30 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <20220314221703.30910-1-dmueller@suse.de>
2022-03-14 22:30 ` [PATCH] scripts/dev-needs: depend on bash in shebang Saravana Kannan

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).