about summary refs log tree commit homepage
path: root/lib/dtas/source/av_ff_common.rb
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2021-05-25 10:22:34 +0000
committerEric Wong <e@80x24.org>2021-05-25 10:24:36 +0000
commite97a5c2b9d26db2e1dda3422d39711a1801e34a3 (patch)
treeaab99a89521d0a462dea4696cc330cdba181da6d /lib/dtas/source/av_ff_common.rb
parent96c43ff8f26cf89934198d2387cddef1f119a271 (diff)
downloaddtas-e97a5c2b9d26db2e1dda3422d39711a1801e34a3.tar.gz
This allows dtas-player to play files with wonky filenames
when piping ffmpeg (or avconv) to sox.  SoX-only code
dtas-player paths are not affected since they don't require
an extra Bourne shell.

All of our internal shell pipelines quote "$INFILE",
anyways, so there was never any need to escape for those.

This may cause compatibility problems for splitfx users, but
splitfx is probably too esoteric to have any users besides
myself.  And I expect anybody editing audio with dtas-splitfx to
pick shell-friendly filenames.

dtas-player is far more general, and likely to encounter
shell-unfriendly filenames which require quoting.
Diffstat (limited to 'lib/dtas/source/av_ff_common.rb')
-rw-r--r--lib/dtas/source/av_ff_common.rb2
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/dtas/source/av_ff_common.rb b/lib/dtas/source/av_ff_common.rb
index 88bfe50..6f92762 100644
--- a/lib/dtas/source/av_ff_common.rb
+++ b/lib/dtas/source/av_ff_common.rb
@@ -186,7 +186,7 @@ module DTAS::Source::AvFfCommon # :nodoc:
 
     e["PROBE"] = @probe_harder ? @probe_harder.join(' ') : nil
     # make sure these are visible to the source command...
-    e["INFILE"] = xs(@infile)
+    e["INFILE"] = @infile
     e["AMAP"] = amap
     e["SSPOS"] = sspos
     e["RGFX"] = rg_state.effect(self) || nil