The default bug view has changed. See this FAQ.

Some packages are registering PACKAGE instead of CONTENT

RESOLVED FIXED

Status

SeaMonkey
Installer
RESOLVED FIXED
11 years ago
9 years ago

People

(Reporter: neil@parkwaycc.co.uk, Assigned: neil@parkwaycc.co.uk)

Tracking

({fixed-seamonkey1.1})

Trunk
fixed-seamonkey1.1

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

11 years ago
PACKAGE = CONTENT | SKIN | LOCALE so it should only be used if one contents.rdf file describes all three types of resources. However our .jar files contain one contents.rdf for each resource and they are all separately registered.
(Assignee)

Comment 1

11 years ago
Created attachment 246966 [details] [diff] [review]
Proposed patch
Assignee: xpi-packages → neil
Status: NEW → ASSIGNED
Attachment #246966 - Flags: review?(ajschult)

Comment 2

11 years ago
Comment on attachment 246966 [details] [diff] [review]
Proposed patch

right.  Someone was actually complaining this week in #seamonkey that those lines were missing from the .zip build's installed-chrome.txt.  :)

note that build/ and mac/ are obsolete...
Attachment #246966 - Flags: review?(ajschult) → review+
(Assignee)

Updated

11 years ago
Attachment #246966 - Flags: superreview?(dveditz)
Comment on attachment 246966 [details] [diff] [review]
Proposed patch

sr=dveditz
Attachment #246966 - Flags: superreview?(dveditz) → superreview+
(Assignee)

Comment 4

11 years ago
Comment on attachment 246966 [details] [diff] [review]
Proposed patch

Dunno if this is branch-worthy.
Attachment #246966 - Flags: approval-seamonkey1.1?
(Assignee)

Comment 5

11 years ago
Fix checked in.
Status: ASSIGNED → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
Comment on attachment 246966 [details] [diff] [review]
Proposed patch

first-a=me given that this has been on trunk for a week

Comment 7

10 years ago
Comment on attachment 246966 [details] [diff] [review]
Proposed patch

a=me for 1.1
Attachment #246966 - Flags: approval-seamonkey1.1? → approval-seamonkey1.1+
(Assignee)

Comment 8

10 years ago
Fix checked in to the branch.
Keywords: fixed-seamonkey1.1

Updated

9 years ago
Component: Installer: XPI Packages → Installer
QA Contact: general
You need to log in before you can comment on or make changes to this bug.