remove atlbase.h check from configure

RESOLVED FIXED in mozilla9

Status

defect
RESOLVED FIXED
8 years ago
2 years ago

People

(Reporter: m_kato, Assigned: matjk7)

Tracking

({dev-doc-complete})

Trunk
mozilla9
x86
Windows Vista

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Since ActiveX support is dropped and accessibility support doesn't need ATL now, I think we can remove atlbase.h checking from configure.
Docs could be updated/flagged as legacy after configure is changed:
  http://developer.mozilla.org/en/atlbase.h

Moot item if the header check is removed -- windows7 may not be paying attention to the --disable- flag.

>> ac_add_options --disable-accessibility
Humm, we still only use ATL header into breakpad although we don't use atlbase.h

http://mxr.mozilla.org/mozilla-central/search?string=atlcomcli.h

IPC code that include atlcomcli.h isn't build target. (Makefile doesn't have win_safe_util.cc entry).

(In reply to comment #1)
> Docs could be updated/flagged as legacy after configure is changed:
>   http://developer.mozilla.org/en/atlbase.h
> 
> Moot item if the header check is removed -- windows7 may not be paying
> attention to the --disable- flag.
> 
> >> ac_add_options --disable-accessibility

This document is invalid now.
(In reply to comment #2)
> Humm, we still only use ATL header into breakpad although we don't use
> atlbase.h

That Breakpad code is part of dump_syms, which we don't build as part of the build (we have prebuilt binaries in the tree).
Assignee: nobody → matjk7
Status: NEW → ASSIGNED
Posted patch patchSplinter Review
Attachment #560741 - Flags: review?(ted.mielczarek)
Attachment #560741 - Flags: review?(ted.mielczarek) → review+
(In reply to Joey Armstrong [:joey] from comment #1)
> Docs could be updated/flagged as legacy after configure is changed:
>   http://developer.mozilla.org/en/atlbase.h

Seems fine to note that this doc is outdated, but we should leave it there because we linked to it from the configure error in older versions.
Flags: in-testsuite-
Keywords: checkin-needed
Keywords: dev-doc-needed
https://hg.mozilla.org/mozilla-central/rev/ec384fc33769
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Duplicate of this bug: 614504
Duplicate of this bug: 674870
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.