From owner-freebsd-ports-bugs@FreeBSD.ORG Sun Oct 24 08:00:17 2010 Return-Path: Delivered-To: freebsd-ports-bugs@hub.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 7D8781065673 for ; Sun, 24 Oct 2010 08:00:17 +0000 (UTC) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:4f8:fff6::28]) by mx1.freebsd.org (Postfix) with ESMTP id 3FBB68FC14 for ; Sun, 24 Oct 2010 08:00:17 +0000 (UTC) Received: from freefall.freebsd.org (localhost [127.0.0.1]) by freefall.freebsd.org (8.14.4/8.14.4) with ESMTP id o9O80HXc060808 for ; Sun, 24 Oct 2010 08:00:17 GMT (envelope-from gnats@freefall.freebsd.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.14.4/8.14.4/Submit) id o9O80HKb060807; Sun, 24 Oct 2010 08:00:17 GMT (envelope-from gnats) Resent-Date: Sun, 24 Oct 2010 08:00:17 GMT Resent-Message-Id: <201010240800.o9O80HKb060807@freefall.freebsd.org> Resent-From: FreeBSD-gnats-submit@FreeBSD.org (GNATS Filer) Resent-To: freebsd-ports-bugs@FreeBSD.org Resent-Reply-To: FreeBSD-gnats-submit@FreeBSD.org, Nicole Reid Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 62AE61065674 for ; Sun, 24 Oct 2010 07:59:36 +0000 (UTC) (envelope-from nobody@FreeBSD.org) Received: from www.freebsd.org (www.freebsd.org [IPv6:2001:4f8:fff6::21]) by mx1.freebsd.org (Postfix) with ESMTP id 36C718FC08 for ; Sun, 24 Oct 2010 07:59:36 +0000 (UTC) Received: from www.freebsd.org (localhost [127.0.0.1]) by www.freebsd.org (8.14.3/8.14.3) with ESMTP id o9O7xZ8I046945 for ; Sun, 24 Oct 2010 07:59:35 GMT (envelope-from nobody@www.freebsd.org) Received: (from nobody@localhost) by www.freebsd.org (8.14.3/8.14.3/Submit) id o9O7xZqI046944; Sun, 24 Oct 2010 07:59:35 GMT (envelope-from nobody) Message-Id: <201010240759.o9O7xZqI046944@www.freebsd.org> Date: Sun, 24 Oct 2010 07:59:35 GMT From: Nicole Reid To: freebsd-gnats-submit@FreeBSD.org X-Send-Pr-Version: www-3.1 Cc: Subject: ports/151677: [audio/cuetools] Fix filename handling for extras/cuetag.sh X-BeenThere: freebsd-ports-bugs@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Ports bug reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 24 Oct 2010 08:00:17 -0000 >Number: 151677 >Category: ports >Synopsis: [audio/cuetools] Fix filename handling for extras/cuetag.sh >Confidential: no >Severity: non-critical >Priority: low >Responsible: freebsd-ports-bugs >State: open >Quarter: >Keywords: >Date-Required: >Class: sw-bug >Submitter-Id: current-users >Arrival-Date: Sun Oct 24 08:00:16 UTC 2010 >Closed-Date: >Last-Modified: >Originator: Nicole Reid >Release: >Organization: >Environment: >Description: The extras/cuetag.sh script in this port doesn't escape filenames properly. There's an open bug for this on the project tracker (http://developer.berlios.de/bugs/?func=detailbug&bug_id=10401&group_id=2130) untouched for nearly four years, so a patch here in Ports is probably the only way we'll see a fix, as evidenced by the existing patch chasing a FLAC command syntax change. Also touch up a typo while we're here :) >How-To-Repeat: Try to run /usr/local/share/examples/cuetools/cuetag.sh on any cue sheet with a spaced file name. >Fix: --- cuetools.orig/files/patch-extras_cuetag.sh 2010-10-24 03:16:03.895004790 -0400 +++ cuetools/files/patch-extras_cuetag.sh 2010-10-24 03:20:58.637410871 -0400 @@ -1,5 +1,5 @@ ---- extras/cuetag.sh.orig 2008-11-28 00:41:26.000000000 +0300 -+++ extras/cuetag.sh 2008-11-28 00:41:32.000000000 +0300 +--- extras/cuetag.sh.orig 2006-02-14 19:10:02.000000000 -0500 ++++ extras/cuetag.sh 2010-10-24 03:20:01.349578391 -0400 @@ -18,7 +18,7 @@ { # FLAC tagging @@ -9,3 +9,47 @@ # Ogg Vorbis tagging # -w overwrites existing comments +@@ -63,7 +63,7 @@ + (for field in $fields; do + value="" + for conv in `eval echo \\$$field`; do +- value=`$CUEPRINT -n $1 -t "$conv\n" $cue_file` ++ value=`$CUEPRINT -n $1 -t "$conv\n" "$cue_file"` + + if [ -n "$value" ]; then + echo "$field=$value" +@@ -96,7 +96,7 @@ + for field in $fields; do + value="" + for conv in `eval echo \\$$field`; do +- value=`$CUEPRINT -n $1 -t "$conv\n" $cue_file` ++ value=`$CUEPRINT -n $1 -t "$conv\n" "$cue_file"` + + if [ -n "$value" ]; then + break +@@ -141,14 +141,14 @@ + cue_file=$1 + shift + +- ntrack=`cueprint -d '%N' $cue_file` ++ ntrack=`cueprint -d '%N' "$cue_file"` + trackno=1 + + if [ $# -ne $ntrack ]; then + echo "warning: number of files does not match number of tracks" + fi + +- for file in $@; do ++ for file in "$@"; do + case $file in + *.[Ff][Ll][Aa][Cc]) + vorbis $trackno "$file" +@@ -160,7 +160,7 @@ + id3 $trackno "$file" + ;; + *) +- echo "$file: uknown file type" ++ echo "$file: unknown file type" + ;; + esac + trackno=$(($trackno + 1)) >Release-Note: >Audit-Trail: >Unformatted: