<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"
"http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head><meta http-equiv="content-type" content="text/html; charset=utf-8" />
<title>[55972] trunk/WebKitTools</title>
</head>
<body>

<style type="text/css"><!--
#msg dl.meta { border: 1px #006 solid; background: #369; padding: 6px; color: #fff; }
#msg dl.meta dt { float: left; width: 6em; font-weight: bold; }
#msg dt:after { content:':';}
#msg dl, #msg dt, #msg ul, #msg li, #header, #footer, #logmsg { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt;  }
#msg dl a { font-weight: bold}
#msg dl a:link    { color:#fc3; }
#msg dl a:active  { color:#ff0; }
#msg dl a:visited { color:#cc6; }
h3 { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; font-weight: bold; }
#msg pre { overflow: auto; background: #ffc; border: 1px #fa0 solid; padding: 6px; }
#logmsg { background: #ffc; border: 1px #fa0 solid; padding: 1em 1em 0 1em; }
#logmsg p, #logmsg pre, #logmsg blockquote { margin: 0 0 1em 0; }
#logmsg p, #logmsg li, #logmsg dt, #logmsg dd { line-height: 14pt; }
#logmsg h1, #logmsg h2, #logmsg h3, #logmsg h4, #logmsg h5, #logmsg h6 { margin: .5em 0; }
#logmsg h1:first-child, #logmsg h2:first-child, #logmsg h3:first-child, #logmsg h4:first-child, #logmsg h5:first-child, #logmsg h6:first-child { margin-top: 0; }
#logmsg ul, #logmsg ol { padding: 0; list-style-position: inside; margin: 0 0 0 1em; }
#logmsg ul { text-indent: -1em; padding-left: 1em; }#logmsg ol { text-indent: -1.5em; padding-left: 1.5em; }
#logmsg > ul, #logmsg > ol { margin: 0 0 1em 0; }
#logmsg pre { background: #eee; padding: 1em; }
#logmsg blockquote { border: 1px solid #fa0; border-left-width: 10px; padding: 1em 1em 0 1em; background: white;}
#logmsg dl { margin: 0; }
#logmsg dt { font-weight: bold; }
#logmsg dd { margin: 0; padding: 0 0 0.5em 0; }
#logmsg dd:before { content:'\00bb';}
#logmsg table { border-spacing: 0px; border-collapse: collapse; border-top: 4px solid #fa0; border-bottom: 1px solid #fa0; background: #fff; }
#logmsg table th { text-align: left; font-weight: normal; padding: 0.2em 0.5em; border-top: 1px dotted #fa0; }
#logmsg table td { text-align: right; border-top: 1px dotted #fa0; padding: 0.2em 0.5em; }
#logmsg table thead th { text-align: center; border-bottom: 1px solid #fa0; }
#logmsg table th.Corner { text-align: left; }
#logmsg hr { border: none 0; border-top: 2px dashed #fa0; height: 1px; }
#header, #footer { color: #fff; background: #636; border: 1px #300 solid; padding: 6px; }
#patch { width: 100%; }
#patch h4 {font-family: verdana,arial,helvetica,sans-serif;font-size:10pt;padding:8px;background:#369;color:#fff;margin:0;}
#patch .propset h4, #patch .binary h4 {margin:0;}
#patch pre {padding:0;line-height:1.2em;margin:0;}
#patch .diff {width:100%;background:#eee;padding: 0 0 10px 0;overflow:auto;}
#patch .propset .diff, #patch .binary .diff  {padding:10px 0;}
#patch span {display:block;padding:0 10px;}
#patch .modfile, #patch .addfile, #patch .delfile, #patch .propset, #patch .binary, #patch .copfile {border:1px solid #ccc;margin:10px 0;}
#patch ins {background:#dfd;text-decoration:none;display:block;padding:0 10px;}
#patch del {background:#fdd;text-decoration:none;display:block;padding:0 10px;}
#patch .lines, .info {color:#888;background:#fff;}
--></style>
<div id="msg">
<dl class="meta">
<dt>Revision</dt> <dd><a href="http://trac.webkit.org/projects/webkit/changeset/55972">55972</a></dd>
<dt>Author</dt> <dd>cjerdonek@webkit.org</dd>
<dt>Date</dt> <dd>2010-03-13 16:23:55 -0800 (Sat, 13 Mar 2010)</dd>
</dl>

