Closed Bug 1542851 Opened 5 years ago Closed 5 years ago

Define ENABLE_MARIONETTE along with config

Categories

(Remote Protocol :: Marionette, defect, P1)

Version 3
defect

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: nobody → ato
Status: NEW → ASSIGNED
Priority: -- → P1
Attachment #9056664 - Attachment is obsolete: true

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
Product: Testing → Remote Protocol
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: