[mac] Check For Framework Include Consistency build phase checks the wrong framework
authormitz@apple.com <mitz@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 14 Sep 2013 21:35:47 +0000 (21:35 +0000)
committermitz@apple.com <mitz@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 14 Sep 2013 21:35:47 +0000 (21:35 +0000)
https://bugs.webkit.org/show_bug.cgi?id=121215

Reviewed by Mark Rowe.

* Scripts/check-for-webkit-framework-include-consistency: Replaced hard-coded paths to
WebKit.framework’s Headers and PrivateHeaders directories with the current target’s
public and private headers folder paths. Also tweaked the error messages’ format to make
Xcode recognize them as errors.

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

Tools/ChangeLog
Tools/Scripts/check-for-webkit-framework-include-consistency

index ee37710..a7dfaed 100644 (file)
@@ -1,3 +1,15 @@
+2013-09-14  Dan Bernstein  <mitz@apple.com>
+
+        [mac] Check For Framework Include Consistency build phase checks the wrong framework
+        https://bugs.webkit.org/show_bug.cgi?id=121215
+
+        Reviewed by Mark Rowe.
+
+        * Scripts/check-for-webkit-framework-include-consistency: Replaced hard-coded paths to
+        WebKit.framework’s Headers and PrivateHeaders directories with the current target’s
+        public and private headers folder paths. Also tweaked the error messages’ format to make
+        Xcode recognize them as errors.
+
 2013-09-14  Alberto Garcia  <berto@igalia.com>
 
         [GTK] Fails to compile if the build directory is in a different partition
index fabc692..dc59c7e 100755 (executable)
@@ -1,6 +1,6 @@
 #!/usr/bin/env ruby
 
-# Copyright (C) 2010 Apple Inc. All rights reserved.
+# Copyright (C) 2010, 2013 Apple Inc. All rights reserved.
 #
 # Redistribution and use in source and binary forms, with or without
 # modification, are permitted provided that the following conditions
@@ -25,7 +25,6 @@
 
 
 base_directory = ENV['TARGET_BUILD_DIR'] or throw "Unable to find TARGET_BUILD_DIR in the environment!"
-is_shallow_bundle = (ENV['SHALLOW_BUNDLE'] || "NO").upcase == "YES"
 
 Dir.chdir base_directory
 
@@ -38,12 +37,11 @@ $error_printed = false
 
 def print_error msg
   $error_printed = true
-  STDERR.puts "ERROR: #{msg}"
+  STDERR.puts "error: #{msg}"
 end
 
-def build_header_maps is_shallow_bundle
-  current_version_path = is_shallow_bundle ? "" : "Versions/A/"
-  all_headers = `find WebKit.framework/#{current_version_path}{,Private}Headers -type f -name '*.h'`.split
+def build_header_maps
+  all_headers = `find #{ENV['PUBLIC_HEADERS_FOLDER_PATH']} #{ENV['PRIVATE_HEADERS_FOLDER_PATH']} -type f -name '*.h'`.split
 
   all_headers.each do |header|
     if /\/Headers\/(.*)/.match(header)
@@ -92,7 +90,7 @@ def verify_includes(header, permitted_types)
   end
 end
 
-build_header_maps is_shallow_bundle
+build_header_maps
 
 $HEADERS_BY_TYPE.each do |header_type, headers|
   permitted_types = $PERMITTED_INCLUDE_TYPES[header_type]