Spaces in foldernames for the -f cli flag

[Sebastian Spaeth]

On Mon, 4 Apr 2011 22:59:49 -0600, Will Styler <william.styler at colorado.edu> wrote:
> I cannot get the other two ("Deleted Messages" and "Sent Message") to
> sync with this command, no matter how I escape them.  I've tried quotes,
> brackets and slashes, backslashes, plain... I'm at the end of my rope.  

no wonder, see below:
 
> So, any suggestions on how I can ask offlineimap to check these other
> folders alongside INBOX from the CLI?

> /opt/local/bin/offlineimap -o -u Noninteractive.Quiet -f INBOX,[Deleted\
> Messages],[Sent\ Messages] >/dev/null

Mmmh, specifying folders on the command line seems to strip away *all*
spaces (which does not make sense in my opinion). This logic has never
been changed so far, so it always behaved like that.
init.py:
            foldernames = options.folders.replace(" ", "").split(",")

Wouldn't it make more sense, to strip just spaces around the commas of a
list? Such as: init.py:

-            foldernames = map(strip, options.folders.split(","))
+            map(lambda s: s.strip(), options.folders.split(","))

? I think that change would make sense, but should be documented in the
Changelog.

The other option (which should currently work without code change), is to override the
config option in the form:

-k "Repository_LocalFOO:folderfilter= lambda f: f in ['INBOX','[Deleted Messages]','[Sent Messages]']"

Does that work?

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/20110405/65fde545/attachment-0001.sig>