summaryrefslogtreecommitdiff
path: root/audio/gqmpeg/patches
diff options
context:
space:
mode:
authorlukem <lukem@pkgsrc.org>2002-01-16 07:59:53 +0000
committerlukem <lukem@pkgsrc.org>2002-01-16 07:59:53 +0000
commit503d601cc6828f449543be388b86921283a5df97 (patch)
treeac7ebb9a17392ba1c257873eafa2ae21a19683a3 /audio/gqmpeg/patches
parent23b692bc6fda05b8103b198d66e3dc38ac1c39fe (diff)
downloadpkgsrc-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-ac31
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)