[Buildroot] [PATCH] support/dependencies/dependencies.sh: simplify an error message
Peter Korsgaard
peter at korsgaard.com
Wed Apr 10 05:46:40 UTC 2019
>>>>> "Markus" == Markus Mayer <mmayer at broadcom.com> writes:
> There is no need to break the "\n" sequence using "%sn". We can just
> escape it. Note: the escaping backslash needs to be escaped too,
> because the shell will process the string before printf gets to see it.
> Signed-off-by: Markus Mayer <mmayer at broadcom.com>
Maybe we should have used single quotes here to get rid of one layer of
quoting.
Committed, thanks.
> ---
> support/dependencies/dependencies.sh | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
> diff --git a/support/dependencies/dependencies.sh b/support/dependencies/dependencies.sh
> index 826874baa2e8..1d5c164aa65b 100755
> --- a/support/dependencies/dependencies.sh
> +++ b/support/dependencies/dependencies.sh
> @@ -37,8 +37,7 @@ case ":${PATH:-unset}:" in
> ;;
> (*"
> "*) printf "\n"
> - # Break the '\n' sequence, or a \n is printed (which is not what we want).
> - printf "Your PATH contains a newline (%sn) character.\n" "\\"
> + printf "Your PATH contains a newline (\\\n) character.\n"
> printf "This doesn't work. Fix you PATH.\n"
> exit 1
> ;;
> --
> 2.17.1
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
--
Bye, Peter Korsgaard
More information about the buildroot
mailing list