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

Sebastian Spaeth Sebastian at SSpaeth.de
Tue Apr 26 11:13:07 BST 2011


On Thu, 21 Apr 2011 19:24:18 -0400, Dan Christensen <jdc at uwo.ca> wrote:
> Sebastian Spaeth <Sebastian at SSpaeth.de> writes:

> 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. Just note that I guess that
pythons command line parser will strip spaces anyway unless things are
properly quoted, so

-f     moo,foo,boo

will be the same as:
-f moo,foo,boo

but different from:

-f " moo,foo,boo"

In any case, I am more than happy to not do any stripping at all, just
fine with me. Care to send the patch to see if Nicolas accepts it :-)?

Sebastian
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://alioth-lists.debian.net/pipermail/offlineimap-project/attachments/20110426/b561c9b1/attachment-0001.sig>


More information about the OfflineIMAP-project mailing list