diff options
author | Eric Wong <e@80x24.org> | 2021-05-25 10:22:34 +0000 |
---|---|---|
committer | Eric Wong <e@80x24.org> | 2021-05-25 10:24:36 +0000 |
commit | e97a5c2b9d26db2e1dda3422d39711a1801e34a3 (patch) | |
tree | aab99a89521d0a462dea4696cc330cdba181da6d /lib/dtas/source/sox.rb | |
parent | 96c43ff8f26cf89934198d2387cddef1f119a271 (diff) | |
download | dtas-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/sox.rb')
-rw-r--r-- | lib/dtas/source/sox.rb | 2 |
1 files changed, 1 insertions, 1 deletions
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 |