Move --enable-ipc-fuzzer to python configure

RESOLVED FIXED in Firefox 51

Status

defect
RESOLVED FIXED
3 years ago
Last year

People

(Reporter: glandium, Assigned: glandium)

Tracking

(Depends on 1 bug, Blocks 1 bug)

unspecified
mozilla51
Dependency tree / graph

Firefox Tracking Flags

(firefox51 fixed)

Details

Attachments

(1 attachment)

Assignee

Description

3 years ago
No description provided.
Comment hidden (mozreview-request)

Comment 2

3 years ago
mozreview-review
Comment on attachment 8780396 [details]
Bug 1294630 - Move --enable-ipc-fuzzer to python configure.

https://reviewboard.mozilla.org/r/71128/#review68882

::: toolkit/moz.configure:484
(Diff revision 1)
> +# Gecko integrated IPC fuzzer
> +# ==============================================================
> +option('--enable-ipc-fuzzer', env='MOZ_FAULTY',
> +       help='Enable IPC fuzzer')
> +
> +@depends_if('--enable-ipc-fuzzer', target)

`depends_if` proceeds if any argument is truthy, which `target` will be, so this fails outright on Windows.
Attachment #8780396 - Flags: review?(cmanchester)
Comment hidden (mozreview-request)

Comment 4

3 years ago
mozreview-review
Comment on attachment 8780396 [details]
Bug 1294630 - Move --enable-ipc-fuzzer to python configure.

https://reviewboard.mozilla.org/r/71128/#review69614
Attachment #8780396 - Flags: review?(cmanchester) → review+

Comment 5

3 years ago
Pushed by mh@glandium.org:
https://hg.mozilla.org/integration/mozilla-inbound/rev/2bbe53526d74
Move --enable-ipc-fuzzer to python configure. r=chmanchester

Comment 6

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/2bbe53526d74
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla51

Updated

Last year
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.