Closed Bug 1032017 Opened 10 years ago Closed 10 years ago

Remove MOZ_SERVICES_FXACCOUNTS flags as it is no longer necessary - Perma-orange on TB trunk: TEST-UNEXPECTED-FAIL | (xpcshell/head.js) | Source file test_hawkrequest.js contains an error

Categories

(Firefox Build System :: General, defect)

defect
Not set
normal

Tracking

(firefox31 unaffected, firefox32 unaffected, firefox33 fixed, firefox-esr24 unaffected)

RESOLVED FIXED
mozilla33
Tracking Status
firefox31 --- unaffected
firefox32 --- unaffected
firefox33 --- fixed
firefox-esr24 --- unaffected

People

(Reporter: jcranmer, Assigned: standard8)

References

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

After a painful session with gdb, the file that Components.utils.import failed on is
reource://gre/modules/Credentials.jsm, defined in services/fxaccounts/Credentials.jsm and only included if MOZ_SERVICES_FXACCOUNTS is defined.

Not sure what the best fix is here.
Flags: needinfo?(standard8)
Jed, Chris, what would you say to moving Credentials.jsm from services/fxaccounts to services/common?

AFAICT Credentials.jsm isn't too fxaccounts specific (everyone currently seems to be using the same PROTOCOL_VERSION)
Flags: needinfo?(standard8)
Flags: needinfo?(jparsons)
Flags: needinfo?(ckarlof)
That build flag was added in Bug 976002 to prevent fxa code from shipping in FxOS 1.4.

But it seems like maybe we can remove the build flag now?

:ferjm, what do you think?
Flags: needinfo?(jparsons) → needinfo?(ferjmoreno)
Credentials.jsm is somewhat FxA specific (it helps derive FxA credentials). If removing that build flag solves the issue and is safe for B2G, I'd prefer that.
Flags: needinfo?(ckarlof)
I am probably not the best person to make the call about FxA being fully ready to remove this build flag.
Flags: needinfo?(spenrose)
Flags: needinfo?(gps)
Flags: needinfo?(ferjmoreno)
We should remove the flag. FxA has landed for FxOS 2.0.
Flags: needinfo?(spenrose)
Flags: needinfo?(gps)
I can do the removal.
Assignee: nobody → standard8
Component: Testing Infrastructure → Build Config
Product: Thunderbird → Core
Summary: Perma-orange on TB trunk: TEST-UNEXPECTED-FAIL | (xpcshell/head.js) | Source file /builds/slave/talos-slave/test/build/xpcshell/tests/services/common/tests/unit/test_hawkrequest.js contains an error → Remove MOZ_SERVICES_FXACCOUNTS flags as it is no longer necessary - Perma-orange on TB trunk: TEST-UNEXPECTED-FAIL | (xpcshell/head.js) | Source file test_hawkrequest.js contains an error
This removes the build flag as suggested.
Attachment #8450144 - Flags: review?(mh+mozilla)
Comment on attachment 8450144 [details] [diff] [review]
Removes MOZ_SERVICES_FXACCOUNTS flag

Review of attachment 8450144 [details] [diff] [review]:
-----------------------------------------------------------------

r+ as long as services.sync.fxaccounts.enabled can be used to disable the feature.
Attachment #8450144 - Flags: review?(mh+mozilla) → review+
(In reply to Mike Hommey [:glandium] from comment #8)
> r+ as long as services.sync.fxaccounts.enabled can be used to disable the
> feature.

Can anyone confirm this?
Flags: needinfo?(jed+bmo)
(In reply to Mark Banner (:standard8) from comment #9)
> (In reply to Mike Hommey [:glandium] from comment #8)
> > r+ as long as services.sync.fxaccounts.enabled can be used to disable the
> > feature.
> 
> Can anyone confirm this?

It's actually identity.fxaccounts.enabled that disables Firefox Accounts, but yes, the service can be disabled.

And as Sam confirms in Comment 5, we can remove this flag now.
Flags: needinfo?(jed+bmo)
https://hg.mozilla.org/integration/mozilla-inbound/rev/1c4f860f8697
OS: Linux → All
Hardware: x86_64 → All
Target Milestone: --- → mozilla33
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.