[PATCH] Make -f option with with folder names with spaces.

Dan Christensen jdc at uwo.ca
Wed Apr 27 15:26:16 BST 2011


Sebastian Spaeth <Sebastian at SSpaeth.de> writes:

> On Thu, 21 Apr 2011 19:24:18 -0400, Dan Christensen <jdc at uwo.ca> wrote:
>
>> I think we should parse this in the simplest way possible, which is to
>> simply split on the commas and leave spaces as is.  Why limit how the
>> user can use this?
>> 
>> So this:
>> 
>>             foldernames = map(lambda s: s.strip(),
>>                               options.folders.split(","))
>> 
>> should be replaced with
>> 
>>             foldernames = options.folders.split(",")
>
> Personally, I am file with that one too.
...
> Care to send the patch to see if Nicolas accepts it :-)?

I'm not set-up to produce patches, so the above is all I can do right
now.  It's not a big deal, but I think it would be better if the above
change was made.

The changelog could say:

Allow leading and trailing spaces in folder names specified on the
command line.

Signed-off-by: Dan Christensen <jdc at uwo.ca>

Dan





More information about the OfflineIMAP-project mailing list