Make generate_offset_extractor.rb architectures argument more robust
authorkeith_miller@apple.com <keith_miller@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 22 Aug 2017 03:19:46 +0000 (03:19 +0000)
committerkeith_miller@apple.com <keith_miller@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 22 Aug 2017 03:19:46 +0000 (03:19 +0000)
https://bugs.webkit.org/show_bug.cgi?id=175809

Reviewed by Joseph Pecoraro.

It turns out that some of our builders pass their architectures as
space separated lists.  I decided to just make the splitting of
our list robust to any reasonable combination of spaces and
commas.

* offlineasm/generate_offset_extractor.rb:

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

Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/offlineasm/generate_offset_extractor.rb

index 686cbedeafcb42b0ae245162743a345b8afe3a31..15e54636bd5ea514868198fb6411d742085c816d 100644 (file)
@@ -1,3 +1,17 @@
+2017-08-21  Keith Miller  <keith_miller@apple.com>
+
+        Make generate_offset_extractor.rb architectures argument more robust
+        https://bugs.webkit.org/show_bug.cgi?id=175809
+
+        Reviewed by Joseph Pecoraro.
+
+        It turns out that some of our builders pass their architectures as
+        space separated lists.  I decided to just make the splitting of
+        our list robust to any reasonable combination of spaces and
+        commas.
+
+        * offlineasm/generate_offset_extractor.rb:
+
 2017-08-21  Keith Miller  <keith_miller@apple.com>
 
         Only generate offline asm for the ARCHS (xcodebuild) or the current system (CMake)
 2017-08-21  Keith Miller  <keith_miller@apple.com>
 
         Only generate offline asm for the ARCHS (xcodebuild) or the current system (CMake)
index 1cc54c189d4567387f30377a9a710a788d679686..052008c0d6f87b4a913bdb09a9ae1829a5e6361c 100644 (file)
@@ -39,7 +39,7 @@ IncludeFile.processIncludeOptions()
 inputFlnm = ARGV.shift
 outputFlnm = ARGV.shift
 
 inputFlnm = ARGV.shift
 outputFlnm = ARGV.shift
 
-validBackends = canonicalizeBackendNames(ARGV.shift.split(","))
+validBackends = canonicalizeBackendNames(ARGV.shift.split(/[,\s]+/))
 $stderr.puts "Only dealing with backends: #{validBackends}"
 includeOnlyBackends(validBackends)
 
 $stderr.puts "Only dealing with backends: #{validBackends}"
 includeOnlyBackends(validBackends)