<h3>Log Message</h3>
<pre>The webkit-patch script now displays a warning if run using
a version of Python less than 2.5.  This will help users
understand why webkit-patch is erroring out if they are
using Python 2.4, for example.

Reviewed by Adam Barth.

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

* Scripts/webkit-patch:
  - Moved most of the file contents to webkitpy/patch/patcher.py
    so the Python version can be checked before interpreting
    any code that can cause the script to error out.
  - Added a configure_logging() method to enable any version
    warnings to show up.
  - Added a main() method with calls to configure_logging(),
    check_version(), and the main webkit patch method.

* Scripts/webkitpy/patch/__init__.py: Copied from WebKitTools/QueueStatusServer/filters/__init__.py.
  - This file is required to make a folder a package.

* Scripts/webkitpy/patch/patcher.py: Added.
  - Moved code from Scripts/webkit-patch.</pre>

<h3>Modified Paths</h3>
<ul>
<li><a href="#trunkWebKitToolsChangeLog">trunk/WebKitTools/ChangeLog</a></li>
<li><a href="#trunkWebKitToolsScriptswebkitpatch">trunk/WebKitTools/Scripts/webkit-patch</a></li>
</ul>

<h3>Added Paths</h3>
<ul>
<li>trunk/WebKitTools/Scripts/webkitpy/patch/</li>
<li><a href="#trunkWebKitToolsScriptswebkitpypatch__init__py">trunk/WebKitTools/Scripts/webkitpy/patch/__init__.py</a></li>
<li><a href="#trunkWebKitToolsScriptswebkitpypatchpatcherpy">trunk/WebKitTools/Scripts/webkitpy/patch/patcher.py</a></li>
</ul>

