Thibaut VARÈNE  <ha...@slashdirt.org> wrote:
> Avoid having /sbin/wifi silently ignore unknown keywords and
> execute "up"; instead display the help message and exit with an
> error.
> 
> Spell out the "up" keyword (which has users), add it to usage
> output, and preserve the implicit assumption that runing
> /sbin/wifi without argument performs "up".
> 
> Signed-off-by: Thibaut VARÈNE <ha...@slashdirt.org>
> ---
>  package/base-files/files/sbin/wifi | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/package/base-files/files/sbin/wifi
> b/package/base-files/files/sbin/wifi index
> 83befc0d6f..f7a10de215 100755
> --- a/package/base-files/files/sbin/wifi
> +++ b/package/base-files/files/sbin/wifi
> @@ -6,7 +6,7 @@
>  
>  usage() {
>       cat <<EOF
> -Usage: $0 [config|down|reload|status]
> +Usage: $0 [config|up|down|reload|status]
>  enables (default), disables or configures devices not yet configured.

so, which one is "enable" is it config, up, down, reload, or
status? Perhaps update this to say which command is the default?

>  EOF
>       exit 1
> @@ -241,5 +241,6 @@ case "$1" in
>       reload) wifi_reload "$2";;
>       reload_legacy) wifi_reload_legacy "$2";;
>       --help|help) usage;;
> -     *) ubus call network reload; wifi_updown "enable" "$2";;
> +     ''|up) ubus call network reload; wifi_updown "enable" "$2";;
> +     *) usage; exit 1;;
>  esac
> -- 
> 2.14.3 (Apple Git-98)
> 
> 
> _______________________________________________
> openwrt-devel mailing list
> openwrt-devel@lists.openwrt.org
> https://lists.openwrt.org/mailman/listinfo/openwrt-devel

Attachment: signature.html
Description: OpenPGP Digital Signature

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

Reply via email to