configure & configure.in changes required for Qt-Mozilla

VERIFIED FIXED

Status

VERIFIED FIXED
18 years ago
14 years ago

People

(Reporter: johng, Assigned: johng)

Tracking

Trunk
x86
Linux
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Assignee)

Description

18 years ago
The attached patches fix some problems with the Qt-Mozilla build:
- incorrect path for Qt include files
- Qt macro required to exclude Qt string processing stuff (qstrlen et al)

The patches were generated against the CVS tip revision as of Feb 28, 2001
(Assignee)

Comment 1

18 years ago
Created attachment 26446 [details] [diff] [review]
Patch for configure
(Assignee)

Comment 2

18 years ago
Created attachment 26447 [details] [diff] [review]
patch for configure.in
(Assignee)

Updated

18 years ago
Blocks: 70511
r=leaf, cc'ing cls for sr.

Comment 4

18 years ago
My only concern is that changing the include path may break builds made with
older versions of qt.  If you no longer support building with old versions of
qt, perhapps we should add a check.  Other than that, sr=cls.

Comment 5

18 years ago
*** Bug 61760 has been marked as a duplicate of this bug. ***
(Assignee)

Updated

18 years ago
Blocks: 73987

Comment 6

18 years ago
Patch checked in. Marking fixed.
Status: NEW → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED

Comment 7

18 years ago
verified.
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.