diff options
author | lukem <lukem@pkgsrc.org> | 2002-01-16 07:59:53 +0000 |
---|---|---|
committer | lukem <lukem@pkgsrc.org> | 2002-01-16 07:59:53 +0000 |
commit | 503d601cc6828f449543be388b86921283a5df97 (patch) | |
tree | ac7ebb9a17392ba1c257873eafa2ae21a19683a3 /audio/gqmpeg/patches | |
parent | 23b692bc6fda05b8103b198d66e3dc38ac1c39fe (diff) | |
download | pkgsrc-503d601cc6828f449543be388b86921283a5df97.tar.gz |
Wrap path name arguments to popen in single quotes, to fix getting info from
filenames containing spaces, etc. (It's still borken for filenames with
single quotes).
XXX: Arguably, io_ogg123.c should be converted to not use popen(3) and
use a pipe mechanism similar to io_mpg123.c, but that's more effort
than I'm interested in doing right now.
Diffstat (limited to 'audio/gqmpeg/patches')
-rw-r--r-- | audio/gqmpeg/patches/patch-ac | 31 |
1 files changed, 31 insertions, 0 deletions
diff --git a/audio/gqmpeg/patches/patch-ac b/audio/gqmpeg/patches/patch-ac new file mode 100644 index 00000000000..afbbbe60c20 --- /dev/null +++ b/audio/gqmpeg/patches/patch-ac @@ -0,0 +1,31 @@ +$NetBSD: patch-ac,v 1.6 2002/01/16 07:59:54 lukem Exp $ + +--- src/io_ogg123.c.orig Thu Jan 3 19:46:03 2002 ++++ src/io_ogg123.c +@@ -105,7 +105,7 @@ + + if (!ogg123_info_found) return NULL; + +- command = g_strdup_printf("%s %s", OGG123_BINARY_INFO, path); ++ command = g_strdup_printf("%s '%s'", OGG123_BINARY_INFO, path); + f = popen(command, "r"); + if (!f) + { +@@ -183,7 +183,7 @@ + + if (!ogg123_comment_found) return NULL; + +- command = g_strdup_printf("%s -l %s", OGG123_BINARY_COMMENT, path); ++ command = g_strdup_printf("%s -l '%s'", OGG123_BINARY_COMMENT, path); + f = popen(command, "r"); + if (!f) + { +@@ -259,7 +259,7 @@ + if (!ogg123_comment_found) return TRUE; + if (!path) return FALSE; + +- command = g_strdup_printf("%s -w %s", OGG123_BINARY_COMMENT, path); ++ command = g_strdup_printf("%s -w '%s'", OGG123_BINARY_COMMENT, path); + + work = comments; + while (work && work->next) |