Duplicate translated file names (security.properties)

VERIFIED FIXED in mozilla0.9.7

Status

()

Core
Security: CAPS
VERIFIED FIXED
17 years ago
16 years ago

People

(Reporter: mkaply, Assigned: Mitchell Stoltz (not reading bugmail))

Tracking

Trunk
mozilla0.9.7
x86
All
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: patch)

Attachments

(1 attachment)

(Reporter)

Description

17 years ago
Both CAPS and netwerk have security.properties:

/caps/src/security.properties 
/netwerk/resources/locale/en-US/security.properties 

This is pretty confusing for translation purposes.
(Assignee)

Comment 1

17 years ago
I could change the one in caps to be caps.properties, would that solve the problem?
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9.7
(Reporter)

Comment 2

17 years ago
That sound great to me.

Comment 3

17 years ago
leaf: if no one objects, would you rename the file on the cvs server to avoid
annotate loss?
(Assignee)

Comment 4

16 years ago
Created attachment 61191 [details] [diff] [review]
Patch - renames security.properties (the one for caps, not for necko)  to caps.properties
(Assignee)

Updated

16 years ago
Whiteboard: patch
(Assignee)

Comment 5

16 years ago
ccing nhotta: could you please review this change?

Comment 6

16 years ago
Looks fine but those localized property files are owned by L10N.
Cc to tao, rchen, please review the patch. Anybody maintaining the files under
mozilla/l10n/?

Comment 7

16 years ago
Comment on attachment 61191 [details] [diff] [review]
Patch - renames security.properties (the one for caps, not for necko)  to caps.properties

The patches for files under
mozilla/l10n are not necessary. otherwise, looks
fine.
Attachment #61191 - Flags: review+
(Assignee)

Comment 8

16 years ago
OK, I won't check those in.
(Assignee)

Comment 9

16 years ago
Comment on attachment 61191 [details] [diff] [review]
Patch - renames security.properties (the one for caps, not for necko)  to caps.properties

rs=jst.
Attachment #61191 - Flags: superreview+
(Assignee)

Comment 10

16 years ago
Fixed.
Status: ASSIGNED → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED

Comment 11

16 years ago
Marking verified as per above developer comments.

Comment 12

16 years ago
Marking it verified now.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.