Hi Petr,

> [...]  
> +should_be_patched() {
> +     local bin="$1"
> +
> +     [ -x "$bin" ] || return 1
> +
> +     case "$bin" in
> +             *.so|*.so.[0-9]*)
> +                     return 1
> +             ;;
> +     *)
> +             file "$bin" | grep -sqE "ELF.*(executable|interpreter)" && 
> return 0

Apart from this apparently inconsistent indentation (tabs vs. spaces?) -
LGTM.

Assuming the white space is straightened out...
Acked-by: Jo-Philipp Wich <[email protected]>

_______________________________________________
openwrt-devel mailing list
[email protected]
https://lists.openwrt.org/mailman/listinfo/openwrt-devel

Reply via email to