search

leahneukirchen / mblaze

Unix utilities to deal with Maildir
Other
419 stars 48 forks source link

mcom: quote sequence for mbnc and mrep #231

Closed codesoap closed 1 year ago

codesoap commented 1 year ago

I was getting the "mrep: needs exactly one mail to reply to" error when trying to reply to messages, where the file was located in a directory with spaces. With this change, I was able to use mrep in such a case. I noticed the same code in mbnc, so I also applied the change there.

leahneukirchen commented 1 year ago

I think this should happen by setting IFS to newline instead, as else the warning is never triggered (and just another error occurs, I think.)

codesoap commented 1 year ago

Oh, right, thanks for the correction. I've updated the commit. The error now occurs, when trying to reply to multiple e-mails, but not when a directory contains a space.

leahneukirchen commented 1 year ago

Unfortunately, set doesn't work inside (). I think you'll need to save and restore it instead.

codesoap commented 1 year ago

Feels like I'm on the amateur hour over here. Sorry. I've updated the commit again. Hopefully it's correct this time.