Closed
Bug 817651
Opened 12 years ago
Closed 12 years ago
Define MOZ_UPDATER in ics_armv7a_gecko builds to get xpcshell update tests
Categories
(Firefox OS Graveyard :: GonkIntegration, defect, P1)
Firefox OS Graveyard
GonkIntegration
Tracking
(blocking-basecamp:+, firefox18 fixed, firefox19 fixed, firefox20 fixed)
RESOLVED
FIXED
blocking-basecamp | + |
People
(Reporter: ahal, Assigned: ahal)
References
Details
Attachments
(1 file)
1.09 KB,
patch
|
catlee
:
review+
|
Details | Diff | Splinter Review |
From bug 787578: > http://dxr.mozilla.org/mozilla-central/toolkit/mozapps/update/test/ > http://dxr.mozilla.org/mozilla-central/toolkit/mozapps/update/test_svc/ don't appear to be present in the tests.zip. They'll need to be added.
Comment 1•12 years ago
|
||
test_svc is windows only I think we need these: test test_timermanager
Assignee | ||
Comment 2•12 years ago
|
||
So it looks like this is happening because MOZ_UPDATER isn't being defined: http://mxr.mozilla.org/mozilla-central/source/toolkit/mozapps/update/Makefile.in#53 Maybe this gets defined in the builds releng provides and this bug isn't an issue. Or maybe we need to define this when building B2G. I'm not the right person to answer this. Brian, if you don't know the answer any idea who might?
Flags: needinfo?(netzen)
Comment 3•12 years ago
|
||
If you want the tests to run, yes it should be defined. If you'd like to make a build that knows nothing about updating then you should not define it.
Flags: needinfo?(netzen)
Assignee | ||
Comment 4•12 years ago
|
||
I downloaded a tests.zip from ftp.m.o and it looks like MOZ_UPDATER isn't defined there either. Morphing title and component.
Component: XPCShell Harness → Builds
Product: Testing → Boot2Gecko
Summary: Update tests not included in test-package-stage → Define MOZ_UPDATER in ics_armv7a_gecko builds to get xpcshell update tests
Updated•12 years ago
|
blocking-basecamp: --- → +
Assignee | ||
Comment 5•12 years ago
|
||
Marshall, do you know the build config to enable this?
blocking-basecamp: + → ---
Flags: needinfo?(marshall)
Comment 7•12 years ago
|
||
I spoke with Andrew about this today. The two requirements for this bug are 1. determine the required build params (see comment 5 for Marshall) 2. add the config to the build (likely on releng) Setting the basecamp flag back to + as Andrew renomed by accident.
blocking-basecamp: ? → +
Comment 8•12 years ago
|
||
Chris - Andrew suggested that someone from releng should set the build config once we know what it is. Can you help find someone to take this?
Assignee: nobody → catlee
Updated•12 years ago
|
Priority: -- → P1
Comment 9•12 years ago
|
||
Can this setting be provided in the mozconfig? https://hg.mozilla.org/mozilla-central/file/e83769592c61/b2g/config/mozconfigs/ics_armv7a_gecko/nightly
Assignee | ||
Comment 10•12 years ago
|
||
I found: https://github.com/mozilla-b2g/gonk-misc/blob/master/default-gecko-config#L44 Not sure if you also need to specify --enable-update-channel or --enable-update-packaging or if there are any negative side effects of doing --enable-updater. Jgriffin or marshall, care to weigh in?
Assignee | ||
Comment 11•12 years ago
|
||
Added --enable-updater only and pushed to try to see what happens: https://tbpl.mozilla.org/?tree=Try&rev=e75ad58d9201
Assignee | ||
Comment 12•12 years ago
|
||
I checked the try run and the update tests do get included in tests.zip with this patch
Attachment #690412 -
Flags: review?(catlee)
Assignee | ||
Updated•12 years ago
|
Assignee: catlee → ahalberstadt
Assignee | ||
Updated•12 years ago
|
Whiteboard: [automation-needed-in-aurora][automation-needed-in-beta]
Updated•12 years ago
|
Attachment #690412 -
Flags: review?(catlee) → review+
Assignee | ||
Comment 13•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/2db0bab6b7ff
Status: NEW → ASSIGNED
Comment 14•12 years ago
|
||
(In reply to Andrew Halberstadt [:ahal] from comment #5) > Marshall, do you know the build config to enable this? Yeah, we automatically enable updates for the "user" and "userdebug" build variants in AOSP, i.e: $ VARIANT=user ./build.sh or.. $ VARIANT=userdebug ./build.sh Alternatively, you can also use B2G_UPDATER=1: $ B2G_UPDATER=1 ./build.sh IIRC, these will also enable app updates
Flags: needinfo?(marshall)
Comment 15•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/2db0bab6b7ff
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 16•12 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/c90d71618b98 https://hg.mozilla.org/releases/mozilla-beta/rev/dcdea948bc16
status-firefox18:
--- → fixed
status-firefox19:
--- → fixed
status-firefox20:
--- → fixed
Whiteboard: [automation-needed-in-aurora][automation-needed-in-beta]
You need to log in
before you can comment on or make changes to this bug.
Description
•