Fair enough I guess.  I would suggest
      that we simply leave the "exec" line there instead of the whole
      structure.  That will require that the target file be executable,
      and will support both scripts and binaries.
      Cheers!
          Diego Rivera
      On 04/30/2013 10:07 AM, Stefan Saraev wrote:
    
      In
        packages/addons/service/multimedia/tvheadend/source/bin/tv_grab_file:
      > @@ -36,6 +37,13 @@ if [[ $# -lt 1 ]]; then
>    if [ "$XMLTV_TYPE" = "FILE" ]; then
>      cat "$XMLTV_LOCATION_FILE"
>      exit 0
> +  elif [ "$XMLTV_TYPE" = "SCRIPT" ]; then
> +    if [ -x "$XMLTV_LOCATION_SCRIPT" ] ; then
> +      exec "$XMLTV_LOCATION_SCRIPT"
> +    else
> +      /bin/sh "$XMLTV_LOCATION_SCRIPT"
> +      exit $?
> +    fi

      my point is: if we give people freedom to write in whatever
        they want, we shouldn't try to use "/bin/sh" anyway
      —
        Reply to this email directly or view
          it on GitHub.