Hi,

> -----Original Message-----
> From: openwrt-devel [mailto:openwrt-devel-boun...@lists.openwrt.org]
> On Behalf Of Yousong Zhou
> Sent: Dienstag, 3. November 2020 13:21
> To: Philip Prindeville <phil...@redfish-solutions.com>
> Cc: Yousong Zhou <yszhou4t...@gmail.com>; openwrt-
> de...@lists.openwrt.org
> Subject: [PATCH 01/10] base-files: upgrade: log with prefix
> 
> Signed-off-by: Yousong Zhou <yszhou4t...@gmail.com>
> ---
>  package/base-files/files/lib/upgrade/common.sh | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/package/base-files/files/lib/upgrade/common.sh
> b/package/base-files/files/lib/upgrade/common.sh
> index 2eb26ba44b..56daabd778 100644
> --- a/package/base-files/files/lib/upgrade/common.sh
> +++ b/package/base-files/files/lib/upgrade/common.sh
> @@ -64,7 +64,7 @@ ask_bool() {
>  }
> 
>  v() {

Generally, I like the idea. I'm not sure whether just v() is a good choice for 
the function name, though.

> -     [ -n "$VERBOSE" ] && [ "$VERBOSE" -ge 1 ] && echo "$@"
> +     [ -n "$VERBOSE" ] && [ "$VERBOSE" -ge 1 ] && echo "== upgrade:
> $@"

Is there a particular reason for choosing "==" as prefix or is this arbitrary? 
Looks a bit odd to me (not the fact of having a prefix, but the prefix itself) 
...

In any case, thanks for the improvements.

Best

Adrian

>  }
> 
>  json_string() {
> 
> _______________________________________________
> openwrt-devel mailing list
> openwrt-devel@lists.openwrt.org
> https://lists.openwrt.org/mailman/listinfo/openwrt-devel

Attachment: openpgp-digital-signature.asc
Description: PGP signature

_______________________________________________
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel

Reply via email to