Commit 0187ea8d authored by eric@webkit.org's avatar eric@webkit.org

2010-12-10 Eric Seidel <eric@webkit.org>

        Reviewed by Adam Barth.

        Move buildbot.py into its own module so we can split it out into one-file-per-class
        https://bugs.webkit.org/show_bug.cgi?id=50806

        We're adding more buildbot logic these days, so it makes sense
        to give buildbot its own module.

        * Scripts/webkitpy/common/net/buildbot/__init__.py: Added.
        * Scripts/webkitpy/common/net/buildbot/buildbot.py: Renamed from WebKitTools/Scripts/webkitpy/common/net/buildbot.py.
        * Scripts/webkitpy/common/net/buildbot/buildbot_unittest.py: Renamed from WebKitTools/Scripts/webkitpy/common/net/buildbot_unittest.py.
        * Scripts/webkitpy/common/net/failuremap.py:
        * Scripts/webkitpy/common/net/regressionwindow.py:
        * Scripts/webkitpy/tool/commands/rebaseline.py:

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@73776 268f45cc-cd09-0410-ab3c-d52691b4dbfc
parent 90c1b24c
2010-12-10 Eric Seidel <eric@webkit.org>
Reviewed by Adam Barth.
Move buildbot.py into its own module so we can split it out into one-file-per-class
https://bugs.webkit.org/show_bug.cgi?id=50806
We're adding more buildbot logic these days, so it makes sense
to give buildbot its own module.
* Scripts/webkitpy/common/net/buildbot/__init__.py: Added.
* Scripts/webkitpy/common/net/buildbot/buildbot.py: Renamed from WebKitTools/Scripts/webkitpy/common/net/buildbot.py.
* Scripts/webkitpy/common/net/buildbot/buildbot_unittest.py: Renamed from WebKitTools/Scripts/webkitpy/common/net/buildbot_unittest.py.
* Scripts/webkitpy/common/net/failuremap.py:
* Scripts/webkitpy/common/net/regressionwindow.py:
* Scripts/webkitpy/tool/commands/rebaseline.py:
2010-12-10 Tony Chang <tony@chromium.org> 2010-12-10 Tony Chang <tony@chromium.org>
Reviewed by Eric Seidel. Reviewed by Eric Seidel.
......
# Required for Python to search this directory for module files
# We only export public API here.
# It's unclear if Builder and Build need to be public.
from .buildbot import BuildBot, Builder, Build
...@@ -28,7 +28,8 @@ ...@@ -28,7 +28,8 @@
import unittest import unittest
from webkitpy.common.net.buildbot import BuildBot, Builder, Build, LayoutTestResults from webkitpy.common.net.layouttestresults import LayoutTestResults
from webkitpy.common.net.buildbot import BuildBot, Builder, Build
from webkitpy.thirdparty.BeautifulSoup import BeautifulSoup from webkitpy.thirdparty.BeautifulSoup import BeautifulSoup
......
...@@ -27,6 +27,7 @@ ...@@ -27,6 +27,7 @@
# OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. # OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
# FIXME: This probably belongs in the buildbot module.
class FailureMap(object): class FailureMap(object):
def __init__(self): def __init__(self):
self._failures = [] self._failures = []
......
...@@ -27,6 +27,7 @@ ...@@ -27,6 +27,7 @@
# OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. # OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
# FIXME: This probably belongs in the buildbot module.
class RegressionWindow(object): class RegressionWindow(object):
def __init__(self, build_before_failure, failing_build, failing_tests=None): def __init__(self, build_before_failure, failing_build, failing_tests=None):
self._build_before_failure = build_before_failure self._build_before_failure = build_before_failure
......
...@@ -31,7 +31,8 @@ import re ...@@ -31,7 +31,8 @@ import re
import shutil import shutil
import urllib import urllib
from webkitpy.common.net.buildbot import BuildBot, LayoutTestResults from webkitpy.common.net.buildbot import BuildBot
from webkitpy.common.net.layouttestresults import LayoutTestResults
from webkitpy.common.system.user import User from webkitpy.common.system.user import User
from webkitpy.layout_tests.port import factory from webkitpy.layout_tests.port import factory
from webkitpy.tool.grammar import pluralize from webkitpy.tool.grammar import pluralize
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment