12932 - [PREF MIGRATION] heed "news.directory", "mail.imap.root_dir", and "mail.directory" upon pref migration

Status: VERIFIED FIXED

(MailNews Core :: Backend, defect, P3)

Description

[(not reading, please use [e-mail] instead)]

dbragg and I talked about this a while back.  this bug is track it.

in mozilla/mailnews/base/src/nsMsgAccountManager.cpp is the code for pref
migration for mail news.

Right now, I'm assuming that after migration, the mail and news files get copied
to the right place under <profile>/Mail, <profile>/News, and <profile>/ImapMail

when dbragg copies the files, he looks at the
"mail.directory","mail.imap.root_dir","news.directory" prefs to find the files.

when we lasted talked we were going to do this:

after migration, he was going to set those prefs to point to the place he copied
to the 4.x files.  on pref migration, I would read those values and use them
when creating the 5.0 server prefs, instead of assuming they got copied to
<profile>/Mail, <profile>/News, and <profile>/ImapMail.

this bug is track that work.

Comment 1

[(not reading, please use [e-mail] instead)]

marking m11.
accepting.

I need dbragg for part of this fix.

dbragg, does m11 work for you?

Comment 2

[(not reading, please use [e-mail] instead)]

adding dependency.

Comment 3

[(not reading, please use [e-mail] instead)]

I've just made the necessary changes to fix this bug.

the code is not turned on, because I'm waiting for #13257 to be fixed.

until then, the I'm going to keep assuming that the Mail, News, and ImapMail
dirs are under the profile directory.

Comment 4

[(not reading, please use [e-mail] instead)]

adding momoi to the cc list, as he is seeing the side effect of this not being
fixed yet.

I have already implemented the code.  as soon as I do a little more testing,
I'll turn it on.  (expect me to mark this fixed in an hour or so.)

Comment 5

[(not reading, please use [e-mail] instead)]

fixed.

momoi, try -installer again.

reminder, don't run -installer twice with out moving aside the place the files
are being copied to.

look for Mail5, ImapMail5, and News5 directories next to your 4.x Mail,
ImapMail, and News directories.  remove the *5 directories before you run
-installer, otherwise you'll install over a previous install, and that can be
bad.

Comment 6

[lchiang]

Grace - can you verify this?  Thanks.

Comment 7

[Grace Bush]

verified in Mac/Win builds for 19991006 and Linux 19991007