Commit 78727835 authored by Valentine Barshak's avatar Valentine Barshak Committed by Peter Korsgaard
Browse files

webkit: Fix build issue with gcc-4.6 and above



This fixes the "Unknown parameter for tags/attrs" build error.
Backported from commit 88e08c43d0200a4b06a298b7d2541965eebc0afe

[PATCH] 2011-04-17  Thierry Reding
<thierry.reding@avionic-design.de>

  Reviewed by Adam Barth.

  Fix build with GCC 4.6.

  * dom/make_names.pl: Execute preprocessor without the -P option. The
  preprocessor in GCC 4.6 eats empty lines, effectively breaking the
  parsing performed by this script. Dropping the -P option when invoking
  the preprocessor keeps the empty lines but as a side-effect also adds
  additional linemarkers.

  From the cpp manpage:

    -P  Inhibit generation of linemarkers in the output from the
        preprocessor. This might be useful when running the preprocessor
        on something that is not C code, and will be sent to a program
        which might be confused by the linemarkers.

  The linemarkers are not problematic, however, because the script
  properly handles them by ignoring all lines starting with a #.

Signed-off-by: default avatarValentine Barshak <gvaxon@gmail.com>
Tested-by: default avatarSamuel Martin <s.martin49@gmail.com>
Acked-by: default avatarSamuel Martin <s.martin49@gmail.com>
Signed-off-by: default avatarPeter Korsgaard <jacmet@sunsite.dk>
parent 6d4cdb08
Loading
Loading
Loading
Loading
+45 −0
Original line number Diff line number Diff line
Backported from commit 88e08c43d0200a4b06a298b7d2541965eebc0afe:
Removed the ChangeLog part.

From: "commit-queue@webkit.org"
 <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date: Mon, 18 Apr 2011 06:53:23 +0000
Subject: [PATCH] 2011-04-17  Thierry Reding 
 <thierry.reding@avionic-design.de>

        Reviewed by Adam Barth.

        Fix build with GCC 4.6.

        * dom/make_names.pl: Execute preprocessor without the -P option. The
        preprocessor in GCC 4.6 eats empty lines, effectively breaking the
        parsing performed by this script. Dropping the -P option when invoking
        the preprocessor keeps the empty lines but as a side-effect also adds
        additional linemarkers.

        From the cpp manpage:

          -P  Inhibit generation of linemarkers in the output from the
              preprocessor. This might be useful when running the preprocessor
              on something that is not C code, and will be sent to a program
              which might be confused by the linemarkers.

        The linemarkers are not problematic, however, because the script
        properly handles them by ignoring all lines starting with a #.

diff --git a/WebCore/dom/make_names.pl b/WebCore/dom/make_names.pl
index 836137e..50386e0 100755
--- a/WebCore/dom/make_names.pl
+++ b/WebCore/dom/make_names.pl
@@ -54,7 +54,7 @@ if ($ENV{CC}) {
 } else {
     $gccLocation = "/usr/bin/gcc";
 }
-my $preprocessor = $gccLocation . " -E -P -x c++";
+my $preprocessor = $gccLocation . " -E -x c++";
 
 GetOptions(
     'tags=s' => \$tagsFile, 
-- 
1.7.11.4