From: Eric Wong <e@80x24.org>
To: dtas-all@nongnu.org
Subject: [PATCH 1/4] archive: support comments, default to none
Date: Fri, 7 Jan 2022 05:35:41 +0000 [thread overview]
Message-ID: <20220107053544.31240-2-e@80x24.org> (raw)
In-Reply-To: <20220107053544.31240-1-e@80x24.org>
The default "Processed by SoX" comment is pointless, and I often
want to archive + tag something in one step to reduce FS I/O.
---
bin/dtas-archive | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/bin/dtas-archive b/bin/dtas-archive
index dd3b537..c88873e 100755
--- a/bin/dtas-archive
+++ b/bin/dtas-archive
@@ -29,6 +29,7 @@
stats = false
keep_going = false
compression = []
+comment = []
OptionParser.new('', 24, ' ') do |op|
op.banner = usage
@@ -36,6 +37,7 @@
op.on('-C', '--compression [FACTOR]', 'compression factor for sox') { |c|
compression = [ '-C', c ]
}
+ op.on('--comment=TEXT', String) { |c| comment.push('--comment', c) }
op.on('-j', '--jobs [JOBS]', Integer) { |j| jobs = j }
op.on('-S', '--stats', 'save stats on the file') { stats = true }
op.on('-k', '--keep-going', 'continue after error') { keep_going = true }
@@ -53,6 +55,8 @@
op.parse!(ARGV)
end
+comment.push('--comment', '') if comment.empty?
+
dst = ARGV.pop
src = ARGV.dup
@@ -137,7 +141,7 @@
if dry_run || !silent
names = job.map { |x| Shellwords.escape(x) }
- cmd = [ 'sox', *names ]
+ cmd = [ 'sox', names[0], *compression, *comment, names[1] ]
if stats
cmd << 'stats'
cmd << "2>#{Shellwords.escape(stats_out)}"
@@ -151,7 +155,7 @@
end
end
- cmd = [ 'sox', input, *compression, output ]
+ cmd = [ 'sox', input, *compression, *comment, output ]
if stats
cmd << 'stats'
cmd = [ *cmd, { err: stats_out } ]
next prev parent reply other threads:[~2022-01-07 5:36 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-01-07 5:35 [PATCH 0/4] a pile of misc improvements Eric Wong
2022-01-07 5:35 ` Eric Wong [this message]
2022-01-07 5:35 ` [PATCH 2/4] splitfx: use Etc.nprocessors for jobs if unspecified Eric Wong
2022-01-07 5:35 ` [PATCH 3/4] dtas-console: set X11 terminal title iff DISPLAY is set Eric Wong
2022-01-08 16:59 ` James Rowe
2022-01-11 17:29 ` Eric Wong
2022-01-11 18:14 ` James Rowe
2022-01-11 18:26 ` Eric Wong
2022-01-07 5:35 ` [PATCH 4/4] dtas-console: add 'i' toggle to show comments (metadata) Eric Wong
2022-01-07 9:34 ` [PATCH 5/4] splitfx: fix track_zpad with integer arg Eric Wong
2022-01-07 9:35 ` [PATCH 6/4] doc: drop ordered map from examples Eric Wong
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: http://80x24.org/dtas/README
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20220107053544.31240-2-e@80x24.org \
--to=e@80x24.org \
--cc=dtas-all@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
http://80x24.org/dtas.git/
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).