Bisect - Handle failing to expand revisions more gracefully. |
||||
Issue descriptionhttps://bugs.chromium.org/p/chromium/issues/detail?id=680223#c3 Here someone put in the wrong revision range, we should handle bad input like that more gracefully and output a helpful error.
,
Jan 13 2017
We probably have enough information on the dashboard so that given two chrome commit positions A and B, to find, say, the first clank hash A' with src@A and the last clank hash B' with src@B. Could that be doable? For humans entering manual ranges it's often much easier to think about commit positions rather than, e.g., clank hashes.
,
Jan 13 2017
Simon: for background on how the dashboard deals with clank revisions, see the discussion in https://github.com/catapult-project/catapult/issues/3014 If a_default_rev were set properly, maybe we could have special handling of multiple revision types in the bisect dialog? It would definitely add complesity, so would depend how big a problem this is.
,
Jan 13 2017
So if I'm understanding correctly, the suggestion is if you're on clank and input chromium revisions, to auto-match them to when clank rolled them and fill in the corresponding clank hashes?
,
Feb 3 2017
|
||||
►
Sign in to add a comment |
||||
Comment 1 by simonhatch@chromium.org
, Jan 12 2017