71641 - Profile mgr makes subdirs in a new profile dir which are not nesc needed

Status: VERIFIED FIXED

(Core Graveyard :: Profile: BackEnd, enhancement)

Description

[Conrad Carlen (not reading bugmail)]

When the profile mgr makes a new profile dir, it creates "IMapMail", "Mail", and
"News" directories in the new profile dir. If an embedding app is not using
mailnews, these extra dirs look kinda silly/sloppy/confusing. Since these dirs
are gotten from directory service, the users of these dirs should get them,
check for their existence, and make them if they don't exist. This may be
happening already - I'm not sure. Bhuvan, do you know? In any case the profile
mgr should not create these dirs up front.

Comment 1

[Conrad Carlen (not reading bugmail)]

Adding embed keyword.

Comment 2

[Conrad Carlen (not reading bugmail)]

Attachment: patch to remove creation of these dirs

Comment 3

[Conrad Carlen (not reading bugmail)]

I verified that all of the users of these file in mailnews check for the
existence of these dirs and create them if they dont exist. The folder called
"Cache" is not needed because, with the old cache, the app created the dir and
the new cache uses a dir called "New Cache" which it checks for and creates. We
can yank this code without fear. Bhuvan, Seth - Can you r=/sr= Want to get this
one in for 0.9.

Comment 4

[Conrad Carlen (not reading bugmail)]

Alec, I know it's last minute but if you have a moment, can you sr? It's low
risk and would be nice for embeddors using 0.9.

Comment 5

[Alec Flett]

sr=alecf, contingent on an r= from sspitzer (i.e. so we know that the mail team
is cool with having to create their own subdirectories)

Comment 6

[Conrad Carlen (not reading bugmail)]

Thanks - I stepped through and found that they already were (creating dirs) but
Seth, if you can verify, that would be good.

Comment 7

[Judson Valeski]

r=valeski.

Comment 8

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

looking at (and testing) this patch now.

Comment 9

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

sorry for the delay.

r=sspitzer

Comment 10

[Conrad Carlen (not reading bugmail)]

Fix checked in.

Comment 11

[Grace Bush]

buiild 2001052504