# [RFC] OSX Mail.app IMAP cache support for git-mailsplit?

4 messages from 2007-11-04 to 2007-11-05. Participants: Michael J. Cohen, Michael Cohen, Johannes Schindelin.
Thread: https://gitlist.dev/t/10632

## Michael J. Cohen, 2007-11-04 05:49

Subject: [RFC] OSX Mail.app IMAP cache support for git-mailsplit?
Message-ID: <06FE21A2-20D0-4AAA-B0C7-35783C604B68@mac.com>
URL: https://gitlist.dev/e/06FE21A2-20D0-4AAA-B0C7-35783C604B68%40mac.com

```
Trivially, adding support for checking for Messages/ inside the  
specified Maildir after cur/ is found not to exist would be enough to  
make this work.

Is there interest in a patch for this at all?

-mjc

```

## Michael Cohen, 2007-11-05 06:36

Subject: [RFC PATCH] OSX Mail.app IMAP cache support for git-mailsplit?
Message-ID: <7B209F05-B720-41D6-AE98-39FAFF04B9F6@mac.com>
URL: https://gitlist.dev/e/7B209F05-B720-41D6-AE98-39FAFF04B9F6%40mac.com
In-Reply-To: <06FE21A2-20D0-4AAA-B0C7-35783C604B68@mac.com>

```
On Nov 4, 2007, at 1:49 AM, Michael J. Cohen wrote:

> Trivially, adding support for checking for Messages/ inside the  
> specified Maildir after cur/ is found not to exist would be enough  
> to make this work.

my repo at git://home.325i.org/git-osxmail.git should have that portion.

unsure as to whether to make it an option, a fallback, a config value,  
or whatever...

-mjc

```

## Johannes Schindelin, 2007-11-05 10:39

Subject: Re: [RFC PATCH] OSX Mail.app IMAP cache support for git-mailsplit?
Message-ID: <Pine.LNX.4.64.0711051034060.4362@racer.site>
URL: https://gitlist.dev/e/Pine.LNX.4.64.0711051034060.4362%40racer.site
In-Reply-To: <7B209F05-B720-41D6-AE98-39FAFF04B9F6@mac.com>

```
Hi,

you have a very weird mail setting; I had to add the git list back to the 
Cc.  This is just annoying enough for me to write an extra paragraph to 
annoy you back ;-)

On Mon, 5 Nov 2007, Michael Cohen wrote:

> On Nov 4, 2007, at 1:49 AM, Michael J. Cohen wrote:
> 
> > Trivially, adding support for checking for Messages/ inside the specified
> > Maildir after cur/ is found not to exist would be enough to make this work.
> 
> my repo at git://home.325i.org/git-osxmail.git should have that portion.
> 
> unsure as to whether to make it an option, a fallback, a config value, or
> whatever...

A fallback would be sufficient.

Several comments (your patch not inlined, since you did not inline it 
either):

- there needs to be a space between the ) and the { in the first if line.

- you probably forgot to remove the original "if (populate...)...".  That 
  means that populate would be called _twice_, even if successful.

- git is written in C.  Therefore, "//" as a way to comment out is wrong.

- if you still return -1 when the dir could not be opened, I wonder what 
  the rationale is to comment the error out.

Ciao,
Dscho

P.S.: You might want to send patches as these right away, without asking 
if anybody cares (you'll see that very soon), but rather in accord with 
Documentation/SubmittingPatches.

```

## Michael Cohen, 2007-11-05 12:09

Subject: Re: [RFC PATCH] OSX Mail.app IMAP cache support for git-mailsplit?
Message-ID: <7B84FFB7-5FE9-4D24-AEC5-43F667B59876@mac.com>
URL: https://gitlist.dev/e/7B84FFB7-5FE9-4D24-AEC5-43F667B59876%40mac.com
In-Reply-To: <Pine.LNX.4.64.0711051034060.4362@racer.site>

```
On Nov 5, 2007, at 5:39 AM, Johannes Schindelin wrote:

> Hi,
>
> you have a very weird mail setting; I had to add the git list back  
> to the
> Cc.  This is just annoying enough for me to write an extra paragraph  
> to
> annoy you back ;-)
Have to get used to this; thank you. :)

> Several comments (your patch not inlined, since you did not inline it
> either):
>
> - there needs to be a space between the ) and the { in the first if  
> line.
Doh. done.

> - you probably forgot to remove the original "if (populate...)...".   
> That
>  means that populate would be called _twice_, even if successful.

good catch.

> - git is written in C.  Therefore, "//" as a way to comment out is  
> wrong.

> - if you still return -1 when the dir could not be opened, I wonder  
> what
>  the rationale is to comment the error out.

More work needs to be done in there and in builtin-mailinfo.c to  
massage the mail format that Apple is using.  Also what I think I need  
to do there is check the path that is being tested and print something  
like "%s/cur could not be found, trying alternate path" on the first  
test?


> P.S.: You might want to send patches as these right away, without  
> asking
> if anybody cares (you'll see that very soon), but rather in accord  
> with
> Documentation/SubmittingPatches.

Thanks for that. 
  

```