Changeset 183738 in webkit for trunk/Source/JavaScriptCore/generate-js-builtins

Timestamp:

May 4, 2015, 1:22:19 AM (10 years ago)

Author:

Csaba Osztrogonác

Message:

[cmake] Fix generate-js-builtins related incremental build issue
​https://bugs.webkit.org/show_bug.cgi?id=144094

Reviewed by Michael Saboff.

• CMakeLists.txt: Generated JSCBuiltins.<cpp|h> should depend on Source/JavaScriptCore/builtins directory.

Pass input directory to generate-js-builtins instead of Source/JavaScriptCore/builtins/*.js.

• DerivedSources.make:

Pass input directory to generate-js-builtins instead of Source/JavaScriptCore/builtins/*.js.

• generate-js-builtins: Accept input files and input directory too.

File:

• trunk/Source/JavaScriptCore/generate-js-builtins (modified) (3 diffs)

trunk/Source/JavaScriptCore/generate-js-builtins

@@ -23 +23 @@
 # OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 
+import argparse
 import filecmp
 import fnmatch
@@ -31 +32 @@
 import datetime
 import json
+
+parser = argparse.ArgumentParser()
+parser.add_argument('input_file', nargs='*', help='Input JS files which builtins generated from')
+parser.add_argument('--input-directory', help='All JS files will be used as input from this directory.')
+parser.add_argument('--output', help='path to output cpp or h file')
+args = parser.parse_args()
 
 copyrightText = """ *
@@ -152 +159 @@
 
 builtins = []
-
-baseName = sys.argv[-1]
-builtin_definitions = sys.argv[1:-1]
-(output_base, _) = os.path.splitext(sys.argv[-1])
-
 copyrights = []
-for file in builtin_definitions:
+(output_base, _) = os.path.splitext(args.output)
+
+if args.input_directory:
+    for file in os.listdir(args.input_directory):
+        args.input_file.append(os.path.join(args.input_directory, file))
+
+for file in args.input_file:
     if fnmatch.fnmatch(file, '*.js'):
         (baseName, functions, objectCopyrights) = generateCode(file)