Typo in xulsecurity.mn

VERIFIED FIXED in mozilla1.0.1

Status

VERIFIED FIXED
17 years ago
3 years ago

People

(Reporter: mikepinkerton, Assigned: mikepinkerton)

Tracking

({topembed})

Trunk
mozilla1.0.1
PowerPC
Mac OS X
topembed

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [needs a= drivers emailed 6/6])

Attachments

(1 attachment, 1 obsolete attachment)

as tao pointed out in email, there's a typo in the xul security manifest file:

 7 locale/XXXX/pipnss, XXXX/locale/XXXX/pippki

which should read

 7 locale/XXXX/pippki, XXXX/locale/XXXX/pippki

patch upcoming
Created attachment 86399 [details] [diff] [review]
fix typo

can i get r/sr here?
(Assignee)

Updated

17 years ago
Status: NEW → ASSIGNED
Keywords: topembed
Target Milestone: --- → mozilla1.0.1
Comment on attachment 86399 [details] [diff] [review]
fix typo

r=ccarlen
Attachment #86399 - Flags: review+

Comment 4

17 years ago
Comment on attachment 86399 [details] [diff] [review]
fix typo

>Index: mozilla/embedding/config/xulsecurity.mn
>===================================================================
>RCS file: /cvsroot/mozilla/embedding/config/xulsecurity.mn,v
>retrieving revision 1.1.2.1
>diff -u -2 -r1.1.2.1 xulsecurity.mn
>--- mozilla/embedding/config/xulsecurity.mn	17 Apr 2002 14:11:14 -0000	1.1.2.1
>+++ mozilla/embedding/config/xulsecurity.mn	5 Jun 2002 13:57:00 -0000
>@@ -5,5 +5,5 @@
> 
> content/pipnss, pipnss/content/pipnss
>-locale/XXXX/pipnss, XXXX/locale/XXXX/pippki

>+locale/XXXX/pippki, XXXX/locale/XXXX/pippki
Should this be "pipnss" like below instead?

+locale/XXXX/pipnss, XXXX/locale/XXXX/pipnss
> 
>
Comment on attachment 86399 [details] [diff] [review]
fix typo

tao, you're right, i'm an idiot.
Attachment #86399 - Attachment is obsolete: true
Created attachment 86412 [details] [diff] [review]
again, the correct fix

Comment 7

17 years ago
Comment on attachment 86412 [details] [diff] [review]
again, the correct fix

r=tao
Attachment #86412 - Flags: review+

Comment 8

17 years ago
Comment on attachment 86412 [details] [diff] [review]
again, the correct fix

sr=alecf
Attachment #86412 - Flags: superreview+
(Assignee)

Updated

17 years ago
Keywords: adt1.0.1, mozilla1.0.1
Whiteboard: [needs a= drivers emailed 6/6]

Comment 9

17 years ago
please checkin to the 1.0.1 branch. once there, remove the "mozilla1.0.1+"
keyword, and add the "fixed1.0.1" keyword.
Keywords: mozilla1.0.1 → mozilla1.0.1+

Updated

17 years ago
Attachment #86412 - Flags: approval+

Comment 10

17 years ago
adding adt1.0.1+ for checkin to 1.0.1
Keywords: adt1.0.1 → adt1.0.1+

Comment 11

17 years ago
please check this into the branch as soon as possible.
landed on trunk and branch
Keywords: adt1.0.1+ → fixed1.0.1
really fixed
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED

Updated

17 years ago
Keywords: mozilla1.0.1+
BRANCH:
http://lxr.mozilla.org/mozilla1.0/source/embedding/config/xulsecurity.mn#7

6 content/pipnss, pipnss/content/pipnss
7 locale/XXXX/pipnss, XXXX/locale/XXXX/pipnss

TRUNK:
http://lxr.mozilla.org/seamonkey/source/embedding/config/xulsecurity.mn#7

6 content/pipnss, pipnss/content/pipnss
7 locale/XXXX/pipnss, XXXX/locale/XXXX/pipnss

Verified FIXED using LXR.
Status: RESOLVED → VERIFIED
Keywords: fixed1.0.1 → verified1.0.1
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.