New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 617170 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Last visit 27 days ago
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Bug

Blocked on:
issue 571215



Sign in to add a comment

Delete most of tools/auto_bisect/builder.py

Project Member Reported by thakis@chromium.org, Jun 3 2016

Issue description

auto_bisect/builder.py has code that checks for Visual Studio on Windows, xcodebuild on Mac, etc. We have used nothing but ninja for desktop chrome in years, so all this code is likely dead (else it would've been fixed long ago).

Please delete the dead bits of this file.
 
Blockedon: 571215
Cc: -chrisphan@chromium.org robert...@chromium.org pras...@chromium.org qyears...@chromium.org
Components: -Tools Tests>AutoBisect>Legacy
Owner: pras...@chromium.org
Status: Assigned (was: Untriaged)
Fortunately, all of tools/auto_bisect/ is the old, deprecated bisect system, which the new bisect system has been slowly replacing over the last year -- so very soon we should be able to easily remove all of tools/auto_bisect.

Prasad, how soon can we delete it?
Even if it takes a while to delete all of it, it'd be good to delete obviously dead code before then (since all code that's checked in and unused can confuse people -- me, for example :-) )
Yeah -- actually builder.py is only used for building locally, which we haven't been doing for a while, so that code is indeed dead.

If we can delete the whole thing within about a week, then that'll be a bit simpler; otherwise let's make a change first to delete builder.py.
Cc: -qyears...@chromium.org
Components: Speed>Bisection

Sign in to add a comment