Make filter-build-webkit testable using LoadAsModule
authorddkilzer@apple.com <ddkilzer@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 17 May 2013 17:34:24 +0000 (17:34 +0000)
committerddkilzer@apple.com <ddkilzer@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 17 May 2013 17:34:24 +0000 (17:34 +0000)
Part 1 of 3: <http://webkit.org/b/116247>

Reviewed by Benjamin Poulain.

* Scripts/filter-build-webkit: Change global variables that are
used in subroutines from 'my' to 'our' to make it possible to
evaluate the script in a package context in LoadAsModule.pm.

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

Tools/ChangeLog
Tools/Scripts/filter-build-webkit

index 5fce68de9590ce64b8d5c556b56ae40aff3de03c..afadf99ad581ee64e0124c67272474394f94b902 100644 (file)
@@ -1,3 +1,14 @@
+2013-05-17  David Kilzer  <ddkilzer@apple.com>
+
+        Make filter-build-webkit testable using LoadAsModule
+        Part 1 of 3: <http://webkit.org/b/116247>
+
+        Reviewed by Benjamin Poulain.
+
+        * Scripts/filter-build-webkit: Change global variables that are
+        used in subroutines from 'my' to 'our' to make it possible to
+        evaluate the script in a package context in LoadAsModule.pm.
+
 2013-05-14  Sam Weinig  <sam@webkit.org>
 
         Add variants of the loading APIs that take user data and a way for the injected bundle to find out about them
index 97a732708635680f31d9d3d91636fa440cde9f72..270a49c50f85baf88f797e14ecb675db2146e9ed 100755 (executable)
@@ -1,6 +1,6 @@
 #!/usr/bin/perl -w
 
-# Copyright (C) 2011 Apple Inc. All rights reserved.
+# Copyright (C) 2011, 2012, 2013 Apple Inc. All rights reserved.
 # 
 # Redistribution and use in source and binary forms, with or without
 # modification, are permitted provided that the following conditions
@@ -71,12 +71,15 @@ sub usageAndExit();
 # Defined in VCSUtils.
 sub possiblyColored($$);
 
-my $showHelp;
+# Global variables used only in global scope.
 my $outputPath = "&STDOUT";
-my $outputFormat = "text";
-my $useColor = -t STDOUT;
-my $unfilteredOutputPath = "build.log";
-my $logUnfilteredOutput;
+my $showHelp;
+
+# Global variables used in global and subroutine scope.
+our $logUnfilteredOutput;
+our $outputFormat = "text";
+our $unfilteredOutputPath = "build.log";
+our $useColor = -t STDOUT;
 
 sub usageAndExit()
 {