Closed Bug 1688487 Opened 1 year ago Closed 1 year ago

Move suite/security to suite/components/security

Categories

(SeaMonkey :: Build Config, task)

Tracking

(seamonkey2.53+ fixed, seamonkey2.57esr? affected)

RESOLVED FIXED
seamonkey 2.84
Tracking Status
seamonkey2.53 + fixed
seamonkey2.57esr ? affected

People

(Reporter: iannbugzilla, Assigned: iannbugzilla)

Details

(Whiteboard: SM2.53.7)

Attachments

(1 file)

I see no reason why SeaMonkey's security prefs should sit in suite/security they would live better under suite/components/security

Just as a note from someone who is (nowadays) an outsider: Some years ago, we were trying to get rid of /components/ and move everything to directories outside of that structure, are things going the other way again now?

(In reply to Robert Kaiser from comment #1)

Just as a note from someone who is (nowadays) an outsider: Some years ago, we were trying to get rid of /components/ and move everything to directories outside of that structure, are things going the other way again now?

Seems so :)

[Approval Request Comment]
Regression caused by (bug #): n/a
User impact if declined: none
Testing completed (on m-c, etc.): 2.53.7
Risk to taking this patch (and alternatives if risky): none
String changes made by this patch: none

Attachment #9198888 - Flags: review?(frgrahl)
Attachment #9198888 - Flags: approval-comm-release?
Attachment #9198888 - Flags: approval-comm-esr60?

Comment on attachment 9198888 [details] [diff] [review]
1688487-security-2537.patch

LGTM. Thought about putting it in prefs but this would probabaly need more work.

Robert we are basically aligning with Firefox and Thunderbird here. Odd man out is usually fine but makes no sense to always search were stuff is in the other products if you need to check something :)

Attachment #9198888 - Flags: review?(frgrahl)
Attachment #9198888 - Flags: review+
Attachment #9198888 - Flags: approval-comm-release?
Attachment #9198888 - Flags: approval-comm-release+
Attachment #9198888 - Flags: approval-comm-esr60?
Attachment #9198888 - Flags: approval-comm-esr60+

Pushed by frgrahl@gmx.net:
https://hg.mozilla.org/comm-central/rev/ed64964f05e6
Move suite/security to suite/components/security. r=frg

Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Whiteboard: SM2.53.7
Target Milestone: --- → Future
Target Milestone: Future → seamonkey 2.84
You need to log in before you can comment on or make changes to this bug.