typo with maxsize and maxage

Nicolas Sebrecht nicolas.s-dev at laposte.net
Sat Jun 25 16:18:55 BST 2011


The commit title (email title) should say:

  folder/IMAP: fix typo with maxsize and maxage


On Fri, Jun 24, 2011 at 10:23:37PM +0800, Haojun Bao wrote:
> Hi,
> 
> I found the maxsize setting unusable with Gmail, the backtrace:

We don't say "hi" or "I found" in a commit message.

>     WARNING: ERROR in syncfolder for Gmail folder LetouWork: Traceback (most recent call last):
>       File "/usr/lib/pymodules/python2.6/offlineimap/accounts.py", line 347, in syncfolder
>         remotefolder.cachemessagelist()
>       File "/usr/lib/pymodules/python2.6/offlineimap/folder/IMAP.py", line 160, in cachemessagelist
>         searchresult = imapobj.search(None, search_condition)
>       File "/usr/lib/pymodules/python2.6/offlineimap/imaplib2.py", line 904, in search
>         return self._simple_command(name, *criteria, **kw)
>       File "/usr/lib/pymodules/python2.6/offlineimap/imaplib2.py", line 1492, in _simple_command
>         return self._command_complete(self._command(name, *args), kw)
>       File "/usr/lib/pymodules/python2.6/offlineimap/imaplib2.py", line 1272, in _command_complete
>         raise self.error('%s command error: %s %s. Data: %.100s' % (rqb.name, typ, dat, rqb.data))
>     error: SEARCH command error: BAD ['Could not parse command']. Data: OCJN5 SEARCH ( SMALLER 500000)
> 
> See the extra space in "( SMALLER 500000)"?

Traceback worth it.

>                                             It's fixed by the following
> patch:
> 
> commit ef32ac1a8c8966d2e2dcac8531b4cd3161952b5b
> Author: baohaojun <baohaojun at gmail.com>
> Date:   Fri Jun 24 22:10:40 2011 +0800
> 
>     maxsize/maxage: fix typo.

These extra lines aren't expected and are cleary wrong. The commit
number won't be the same in the git history. The patch exists as a plain
text file or mbox, for now. The number will be generated while applying
it in my local repository.

At the end of the mail you're missing the --  field.

Notice that 'git format-patch' and 'git send-email' do care about most
of these issues for you. ,-)

Please, fix and resend!

-- 
Nicolas Sebrecht




More information about the OfflineIMAP-project mailing list