711325 - Remove "security.enable_ssl2" remnant in nsThunderbirdProfileMigrator.cpp

Status: VERIFIED FIXED

(SeaMonkey :: Startup & Profiles, defect)

Description

[Serge Gautherie (:sgautherie)]

{
/suite/profile/migration/src/nsThunderbirdProfileMigrator.cpp
    * line 497 -- MAKESAMETYPEPREFTRANSFORM("security.enable_ssl2", Bool),
}

Just delete the line.

Comment 1

[pallavi]

Attachment: "security.enable_ssl2" remnant in nsThunderbirdProfileMigrator.cpp has been removed

Comment 2

[Serge Gautherie (:sgautherie)]

Pallavi, see
https://developer.mozilla.org/en/Mercurial_FAQ#How_can_I_generate_a_patch_for_somebody_else_to_check-in_for_me.3F

Comment 3

[Serge Gautherie (:sgautherie)]

Please, don't 'checkin-needed' until you have attached a patch according to comment 2.

Comment 4

[pallavi]

Oh ya, thank you

Comment 5

[Philip Chee]

> Please, don't 'checkin-needed' until you have attached a patch
Err, Serge, there's a patch attached with r+

Comment 6

[Serge Gautherie (:sgautherie)]

(In reply to Philip Chee from comment #5)
> > Please, don't 'checkin-needed' until you have attached a patch
> Err, Serge, there's a patch attached with r+

Err, read my comment to its end...

Comment 7

[pallavi]

Attachment: "security.enable_ssl2" remnant in nsThunderbirdProfileMigrator.cpp has been removed

Comment 8

[Serge Gautherie (:sgautherie)]

Comment on attachment 607134 [details] [diff] [review]
"security.enable_ssl2" remnant in nsThunderbirdProfileMigrator.cpp has been removed

That's almost it:

>"security.enable_ssl2" remnant in nsThunderbirdProfileMigrator.cpp has been removed

You must add the bug number.
In this case, you can just use
"Bug 711325. Remove "security.enable_ssl2" remnant in nsThunderbirdProfileMigrator.cpp. f=sgautherie r=IanN."

Comment 9

[pallavi]

Sir,is this the only change i need to make?

Comment 10

[pallavi]

Attachment: patch3

Comment 11

[Ryan VanderMeulen [:RyanVM]]

Thanks for the patch!

http://hg.mozilla.org/comm-central/rev/6665f0a5e9c9

Comment 12

[Serge Gautherie (:sgautherie)]

V.Fixed, per MXR search.

Comment 13

[pallavi]

Thanks for verification...:)