Detect unrecognized options in build-jsc

https://bugs.webkit.org/show_bug.cgi?id=218077

Patch by Angelos Oikonomopoulos <[email protected]> on 2020-10-22
Reviewed by Yusuke Suzuki.

Currently, Getopt::Long is configured with pass_through, in order to
be able to forward arbitrary arguments in buildMyProject. However, that
means that typos in option names (e.g. using --cmake-args instead of
--cmakeargs) go undetected and the option is silently ignored.

For cmake builds, there is no such forwarding, so check that there are
no remaining arguments in ARGV and refuse to continue if so. This runs
the risk of breaking wrapper scripts that incorrectly pass unrecognized
options, but that seems like a good thing.

* Scripts/build-jsc:

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

Author: [email protected]

Tools/ChangeLog

@@ -1,3 +1,22 @@
+2020-10-22  Angelos Oikonomopoulos  <[email protected]>
+
+        Detect unrecognized options in build-jsc
+        https://bugs.webkit.org/show_bug.cgi?id=218077
+
+        Reviewed by Yusuke Suzuki.
+
+        Currently, Getopt::Long is configured with pass_through, in order to
+        be able to forward arbitrary arguments in buildMyProject. However, that
+        means that typos in option names (e.g. using --cmake-args instead of
+        --cmakeargs) go undetected and the option is silently ignored.
+ 
+        For cmake builds, there is no such forwarding, so check that there are
+        no remaining arguments in ARGV and refuse to continue if so. This runs
+        the risk of breaking wrapper scripts that incorrectly pass unrecognized
+        options, but that seems like a good thing.
+
+        * Scripts/build-jsc:
+
 2020-10-22  Peng Liu  <[email protected]>
 
         Let webkitDisplayingFullscreen() return true when a video element’s fullscreen mode is not VideoFullscreenModeNone

Tools/Scripts/build-jsc

@@ -155,6 +155,17 @@ if ($forceCLoop) {
 }
 
 if (isCMakeBuild()) {
+    if (scalar(@ARGV) > 0) {
+        foreach (@ARGV) {
+            my $arg = $_;
+            if ($arg =~ /^-.*/) {
+                print STDERR "Unrecognized option `$arg'\n";
+            } else {
+                print STDERR "Stray anonymous argument `$arg'\n";
+            }
+        }
+        exit 2;
+    }
     if ($forceCLoop) {
         push @cmakeArgs, " -DENABLE_JIT=OFF";
         push @cmakeArgs, " -DENABLE_C_LOOP=ON";