[FFmpeg-devel] [PATCH 1/3] configure: Support backslashes in TMPDIR on OS/2.

KO Myung-Hun komh78 at gmail.com
Sat Apr 16 06:39:44 CEST 2016


Hi/2.

Dave Yeo wrote:
> From b322802ecc6d87fae3afb17fbca8217993d8d0c7 Mon Sep 17 00:00:00 2001
> From: Dmitriy Kuminov <coding at dmik.org>
> Date: Thu, 14 Apr 2016 01:41:30 +0400
> Subject: [PATCH 1/3] configure: Support backslashes in TMPDIR on OS/2.
> 
> Signed-off-by: Dave Yeo <daveryeo at telus.net>
> ---
>  configure | 13 +++++++++----
>  1 file changed, 9 insertions(+), 4 deletions(-)
> 
> diff --git a/configure b/configure
> index 0ff1b9d..ff80aee 100755
> --- a/configure
> +++ b/configure
> @@ -734,7 +734,12 @@ check_deps(){
>  
>  print_config(){
>      pfx=$1
> -    files=$2
> +    if test "$host_os" = "os/2"; then
> +        # awk treats \ as escape chars, convert them to / in paths
> +        files=`echo "$2" | tr '\\\' /`

echoing back-slashes with echo is not portable. For example, on
ksh-compatible shells such as pdksh and mksh, back-slashes are escaped
by echo. Please use expr.

> +    else
> +        files=$2
> +    fi
>      shift 2
>      map 'eval echo "$v \${$v:-no}"' "$@" |
>      awk "BEGIN { split(\"$files\", files) }
> @@ -3550,10 +3555,10 @@ fi
>  
>  tmpfile(){
>      tmp=$(mktemp -u "${TMPDIR}/ffconf.XXXXXXXX")$2 &&
> -        (set -C; exec > $tmp) 2>/dev/null ||
> +        (set -C; exec > "$tmp") 2>/dev/null ||
>          die "Unable to create temporary file in $TMPDIR."
> -    append TMPFILES $tmp
> -    eval $1=$tmp
> +    append TMPFILES "$tmp"
> +    eval $1=\"$tmp\"
>  }
>  
>  trap 'rm -f -- $TMPFILES' EXIT

-- 
KO Myung-Hun

Using Mozilla SeaMonkey 2.7.2
Under OS/2 Warp 4 for Korean with FixPak #15
In VirtualBox v4.1.32 on Intel Core i7-3615QM 2.30GHz with 8GB RAM

Korean OS/2 User Community : http://www.ecomstation.co.kr



More information about the ffmpeg-devel mailing list