Closed
Bug 480079
Opened 16 years ago
Closed 16 years ago
Move comm-central 7zSD.sfxes out of mozilla-central
Categories
(MailNews Core :: Build Config, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: philor, Assigned: robert.strong.bugs)
Details
Attachments
(2 files)
It looks like the CVS import for mozilla-central just swept up all of other-licenses/7zstub/, including seamonkey/, sunbird/, and thunderbird/. Having the 7zSD.sfx in a different repo than the app.tag makes nthomas ranty while doing hacks like attachment 364027 [details] [diff] [review] to make signing work, and we always want to save our "make him ranty" for times when we really need it, so if it's as easy as I naively think, just changing what we set SFX_MODULE to, we should.
Assignee | ||
Comment 1•16 years ago
|
||
Here is the comm-central patch... I verified this works with both Thunderbird and SeaMonkey
Assignee: nobody → robert.bugzilla
Attachment #364031 -
Flags: review?(philringnalda)
Assignee | ||
Updated•16 years ago
|
Attachment #364031 -
Flags: review?(bugzilla)
Assignee | ||
Updated•16 years ago
|
Attachment #364031 -
Flags: review?(ctalbert)
Assignee | ||
Comment 2•16 years ago
|
||
Attachment #364032 -
Flags: review?(philringnalda)
Assignee | ||
Updated•16 years ago
|
Attachment #364032 -
Flags: review?(bugzilla)
Assignee | ||
Updated•16 years ago
|
Attachment #364032 -
Flags: review?(ctalbert)
Reporter | ||
Comment 3•16 years ago
|
||
Comment on attachment 364031 [details] [diff] [review]
patch rev1
Sweet.
Attachment #364031 -
Flags: review?(philringnalda) → review+
Reporter | ||
Updated•16 years ago
|
Attachment #364032 -
Flags: review?(philringnalda) → review+
Assignee | ||
Comment 4•16 years ago
|
||
I also verified this works with Sunbird
Assignee | ||
Updated•16 years ago
|
Attachment #364032 -
Flags: review?(mschroeder)
Assignee | ||
Comment 5•16 years ago
|
||
Comment on attachment 364032 [details] [diff] [review]
mozilla-central patch rev1
Martin, could you review these two patches for Sunbird? Thanks
Assignee | ||
Updated•16 years ago
|
Attachment #364031 -
Flags: review?(mschroeder)
Updated•16 years ago
|
Attachment #364032 -
Flags: review?(bugzilla) → review+
Updated•16 years ago
|
Attachment #364031 -
Flags: review?(bugzilla) → review+
Updated•16 years ago
|
Attachment #364031 -
Flags: review?(mschroeder)
Attachment #364031 -
Flags: review?(ctalbert)
Attachment #364031 -
Flags: review+
Comment 6•16 years ago
|
||
Comment on attachment 364031 [details] [diff] [review]
patch rev1
r=mschroeder
Updated•16 years ago
|
Attachment #364032 -
Flags: review?(mschroeder)
Attachment #364032 -
Flags: review?(ctalbert)
Attachment #364032 -
Flags: review+
Comment 7•16 years ago
|
||
Comment on attachment 364032 [details] [diff] [review]
mozilla-central patch rev1
r=mschroeder
Assignee | ||
Comment 8•16 years ago
|
||
Pushed to comm-central
http://hg.mozilla.org/comm-central/rev/c6229432c088
Pushed to mozilla-central
http://hg.mozilla.org/mozilla-central/rev/58b0b3e2533b
Pushed to mozilla-1.9.1
http://hg.mozilla.org/releases/mozilla-1.9.1/rev/e3687f86afb4
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•