8.2.0b - virtual folders not working

Hi Julian,

Does

     M-: (vm-visit-folder FOLDER nil t)

 work for these folders?

Cheers,
Uday

I couldn't reproduce this problem with 8.2.0b using a couple of folders that
I freshly converted into MMDF format. You would also need to watch and see
if something is going wrong when your component folders are being visited.

Cheers,
Uday

> M-: (vm-visit-folder FOLDER nil t)

No, it doesn't work. Unrecognized folder type.

It's not mmdf-specific, happens with vanilla mbox folders too.

Julian Bradfield writes:

> > M-: (vm-visit-folder FOLDER nil t)
>
> No, it doesn't work. Unrecognized folder type.
>
> It's not mmdf-specific, happens with vanilla mbox folders too.

So, this call is breaking for you for all mail folders?

Does it work ok if the last argument is nil?

I still can't reproduce the problem. Checked it on XEmas too. So, you will
need to narrow down the problem. I suppose you know what to do? What
exactly is vm-folder-type after the folder is visited?

Cheers,
Uday

>Does it work ok if the last argument is nil?

Yes, it does.

>I still can't reproduce the problem. Checked it on XEmas too. So, you will

How do you get vm into XEmacs? I can't even load new VM unless I first
load vm-macro and vm-vars .
(With 21.4 and 21.5, vanilla, no .vm)

>need to narrow down the problem. I suppose you know what to do? What
>exactly is vm-folder-type after the folder is visited?

'mmdf

As far as I can see, vm-build-message-list is not being called.

Ok, I found the problem. I was probably not using the exact released
version when I tested at home.

The fix is in revision 1310 of the trunk, the day after the release. The
log doesn't say anything about it. I guess I somehow missed putting this in
the release and forgot all about it.

> How do you get vm into XEmacs? I can't even load new VM unless I first
> load vm-macro and vm-vars .
> (With 21.4 and 21.5, vanilla, no .vm)

I have this in my init file.

(if running-fsf
    (require 'vm-autoloads)
  ;; Use this only if there is no site-package installed
  (load-library (substitute-in-file-name "$VMDIR/lisp/auto-autoloads"))
  )

VMDIR is the directory where I built VM. I don't use a "package" normally,
but if the package is installed, XEmacs is supposed to load its
auto-autoloads file automatcially.

Cheers,
Uday

Is the fix for this the following:

@@ -285,7 +286,7 @@
       ;; builds message list, reads attributes if they weren't
       ;; read from an index file.
       ;; but that is not what the code is doing! - USR, 2011-04-24
- (unless revisit
+ (unless revisiting
        (vm-assimilate-new-messages :read-attributes t
                                    :gobble-order (not did-read-index-file)
                                    :run-hooks nil))

If so, this is already in the 8.2.x branch. I just wanted to make sure it'll be in 8.2.0

Yup. Whatever is in 8.2.x branch will get included in 8.2.0.

Julian, do you want to download the tarball from

  http://bazaar.launchpad.net/~vm/vm/8.2.x/revision/1323

and see if it works?

Cheers,
Uday

I've already tried 1310, and it was ok.

Notes:

The change seems to have been made in rev. 1194, to handle Bug 614413.

Actually the original change was made in rev. 653.1.107, to handle Bug 561718.

Discussion on viewmail-info (2012-02-02):

>Hi Julian, this might be a duplicate of
> https://bugs.launchpad.net/vm/+bug/909501
>Can you try revision 1311 please?

Yes, that fixes the folder changing.
However, it's still creating Summary and Presentation buffers for all
the component folders, which seems highly undesirable. (Especially
since I sometimes use virtual folders with a hundred component
folders!)

Rev. 1351 suppresses the summary generation and retrieval of new mail etc. Presentation buffer is still created. (This was present in VM 8.1.1 as well.)

We might consider getting rid of the Presentation buffer sometime down the road.

Julian, can you test to see if this works ok?

Cheers,
Uday