Closed
Bug 1542851
Opened 5 years ago
Closed 5 years ago
Define ENABLE_MARIONETTE along with config
Categories
(Remote Protocol :: Marionette, defect, P1)
Tracking
(Not tracked)
RESOLVED
INVALID
People
(Reporter: ato, Assigned: ato)
Details
Attachments
(1 obsolete file)
Near the bottom of browser/installer/Makefile.in, we have this:
ifdef ENABLE_MARIONETTE
DEFINES += -DENABLE_MARIONETTE=1
endif
The define should instead be installed in toolkit/moz.configure,
along with the config option.
Assignee | ||
Updated•5 years ago
|
Assignee: nobody → ato
Status: NEW → ASSIGNED
Priority: -- → P1
Assignee | ||
Comment 1•5 years ago
|
||
Updated•5 years ago
|
Attachment #9056664 -
Attachment is obsolete: true
Assignee | ||
Comment 2•5 years ago
|
||
Generally we should not set single package-time defines using
moz.configure. It is preferable to keep this in
browser/installer/Makefile.in:
DEFINES += -DENABLE_MARIONETTE=1
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → INVALID
Updated•1 year ago
|
Product: Testing → Remote Protocol
You need to log in
before you can comment on or make changes to this bug.
Description
•