[PATCH] Move sanitizer bots's configs from an external repo to zorg

Reid Kleckner rnk at google.com
Thu Oct 2 15:32:21 PDT 2014


This seems like a reasonable place to put these scripts to me.

>>! In D5531#7, @rengolin wrote:
> Stupid question, but did you have those scripts before Zorg had any builders? They seem completely disconnected with anything Zorg does. I was expecting a Builder class, like Buildbot docs describe...

Yeah, see http://llvm.org/viewvc/llvm-project/zorg/trunk/zorg/buildbot/builders/SanitizerBuilder.py?view=markup . We would change that to point the svn checkout at zorg instead of some Google Code svn repo.

================
Comment at: zorg/buildbot/builders/sanitizers/buildbot_chrome_asan.sh:1
@@ +1,2 @@
+#!/bin/bash
+
----------------
I doubt we need these *_chrome*.sh scripts.

================
Comment at: zorg/buildbot/builders/sanitizers/slaveutil.py:1
@@ +1,2 @@
+#!/usr/bin/python
+
----------------
This file looks dead, even in the ASan repo. We probably don't need it.

http://reviews.llvm.org/D5531






More information about the llvm-commits mailing list