2011-03-02 Andras Becsi <abecsi@webkit.org>
authorabecsi@webkit.org <abecsi@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 2 Mar 2011 16:01:41 +0000 (16:01 +0000)
committerabecsi@webkit.org <abecsi@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 2 Mar 2011 16:01:41 +0000 (16:01 +0000)
        Reviewed by Laszlo Gombos.

        [Qt][WK2] generate-forwarding-headers.pl should generate paths with framework names
        https://bugs.webkit.org/show_bug.cgi?id=55478

        On case insensitive systems the generated forwarding headers cause build problems.

        * Source/WebKit.pri: Adjust INCLUDEPATH for the project.
2011-03-02  Andras Becsi  <abecsi@webkit.org>

        Reviewed by Laszlo Gombos.

        [Qt][WK2] generate-forwarding-headers.pl should generate paths with framework names
        https://bugs.webkit.org/show_bug.cgi?id=55478

        On case insensitive systems the generated forwarding headers cause build problems.

        * Scripts/generate-forwarding-headers.pl: The generated header should
        also contain the framework name.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@80131 268f45cc-cd09-0410-ab3c-d52691b4dbfc

ChangeLog
Source/WebKit.pri
Source/WebKit2/ChangeLog
Source/WebKit2/Scripts/generate-forwarding-headers.pl

index 97bab90..cf94a3f 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,14 @@
+2011-03-02  Andras Becsi  <abecsi@webkit.org>
+
+        Reviewed by Laszlo Gombos.
+
+        [Qt][WK2] generate-forwarding-headers.pl should generate paths with framework names
+        https://bugs.webkit.org/show_bug.cgi?id=55478
+
+        On case insensitive systems the generated forwarding headers cause build problems.
+
+        * Source/WebKit.pri: Adjust INCLUDEPATH for the project.
+
 2011-03-01  Sheriff Bot  <webkit.review.bot@gmail.com>
 
         Unreviewed, rolling out r80079.
index 51d13f0..f61b489 100644 (file)
@@ -68,15 +68,12 @@ CONFIG(release, debug|release) {
 }
 
 INCLUDEPATH += \
+    $$PWD \
     $$OUTPUT_DIR/include/QtWebKit \
-    $$OUTPUT_DIR/include
-INCLUDEPATH += $$QT.script.includes
+    $$OUTPUT_DIR/include \
+    $$QT.script.includes
 
-webkit2 {
-    INCLUDEPATH += $$OUTPUT_DIR/include/WebKit2
-    # FIXME: Once the public header are well defined for WebKit2, this must go away.
-    INCLUDEPATH += $$PWD/WebKit2/
-}
+webkit2:INCLUDEPATH *= $$OUTPUT_DIR/include/WebKit2
 
 # Pick up 3rdparty libraries from INCLUDE/LIB just like with MSVC
 win32-g++* {
index 22f7012..3975c54 100644 (file)
@@ -1,3 +1,15 @@
+2011-03-02  Andras Becsi  <abecsi@webkit.org>
+
+        Reviewed by Laszlo Gombos.
+
+        [Qt][WK2] generate-forwarding-headers.pl should generate paths with framework names
+        https://bugs.webkit.org/show_bug.cgi?id=55478
+
+        On case insensitive systems the generated forwarding headers cause build problems.
+
+        * Scripts/generate-forwarding-headers.pl: The generated header should
+        also contain the framework name.
+
 2011-03-01  Jeff Miller  <jeffm@apple.com>
 
         Reviewed by Darin Adler.
index 883f803..5a53d81 100755 (executable)
@@ -93,7 +93,7 @@ sub createForwardingHeadersForFramework {
         # If we found more headers with the same name, only generate a forwarding header for the current platform
         if(grep($_ =~ "/$headerName\$", @frameworkHeaders) == 1 || $header =~ "/$platform/" ) {
             my $forwardingHeaderPath = File::Spec->catfile($targetDirectory, $headerName);
-            my $expectedIncludeStatement = "#include \"$header\"";
+            my $expectedIncludeStatement = "#include \"$framework/$header\"";
             my $foundIncludeStatement = 0;
 
             $foundIncludeStatement = <EXISTING_HEADER> if open(EXISTING_HEADER, "<$forwardingHeaderPath");