Closed Bug 1368084 Opened 5 years ago Closed 5 years ago

Temporarily disable geckodriver until it is made opt-in


(Testing :: Marionette, enhancement, P1)

Version 3


(firefox55 fixed)

Tracking Status
firefox55 --- fixed


(Reporter: ato, Assigned: ato)




(1 file, 1 obsolete file)

Windows developers are having issues compiling Firefox due as described in  Instead of backing out the patches landed in, which amount to several hundred thousand lines of code, we can disable geckodriver building temporarily by commenting out DIRS += ["testing/geckodriver"] in toolkit/toolkit.mozbuild until has been fixed to make it opt-in for local developers.
Assignee: nobody → ato
Severity: normal → blocker
Priority: -- → P1
Blocks: 1367995
I’m apparently not authorised to push this patch to m-c:

> % git push
> Bundling 1 changesets
> Bundling 1 changesets
> Bundling 1 manifests
> Bundling 1 manifests
> Bundling 0 files
> Bundling 1 files
> Traceback (most recent call last):
>   File "/usr/local/libexec/git-core/git-remote-hg", line 46, in <module>
>     run(main)
>   File "/usr/local/libexec/git-core/pythonlib/cinnabar/", line 517, in run
>     retcode = func(sys.argv[1:])
>   File "/usr/local/libexec/git-core/git-remote-hg", line 40, in main
>   File "/usr/local/libexec/git-core/pythonlib/cinnabar/", line 88, in run
>     func(*args)
>   File "/usr/local/libexec/git-core/pythonlib/cinnabar/", line 393, in push
>     self._branchmap.names(), self._dry_run)
>   File "/usr/local/libexec/git-core/pythonlib/cinnabar/hg/", line 711, in push
>     reply = repo.unbundle(cg, repo_heads, '')
>   File "/usr/lib/python2.7/dist-packages/mercurial/", line 457, in unbundle
>     stream = self._calltwowaystream('unbundle', cg, heads=heads)
>   File "/usr/lib/python2.7/dist-packages/mercurial/", line 265, in _calltwowaystream
>     return self._callstream(cmd, data=fp_, headers=headers, **args)
>   File "/usr/lib/python2.7/dist-packages/mercurial/", line 157, in _callstream
>     raise error.Abort(_('authorization failed'))
> mercurial.error.Abort: authorization failed
> error: failed to push some refs to 'hg::'

I’ve attached the patch to this bug.
Flags: needinfo?(ryanvm)
Pushed by
Temporarily disable building testing/geckodriver; r=me a=RyanVM
Oops, I was pushing to https: instead of ssh:.  Landed this in m-c:
Closed: 5 years ago
Resolution: --- → FIXED
Glad you figured it out. Thanks for getting this done.
Flags: needinfo?(ryanvm)
Target Milestone: --- → mozilla55
Well I am really not sure that is the real issue here.  cl.exe exists on the system and is in the command search path.  I think the code in bug 1340637 is just wrong!
Hint try the Which command as in "which cl" to find the correct path to the command.
You need to log in before you can comment on or make changes to this bug.