Bug 52 – /Library/Frameworks not included in standard include paths

PySide Bugzilla Closed for New Bugs

PySide is now a Qt Add-on and uses the Qt Project's JIRA Bug Tracker instead of this Bugzilla instance. This Bugzilla is left for reference purposes.

Bug 52 - /Library/Frameworks not included in standard include paths

Summary:

/Library/Frameworks not included in standard include paths

Status:	CLOSED FIXED

Product:	PySide
Classification:	Unclassified
Component:	ApiExtractor
Version:	HEAD
Platform:	Macintosh Mac OS

Importance:	P2 major
Assigned To:	renato filho

URL:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2009-09-10 14:36 EEST by Brendan Duncan
Modified:	2009-10-14 10:58 EEST (History)
CC List:	5 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Brendan Duncan 2009-09-10 14:36:11 EEST

The APIExtractor preprocessor is already designed to search for frameworks on
OSX, but it's not including the /Library/Frameworks directory as a search path,
so it can't actually find the framework directories unless /Library/Frameworks
is added with --include-paths.  This should be a built-in search path.

apiextractor.cpp
In static bool preprocess(...)

After
preprocess.push_include_path("/usr/include");

Add
#ifdef Q_OS_MAC
preprocess.push_include_path("/Library/Frameworks");
#endif

This will fix the problems where the generator can't find the Qt headers on
OSX.

Comment 1 renato filho 2009-09-11 12:19:22 EEST

For fix this you can use in cmake command line: cmake ../
-DQT_INCLUDE_DIR=/Library/Frameworks

works for me, can you test this?

Comment 2 Brendan Duncan 2009-09-11 13:08:01 EEST

That works, though a bit awkward, in that one would expect system level include
directories to be added by default, and the errors you get if you don't specify
it are not indicative of the solution in any way...

Comment 3 renato filho 2009-09-11 15:05:06 EEST

We can set the default value in cmake for this QT_INCLUDE_DIR for each
plataform.

Comment 4 renato filho 2009-09-21 11:01:33 EEST

fixed on commit: 39df98619bd51dde04237ff6bd0a3fc9402dd396