Closed
Bug 732962
Opened 13 years ago
Closed 13 years ago
Implement signing on demand for xulrunner
Categories
(Firefox Build System :: General, defect, P2)
Firefox Build System
General
Tracking
(firefox-esr10 fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox-esr10 | --- | fixed |
People
(Reporter: rail, Assigned: rail)
References
Details
(Whiteboard: [Leave open after merge])
Attachments
(1 file)
1.78 KB,
patch
|
catlee
:
review+
ted
:
review+
lsblakk
:
approval-mozilla-beta+
lsblakk
:
approval-mozilla-release+
|
Details | Diff | Splinter Review |
* xulrunner shouldn't be codesigned internally (our current practice)
* GPG sign SDK
Attachment #602895 -
Flags: review?(ted.mielczarek)
Attachment #602895 -
Flags: review?(catlee)
Updated•13 years ago
|
Attachment #602895 -
Flags: review?(catlee) → review+
Updated•13 years ago
|
Attachment #602895 -
Flags: review?(ted.mielczarek) → review+
Assignee | ||
Comment 1•13 years ago
|
||
Comment on attachment 602895 [details] [diff] [review]
sign xulrunner
http://hg.mozilla.org/integration/mozilla-inbound/rev/5b2ba557ec83
Attachment #602895 -
Flags: checkin+
Assignee | ||
Updated•13 years ago
|
Whiteboard: [Leave open after merge]
Comment 2•13 years ago
|
||
Assignee | ||
Comment 3•13 years ago
|
||
Comment on attachment 602895 [details] [diff] [review]
sign xulrunner
This patch implements signing on demand for xulrunner. Xulrunner isn't a part of Firefox CI process (only nightlies) and exists in release builds. The patch simplifies the release build procedure by removing xulrunner signing step which is manual right now.
Regression caused by (bug #): none
Testing completed (on m-c, etc.): tested in dev environment by running staging releases
Risk to taking this patch (and alternatives if risky): very low
String changes made by this patch: none
Attachment #602895 -
Flags: approval-mozilla-beta?
Comment 4•13 years ago
|
||
Comment on attachment 602895 [details] [diff] [review]
sign xulrunner
[Triage comment]
Approval for low-risk, non-CI affecting.
Attachment #602895 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Assignee | ||
Comment 5•13 years ago
|
||
Comment on attachment 602895 [details] [diff] [review]
sign xulrunner
http://hg.mozilla.org/releases/mozilla-beta/rev/438dd47e6f69
Assignee | ||
Comment 6•13 years ago
|
||
Comment on attachment 602895 [details] [diff] [review]
sign xulrunner
I would like to land this patch on m-r and esr10 branches. 12.0b1 release passed without any problems. The patch will remove error-prone manual steps used by releng.
Attachment #602895 -
Flags: approval-mozilla-release?
Attachment #602895 -
Flags: approval-mozilla-esr10?
Comment 7•13 years ago
|
||
Comment on attachment 602895 [details] [diff] [review]
sign xulrunner
[Triage Comment]
Looks foo, low risk, please go ahead.
Attachment #602895 -
Flags: approval-mozilla-release?
Attachment #602895 -
Flags: approval-mozilla-release+
Attachment #602895 -
Flags: approval-mozilla-esr10?
Attachment #602895 -
Flags: approval-mozilla-esr10+
Assignee | ||
Comment 8•13 years ago
|
||
Assignee | ||
Updated•13 years ago
|
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
status-firefox-esr10:
--- → fixed
Updated•7 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•