[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