[CMake] build-webkit should print the cmake command executed
authorclopez@igalia.com <clopez@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 14 Feb 2017 16:46:49 +0000 (16:46 +0000)
committerclopez@igalia.com <clopez@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 14 Feb 2017 16:46:49 +0000 (16:46 +0000)
https://bugs.webkit.org/show_bug.cgi?id=168318

Reviewed by Michael Catanzaro.

Run the cmake commands to configure or start the build through
a wrapper that prints the command before executing it.

* Scripts/webkitdirs.pm:
(systemVerbose):
(generateBuildSystemFromCMakeProject):
(buildCMakeGeneratedProject):
(cleanCMakeGeneratedProject):

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

Tools/ChangeLog
Tools/Scripts/webkitdirs.pm

index ef3841d..1bcd97f 100644 (file)
@@ -1,3 +1,19 @@
+2017-02-14  Carlos Alberto Lopez Perez  <clopez@igalia.com>
+
+        [CMake] build-webkit should print the cmake command executed
+        https://bugs.webkit.org/show_bug.cgi?id=168318
+
+        Reviewed by Michael Catanzaro.
+
+        Run the cmake commands to configure or start the build through
+        a wrapper that prints the command before executing it.
+
+        * Scripts/webkitdirs.pm:
+        (systemVerbose):
+        (generateBuildSystemFromCMakeProject):
+        (buildCMakeGeneratedProject):
+        (cleanCMakeGeneratedProject):
+
 2017-02-14  Jonathan Bedard  <jbedard@apple.com>
 
         run-webkit-tests for iOS Simulator always complains that stopping WebKitTestRunnerApp.app times out
index 9516fd7..030cd9f 100755 (executable)
@@ -287,6 +287,11 @@ sub determineBaseProductDir
     }
 }
 
+sub systemVerbose {
+    print "+ @_\n";
+    return system(@_);
+}
+
 sub setBaseProductDir($)
 {
     ($baseProductDir) = @_;
@@ -1967,7 +1972,7 @@ sub generateBuildSystemFromCMakeProject
     # We call system("cmake @args") instead of system("cmake", @args) so that @args is
     # parsed for shell metacharacters.
     my $wrapper = join(" ", wrapperPrefixIfNeeded()) . " ";
-    my $returnCode = system($wrapper . "cmake @args");
+    my $returnCode = systemVerbose($wrapper . "cmake @args");
 
     chdir($originalWorkingDirectory);
     return $returnCode;
@@ -2001,7 +2006,7 @@ sub buildCMakeGeneratedProject($)
     # We call system("cmake @args") instead of system("cmake", @args) so that @args is
     # parsed for shell metacharacters. In particular, $makeArgs may contain such metacharacters.
     my $wrapper = join(" ", wrapperPrefixIfNeeded()) . " ";
-    return system($wrapper . "$command @args");
+    return systemVerbose($wrapper . "$command @args");
 }
 
 sub cleanCMakeGeneratedProject()
@@ -2009,7 +2014,7 @@ sub cleanCMakeGeneratedProject()
     my $config = configuration();
     my $buildPath = File::Spec->catdir(baseProductDir(), $config);
     if (-d $buildPath) {
-        return system("cmake", "--build", $buildPath, "--config", $config, "--target", "clean");
+        return systemVerbose("cmake", "--build", $buildPath, "--config", $config, "--target", "clean");
     }
     return 0;
 }