Bug 20105 – The preprocessor is assumed not detected for IDL parsing

• Status: RESOLVED
• Resolution: INVALID
• Component: WebCore Misc.

[Connor Behan]

The --preprocessor $(CPP) variable should get passed to certain perl scripts on platforms where gcc is not in /usr/bin. This just fixes WebCore/GNUmakefile.am so it does that.

[Connor Behan]

Created attachment 22379 [details]
Fix

Choose whatever license you want. BSD and LGPL are both fine with me.

[Connor Behan]

Created attachment 22381 [details]
Better fix

Actually this one is better. Autotools don't append enough flags to $(CPP).

[Mark Rowe (bdash)]

That patch assumes that CPP is gcc and thus accepts "-P -x c++".  I don't think that is necessarily a valid assumption.

[Connor Behan]

Well without the patch, the preprocessor will always be "/usr/bin/gcc -E -P -x c++" which only works if gcc is in /usr/bin, so it can never work on windows for instance. Autotools is supposed to detect the preprocessor and store it in $CPP so it would be great if the patch could just use --preprocessor $(CPP). But I think autotools detecs the C preprocessor not the C++ one, so extra flags need to be added. Fixing the bug as to what preprocessor is detected would be the best solution but I have no idea how to do that.

[Mark Rowe (bdash)]

Copying Jan Michael Alonzo on this as he's done a bunch of work on the autotools support of the GTK port.  Jan, any ideas how we can have autoconf detect what we need here?

[Jan Alonzo]

You should use CXXCPP for the C++ preprocessor. The problem is I still have to pass "-P -x c++" to get it to compile properly. I'm not sure what to do here though. Either we detect gcc in the Makefiles and pass the required flags or do it in make_names.pl. What do you think?

[Martin Robinson]

The patch is long invalid. Please create a new bug with a patch against the CMake build if this is still an issue.