[PATCH 01/10] base-files: upgrade: log with prefix

Adrian Schmutzler mail at adrianschmutzler.de
Tue Nov 3 08:02:40 EST 2020


Hi,

> -----Original Message-----
> From: openwrt-devel [mailto:openwrt-devel-bounces at lists.openwrt.org]
> On Behalf Of Yousong Zhou
> Sent: Dienstag, 3. November 2020 13:21
> To: Philip Prindeville <philipp at redfish-solutions.com>
> Cc: Yousong Zhou <yszhou4tech at gmail.com>; openwrt-
> devel at lists.openwrt.org
> Subject: [PATCH 01/10] base-files: upgrade: log with prefix
> 
> Signed-off-by: Yousong Zhou <yszhou4tech at 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 at lists.openwrt.org
> https://lists.openwrt.org/mailman/listinfo/openwrt-devel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: openpgp-digital-signature.asc
Type: application/pgp-signature
Size: 834 bytes
Desc: not available
URL: <http://lists.openwrt.org/pipermail/openwrt-devel/attachments/20201103/9b822679/attachment.sig>


More information about the openwrt-devel mailing list