</div>
<div id="patch">
<h3>Diff</h3>
<a id="trunkWebKitToolsChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/WebKitTools/ChangeLog (55971 => 55972)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/WebKitTools/ChangeLog        2010-03-13 23:48:20 UTC (rev 55971)
+++ trunk/WebKitTools/ChangeLog        2010-03-14 00:23:55 UTC (rev 55972)
</span><span class="lines">@@ -2,6 +2,32 @@
</span><span class="cx"> 
</span><span class="cx">         Reviewed by Adam Barth.
</span><span class="cx"> 
</span><ins>+        The webkit-patch script now displays a warning if run using
+        a version of Python less than 2.5.  This will help users
+        understand why webkit-patch is erroring out if they are
+        using Python 2.4, for example.
+
+        https://bugs.webkit.org/show_bug.cgi?id=31533
+
+        * Scripts/webkit-patch:
+          - Moved most of the file contents to webkitpy/patch/patcher.py
+            so the Python version can be checked before interpreting
+            any code that can cause the script to error out.
+          - Added a configure_logging() method to enable any version
+            warnings to show up.
+          - Added a main() method with calls to configure_logging(),
+            check_version(), and the main webkit patch method.
+
+        * Scripts/webkitpy/patch/__init__.py: Copied from WebKitTools/QueueStatusServer/filters/__init__.py.
+          - This file is required to make a folder a package.
+
+        * Scripts/webkitpy/patch/patcher.py: Added.
+          - Moved code from Scripts/webkit-patch.
+
+2010-03-13  Chris Jerdonek  &lt;cjerdonek@webkit.org&gt;
+
+        Reviewed by Adam Barth.
+
</ins><span class="cx">         Changed test-webkitpy so that messages logged as a side-effect
</span><span class="cx">         of unit-testing code do not get displayed to the screen.  These
</span><span class="cx">         messages clutter up the unit test results if not filtered out.
</span></span></pre></div>
<a id="trunkWebKitToolsScriptswebkitpatch"></a>
<div class="modfile"><h4>Modified: trunk/WebKitTools/Scripts/webkit-patch (55971 => 55972)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/WebKitTools/Scripts/webkit-patch        2010-03-13 23:48:20 UTC (rev 55971)
+++ trunk/WebKitTools/Scripts/webkit-patch        2010-03-14 00:23:55 UTC (rev 55972)
</span><span class="lines">@@ -1,6 +1,7 @@
</span><span class="cx"> #!/usr/bin/env python
</span><span class="cx"> # Copyright (c) 2009, Google Inc. All rights reserved.
</span><span class="cx"> # Copyright (c) 2009 Apple Inc. All rights reserved.
</span><ins>+# Copyright (C) 2010 Chris Jerdonek (cjerdonek@webkit.org)
</ins><span class="cx"> #
</span><span class="cx"> # Redistribution and use in source and binary forms, with or without
</span><span class="cx"> # modification, are permitted provided that the following conditions are
</span><span class="lines">@@ -30,80 +31,40 @@
</span><span class="cx"> #
</span><span class="cx"> # A tool for automating dealing with bugzilla, posting patches, committing patches, etc.
</span><span class="cx"> 
</span><ins>+import logging
</ins><span class="cx"> import os
</span><ins>+import sys
</ins><span class="cx"> 
</span><del>-from webkitpy.bugzilla import Bugzilla
-from webkitpy.buildbot import BuildBot
-from webkitpy.commands.download import *
-from webkitpy.commands.early_warning_system import *
-from webkitpy.commands.openbugs import OpenBugs
-from webkitpy.commands.queries import *
-from webkitpy.commands.queues import *
-from webkitpy.commands.upload import *
-from webkitpy.executive import Executive
-from webkitpy.webkit_logging import log
-from webkitpy.multicommandtool import MultiCommandTool
-from webkitpy.scm import detect_scm_system
-from webkitpy.user import User
</del><ins>+import webkitpy.init.versioning as versioning
</ins><span class="cx"> 
</span><span class="cx"> 
</span><del>-class WebKitPatch(MultiCommandTool):
-    global_options = [
-        make_option(&quot;--dry-run&quot;, action=&quot;store_true&quot;, dest=&quot;dry_run&quot;, default=False, help=&quot;do not touch remote servers&quot;),
-        make_option(&quot;--status-host&quot;, action=&quot;store&quot;, dest=&quot;status_host&quot;, type=&quot;string&quot;, nargs=1, help=&quot;Hostname (e.g. localhost or commit.webkit.org) where status updates should be posted.&quot;),
-    ]
</del><ins>+# FIXME: Consider logging more than just messages from the
+#        version-checking module.  For now we enable only enough
+#        for the version warning to show up.
+def configure_logging():
+    &quot;&quot;&quot;Configure logging for the version-checker.&quot;&quot;&quot;
+    handler = logging.StreamHandler(sys.stderr)
+    formatter = logging.Formatter(&quot;%(levelname)s: %(message)s&quot;)
+    handler.setFormatter(formatter)
</ins><span class="cx"> 
</span><del>-    def __init__(self):
-        MultiCommandTool.__init__(self)
</del><ins>+    logger = logging.getLogger(versioning.__name__)
+    logger.setLevel(logging.INFO)
+    logger.addHandler(handler)
</ins><span class="cx"> 
</span><del>-        self.bugs = Bugzilla()
-        self.buildbot = BuildBot()
-        self.executive = Executive()
-        self.user = User()
-        self._scm = None
-        self.status_server = StatusServer()
</del><span class="cx"> 
</span><del>-    def scm(self):
-        # Lazily initialize SCM to not error-out before command line parsing (or when running non-scm commands).
-        original_cwd = os.path.abspath(&quot;.&quot;)
-        if not self._scm:
-            self._scm = detect_scm_system(original_cwd)
</del><ins>+def main():
+    configure_logging()
</ins><span class="cx"> 
</span><del>-        if not self._scm:
-            script_directory = os.path.abspath(sys.path[0])
-            webkit_directory = os.path.abspath(os.path.join(script_directory, &quot;../..&quot;))
-            self._scm = detect_scm_system(webkit_directory)
-            if self._scm:
-                log(&quot;The current directory (%s) is not a WebKit checkout, using %s&quot; % (original_cwd, webkit_directory))
-            else:
-                error(&quot;FATAL: Failed to determine the SCM system for either %s or %s&quot; % (original_cwd, webkit_directory))
</del><ins>+    versioning.check_version()
</ins><span class="cx"> 
</span><del>-        return self._scm
</del><ins>+    # Import webkit-patch code only after version-checking so that
+    # script doesn't error out before having a chance to report the
+    # version warning.
+    from webkitpy.patch.patcher import WebKitPatch
</ins><span class="cx"> 
</span><del>-    def path(self):
-        return __file__
</del><ins>+    WebKitPatch().main()
</ins><span class="cx"> 
</span><del>-    def should_show_in_main_help(self, command):
-        if not command.show_in_main_help:
-            return False
-        if command.requires_local_commits:
-            return self.scm().supports_local_commits()
-        return True
</del><span class="cx"> 
</span><del>-    # FIXME: This may be unnecessary since we pass global options to all commands during execute() as well.
-    def handle_global_options(self, options):
-        if options.dry_run:
-            self.scm().dryrun = True
-            self.bugs.dryrun = True
-        if options.status_host:
-            self.status_server.set_host(options.status_host)
-
-    def should_execute_command(self, command):
-        if command.requires_local_commits and not self.scm().supports_local_commits():
-            failure_reason = &quot;%s requires local commits using %s in %s.&quot; % (command.name, self.scm().display_name(), self.scm().checkout_root)
-            return (False, failure_reason)
-        return (True, None)
-
-
</del><span class="cx"> if __name__ == &quot;__main__&quot;:
</span><del>-    WebKitPatch().main()
</del><ins>+
+    main()
</ins></span></pre></div>
<a id="trunkWebKitToolsScriptswebkitpypatch__init__py"></a>
<div class="addfile"><h4>Added: trunk/WebKitTools/Scripts/webkitpy/patch/__init__.py (0 => 55972)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/WebKitTools/Scripts/webkitpy/patch/__init__.py                                (rev 0)
+++ trunk/WebKitTools/Scripts/webkitpy/patch/__init__.py        2010-03-14 00:23:55 UTC (rev 55972)
</span><span class="lines">@@ -0,0 +1 @@
</span><ins>+# Required for Python to search this directory for module files
</ins></span></pre></div>
<a id="trunkWebKitToolsScriptswebkitpypatchpatcherpyfromrev55971trunkWebKitToolsScriptswebkitpatch"></a>
<div class="copfile"><h4>Copied: trunk/WebKitTools/Scripts/webkitpy/patch/patcher.py (from rev 55971, trunk/WebKitTools/Scripts/webkit-patch) (0 => 55972)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/WebKitTools/Scripts/webkitpy/patch/patcher.py                                (rev 0)
+++ trunk/WebKitTools/Scripts/webkitpy/patch/patcher.py        2010-03-14 00:23:55 UTC (rev 55972)
</span><span class="lines">@@ -0,0 +1,105 @@
</span><ins>+#!/usr/bin/env python
+# Copyright (c) 2009, Google Inc. All rights reserved.
+# Copyright (c) 2009 Apple Inc. All rights reserved.
+#
+# Redistribution and use in source and binary forms, with or without
+# modification, are permitted provided that the following conditions are
+# met:
+#
+#     * Redistributions of source code must retain the above copyright
+# notice, this list of conditions and the following disclaimer.
+#     * Redistributions in binary form must reproduce the above
+# copyright notice, this list of conditions and the following disclaimer
+# in the documentation and/or other materials provided with the
+# distribution.
+#     * Neither the name of Google Inc. nor the names of its
+# contributors may be used to endorse or promote products derived from
+# this software without specific prior written permission.
+#
+# THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+# &quot;AS IS&quot; AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+# LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+# A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+# OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+# SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+# LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+# DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+# THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+# (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+# OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+#
+# A tool for automating dealing with bugzilla, posting patches, committing patches, etc.
+
+import os
+
+from webkitpy.bugzilla import Bugzilla
+from webkitpy.buildbot import BuildBot
+from webkitpy.commands.download import *
+from webkitpy.commands.early_warning_system import *
+from webkitpy.commands.openbugs import OpenBugs
+from webkitpy.commands.queries import *
+from webkitpy.commands.queues import *
+from webkitpy.commands.upload import *
+from webkitpy.executive import Executive
+from webkitpy.webkit_logging import log
+from webkitpy.multicommandtool import MultiCommandTool
+from webkitpy.scm import detect_scm_system
+from webkitpy.user import User
+
+
+class WebKitPatch(MultiCommandTool):
+    global_options = [
+        make_option(&quot;--dry-run&quot;, action=&quot;store_true&quot;, dest=&quot;dry_run&quot;, default=False, help=&quot;do not touch remote servers&quot;),
+        make_option(&quot;--status-host&quot;, action=&quot;store&quot;, dest=&quot;status_host&quot;, type=&quot;string&quot;, nargs=1, help=&quot;Hostname (e.g. localhost or commit.webkit.org) where status updates should be posted.&quot;),
+    ]
+
+    def __init__(self):
+        MultiCommandTool.__init__(self)
+
+        self.bugs = Bugzilla()
+        self.buildbot = BuildBot()
+        self.executive = Executive()
+        self.user = User()
+        self._scm = None
+        self.status_server = StatusServer()
+
+    def scm(self):
+        # Lazily initialize SCM to not error-out before command line parsing (or when running non-scm commands).
+        original_cwd = os.path.abspath(&quot;.&quot;)
+        if not self._scm:
+            self._scm = detect_scm_system(original_cwd)
+
+        if not self._scm:
+            script_directory = os.path.abspath(sys.path[0])
+            webkit_directory = os.path.abspath(os.path.join(script_directory, &quot;../..&quot;))
+            self._scm = detect_scm_system(webkit_directory)
+            if self._scm:
+                log(&quot;The current directory (%s) is not a WebKit checkout, using %s&quot; % (original_cwd, webkit_directory))
+            else:
+                error(&quot;FATAL: Failed to determine the SCM system for either %s or %s&quot; % (original_cwd, webkit_directory))
+
+        return self._scm
+
+    def path(self):
+        return __file__
+
+    def should_show_in_main_help(self, command):
+        if not command.show_in_main_help:
+            return False
+        if command.requires_local_commits:
+            return self.scm().supports_local_commits()
+        return True
+
+    # FIXME: This may be unnecessary since we pass global options to all commands during execute() as well.
+    def handle_global_options(self, options):
+        if options.dry_run:
+            self.scm().dryrun = True
+            self.bugs.dryrun = True
+        if options.status_host:
+            self.status_server.set_host(options.status_host)
+
+    def should_execute_command(self, command):
+        if command.requires_local_commits and not self.scm().supports_local_commits():
+            failure_reason = &quot;%s requires local commits using %s in %s.&quot; % (command.name, self.scm().display_name(), self.scm().checkout_root)
+            return (False, failure_reason)
+        return (True, None)
</ins></span></pre>
</div>
</div>

</body>
</html>