Skip to content
Snippets Groups Projects
Commit 4d601cff authored by nweiz@google.com's avatar nweiz@google.com
Browse files

Properly detect whether we're running on a buildbot.

BUILDBOT_BUILDERNAME is only set for bots using annotated steps, which
isn't all of them. This detects whether the username is "chrome-bot"
instead.

This check is currently only being used to decide whether to use
unittest's compact VM config or the default config.

R=rnystrom@google.com, sigmund@google.com

Review URL: https://codereview.chromium.org//58933006

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart/pkg/unittest@30206 260f80e4-7a28-3924-810f-c04153c831b5
parent 58435802
No related branches found
No related tags found
No related merge requests found
......@@ -190,7 +190,7 @@ class CompactVMConfiguration extends VMConfiguration {
void useCompactVMConfiguration() {
// If the test is running on the Dart buildbots, we don't want to use this
// config since it's output may not be what the bots expect.
if (Platform.environment.containsKey('BUILDBOT_BUILDERNAME')) {
if (Platform.environment['LOGNAME'] == 'chrome-bot') {
return;
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment