uchicago-library / attachment-converter

Attachment Converter: tool for batch converting attachments in an email mailbox
GNU General Public License v2.0
8 stars 3 forks source link

Progress bar should print to `/dev/tty` #72

Closed bufordrat closed 1 year ago

bufordrat commented 1 year ago

Per our discussion on the #email_archives Slack channel, it would make more sense for the progress bar to print to /dev/tty rather than to stderr.

Currently, the progress bar is getting printed to stderr, which makes it visible by default. However, if the user redirects stderr to a file, which they may want to do in order to carefully inspect the error messages, then the progress bar is silenced. (The reason for this is that our current progress bar-printing function only prints conditionally on stderr being a tty.)

It should probably instead implement the following logic: