qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Gerd Hoffmann <kraxel@redhat.com>
To: berendt@b1-systems.de
Cc: qemu-devel@nongnu.org, Gerd Hoffmann <kraxel@redhat.com>
Subject: [Qemu-devel] [buildbot patch 1/6] add linebreaks to make create_build_factory more readable
Date: Fri,  1 Feb 2013 15:02:20 +0100	[thread overview]
Message-ID: <1359727345-10477-2-git-send-email-kraxel@redhat.com> (raw)
In-Reply-To: <1359727345-10477-1-git-send-email-kraxel@redhat.com>

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
---
 qemu-master.cfg |   24 ++++++++++++++++++------
 1 file changed, 18 insertions(+), 6 deletions(-)

diff --git a/qemu-master.cfg b/qemu-master.cfg
index 03db6ee..2f793c1 100644
--- a/qemu-master.cfg
+++ b/qemu-master.cfg
@@ -681,17 +681,29 @@ from buildbot.steps.shell import Configure, Compile, ShellCommand, Test
 #  test=None
 #)
 
-def create_build_factory(repourl, branch="HEAD", make="make", configure_args=[], outoftree=False):
+def create_build_factory(repourl, branch="HEAD",
+                         make="make", configure_args=[],
+                         outoftree=False):
     """Return a BuildFactory pre-configured with common build steps"""
     f = factory.BuildFactory()
-    f.addStep(Git(repourl=repourl, timeout=2400, mode='copy', retry=(5 * 60, 3), locks=repo_locks_from_url(repourl)))
+    f.addStep(Git(repourl=repourl,
+                  timeout=2400, mode='copy', retry=(5 * 60, 3),
+                  locks=repo_locks_from_url(repourl)))
     workdir = f.workdir
     if outoftree:
-        f.addStep(ShellCommand(command="mkdir -p outoftree", env={'LANG': 'C'}))
+        f.addStep(ShellCommand(command="mkdir -p outoftree",
+                               env={'LANG': 'C'}))
         workdir = os.path.join(workdir, 'outoftree')
-    f.addStep(Configure(command=["./configure"] + list(configure_args), env={'LANG': 'C'}, workdir=workdir))
-    f.addStep(Compile(command=[make, "CFLAGS=-O2"], env={'LANG': 'C'}, timeout=2400, workdir=workdir))
-    f.addStep(Test(command=[make, "check"], env={'LANG': 'C'}, maxTime=2400, workdir=workdir))
+    f.addStep(Configure(command=["./configure"] + list(configure_args),
+                        env={'LANG': 'C'}, workdir=workdir))
+    f.addStep(Compile(command=[make, "CFLAGS=-O2"],
+                      env={'LANG': 'C'},
+                      timeout=2400,
+                      workdir=workdir))
+    f.addStep(Test(command=[make, "check"],
+                   env={'LANG': 'C'},
+                   maxTime=2400,
+                   workdir=workdir))
     return f
 
 f_default = create_build_factory("git://git.qemu-project.org/qemu.git")
-- 
1.7.9.7

  reply	other threads:[~2013-02-01 15:22 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-02-01 14:02 [Qemu-devel] [buildbot patch 0/6] buildbot update Gerd Hoffmann
2013-02-01 14:02 ` Gerd Hoffmann [this message]
2013-02-01 14:02 ` [Qemu-devel] [buildbot patch 2/6] make tests configurable Gerd Hoffmann
2013-02-01 14:02 ` [Qemu-devel] [buildbot patch 3/6] fix out-of-tree build Gerd Hoffmann
2013-02-01 14:02 ` [Qemu-devel] [buildbot patch 4/6] use --disable-debug-info Gerd Hoffmann
2013-02-01 14:02 ` [Qemu-devel] [buildbot patch 5/6] save config.log Gerd Hoffmann
2013-02-01 14:02 ` [Qemu-devel] [buildbot patch 6/6] add log tail to failure reports Gerd Hoffmann

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1359727345-10477-2-git-send-email-kraxel@redhat.com \
    --to=kraxel@redhat.com \
    --cc=berendt@b1-systems.de \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).