[WIN] Set MSBuild environment variable to avoid long-lived processes locking log...
authorbfulgham@apple.com <bfulgham@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 14 Jan 2014 22:33:56 +0000 (22:33 +0000)
committerbfulgham@apple.com <bfulgham@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 14 Jan 2014 22:33:56 +0000 (22:33 +0000)
https://bugs.webkit.org/show_bug.cgi?id=127002

Reviewed by David Kilzer.

* Scripts/webkitdirs.pm:
(setupAppleWinEnv): Export MSBUILDDISABLENODEREUSE=1 to the environment.

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

Tools/ChangeLog
Tools/Scripts/webkitdirs.pm

index 83c86ca..723ed7d 100644 (file)
@@ -1,3 +1,13 @@
+2014-01-14  Brent Fulgham  <bfulgham@apple.com>
+
+        [WIN] Set MSBuild environment variable to avoid long-lived processes locking log files
+        https://bugs.webkit.org/show_bug.cgi?id=127002
+
+        Reviewed by David Kilzer.
+
+        * Scripts/webkitdirs.pm:
+        (setupAppleWinEnv): Export MSBUILDDISABLENODEREUSE=1 to the environment.
+
 2014-01-14  Commit Queue  <commit-queue@webkit.org>
 
         Unreviewed, rolling out r162000.
index ece5661..4d4b64c 100755 (executable)
@@ -1436,6 +1436,7 @@ sub setupAppleWinEnv()
         # Those environment variables must be set to be able to build inside Visual Studio.
         $variablesToSet{WEBKIT_LIBRARIES} = windowsLibrariesDir() unless $ENV{WEBKIT_LIBRARIES};
         $variablesToSet{WEBKIT_OUTPUTDIR} = windowsOutputDir() unless $ENV{WEBKIT_OUTPUTDIR};
+        $variablesToSet{MSBUILDDISABLENODEREUSE} = "1" unless $ENV{MSBUILDDISABLENODEREUSE};
 
         foreach my $variable (keys %variablesToSet) {
             print "Setting the Environment Variable '" . $variable . "' to '" . $variablesToSet{$variable} . "'\n\n";
@@ -1447,16 +1448,20 @@ sub setupAppleWinEnv()
             print "Please restart your computer before attempting to build inside Visual Studio.\n\n";
         }
     } else {
-        if (!$ENV{'WEBKIT_LIBRARIES'}) {
+        if (!defined $ENV{'WEBKIT_LIBRARIES'} || !$ENV{'WEBKIT_LIBRARIES'}) {
             print "Warning: You must set the 'WebKit_Libraries' environment variable\n";
             print "         to be able build WebKit from within Visual Studio 2013 and newer.\n";
             print "         Make sure that 'WebKit_Libraries' points to the\n";
             print "         'WebKitLibraries/win' directory, not the 'WebKitLibraries/' directory.\n\n";
         }
-        if (!$ENV{'WEBKIT_OUTPUTDIR'}) {
+        if (!defined $ENV{'WEBKIT_OUTPUTDIR'} || !$ENV{'WEBKIT_OUTPUTDIR'}) {
             print "Warning: You must set the 'WebKit_OutputDir' environment variable\n";
             print "         to be able build WebKit from within Visual Studio 2013 and newer.\n\n";
         }
+        if (!defined $ENV{'MSBUILDDISABLENODEREUSE'} || !$ENV{'MSBUILDDISABLENODEREUSE'}) {
+            print "Warning: You should set the 'MSBUILDDISABLENODEREUSE' environment variable to '1'\n";
+            print "         to avoid periodic locked log files when building.\n\n";
+        }
     }
     # FIXME (125180): Remove the following temporary 64-bit support once official support is available.
     if (isWin64() and !$ENV{'WEBKIT_64_SUPPORT'}) {