--enable-activex-scripting horks non-win32 builds

RESOLVED FIXED in mozilla1.4alpha

Status

SeaMonkey
Build Config
RESOLVED FIXED
15 years ago
13 years ago

People

(Reporter: Benjamin Smedberg, Assigned: Benjamin Smedberg)

Tracking

Trunk
mozilla1.4alpha
x86
Linux

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

947 bytes, patch
cls
: review+
Details | Diff | Splinter Review
(Assignee)

Description

15 years ago
If --enable-activex-scripting is passed to configure on non-win32 platforms, it
should be ignored (this allows one .mozconfig file to be used for multiple
platforms).

Patch coming up...
(Assignee)

Comment 1

15 years ago
Created attachment 115794 [details] [diff] [review]
configure patch
(Assignee)

Updated

15 years ago
Attachment #115794 - Flags: review?(seawood)
Comment on attachment 115794 [details] [diff] [review]
configure patch

Nit: if you're going to bother reindenting that block (thereby screwing up cvs
blame), follow the conventions stated at the top of the file and indent 4
spaces.
Attachment #115794 - Flags: review?(seawood) → review-
(Assignee)

Comment 3

15 years ago
Created attachment 116762 [details] [diff] [review]
patch, updated indenting
Attachment #115794 - Attachment is obsolete: true
(Assignee)

Updated

15 years ago
Attachment #116762 - Flags: review?(seawood)

Comment 4

15 years ago
Comment on attachment 116762 [details] [diff] [review]
patch, updated indenting

r=cls
Attachment #116762 - Flags: review?(seawood) → review+
Patch has been checked in.
Status: NEW → RESOLVED
Last Resolved: 15 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.4alpha
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.