about summary refs log tree commit homepage
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
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.
-rw-r--r--lib/dtas/partstats.rb3
-rw-r--r--lib/dtas/source/av_ff_common.rb2
-rw-r--r--lib/dtas/source/sox.rb2
-rw-r--r--lib/dtas/splitfx.rb8
4 files changed, 6 insertions, 9 deletions
diff --git a/lib/dtas/partstats.rb b/lib/dtas/partstats.rb
index 75f9236..45eff34 100644
--- a/lib/dtas/partstats.rb
+++ b/lib/dtas/partstats.rb
@@ -3,7 +3,6 @@
 # License: GPL-3.0+ <https://www.gnu.org/licenses/gpl-3.0.txt>
 # frozen_string_literal: true
 require_relative '../dtas'
-require_relative 'xs'
 require_relative 'process'
 require_relative 'sigevent'
 
@@ -56,7 +55,7 @@ class DTAS::PartStats # :nodoc:
     rd, wr = IO.pipe
     env = opts[:env]
     env = env ? env.dup : {}
-    env["INFILE"] = xs(@infile)
+    env["INFILE"] = @infile
     env["TRIMFX"] = "trim #{trim_part.tbeg}s #{trim_part.tlen}s"
     opts = { pgroup: true, close_others: true, err: wr }
     pid = spawn(env, CMD, opts)
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
diff --git a/lib/dtas/source/sox.rb b/lib/dtas/source/sox.rb
index d63f0df..3a7fe7d 100644
--- a/lib/dtas/source/sox.rb
+++ b/lib/dtas/source/sox.rb
@@ -113,7 +113,7 @@ class DTAS::Source::Sox # :nodoc:
   def src_spawn(player_format, rg_state, opts)
     raise "BUG: #{self.inspect}#src_spawn called twice" if @to_io
     e = @env.merge!(player_format.to_env)
-    e["INFILE"] = xs(@infile)
+    e["INFILE"] = @infile
 
     # make sure these are visible to the "current" command...
     e["TRIMFX"] = trimfx
diff --git a/lib/dtas/splitfx.rb b/lib/dtas/splitfx.rb
index 58c8ff4..9e1cfd0 100644
--- a/lib/dtas/splitfx.rb
+++ b/lib/dtas/splitfx.rb
@@ -4,7 +4,6 @@
 require_relative '../dtas'
 require_relative 'format'
 require_relative 'process'
-require_relative 'xs'
 require 'tempfile'
 
 # The backend for dtas-splitfx(1) command, but also supported by dtas-player
@@ -13,7 +12,6 @@ require 'tempfile'
 class DTAS::SplitFX # :nodoc:
   CMD = 'sox "$INFILE" $COMMENTS $OUTFMT $OUTDST $TRIMFX $FX $RATEFX $DITHERFX'
   include DTAS::Process
-  include DTAS::XS
   attr_reader :infile, :env, :command
 
   # for --trim on the command-line
@@ -404,10 +402,10 @@ class DTAS::SplitFX # :nodoc:
   end
 
   def infile_env(env, infile)
-    env["INFILE"] = xs(infile)
+    env["INFILE"] = infile
     dir, base = File.split(File.expand_path(infile))
-    env["INDIR"] = xs(dir)
-    env["INBASE"] = xs(base)
+    env["INDIR"] = dir
+    env["INBASE"] = base
   end
 
   def expand_cmd(env, command) # for display purposes only