[telemetry] Remove all pyc files before doing presumibit work

[telemetry] Remove all pyc files before doing presumibit work

Avoids presubmit failures when stale pyc files are found

NOTRY=True

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=207955

[tonyg, 2013-06-21 19:02:38]

lgtm

I think the discover code may need to do the same thing

https://codereview.chromium.org/17560002/diff/1/tools/telemetry/build/update_...
File tools/telemetry/build/update_docs.py (right):

https://codereview.chromium.org/17560002/diff/1/tools/telemetry/build/update_...
tools/telemetry/build/update_docs.py:31: pycpath = os.path.join(dirname,
filename)
I'd find these more readable as pyc_path and py_path.

[dtu, 2013-06-21 19:29:32]

On 2013/06/21 19:02:38, tonyg wrote:
> lgtm
> 
> I think the discover code may need to do the same thing
> 
Hrm, the problem was just a regular Python import... I don't know what we can do
about those.
https://code.google.com/p/chromium/issues/detail?id=252808

lgtm

[commit-bot: I haz the power, 2013-06-21 23:25:35]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nduca@chromium.org/17560002/6001

[commit-bot: I haz the power, 2013-06-21 23:31:55]

Change committed as 207955

[achuithb, 2013-06-24 22:05:06]

https://chromiumcodereview.appspot.com/17560002/diff/6001/tools/telemetry/bui...
File tools/telemetry/build/update_docs.py (right):

https://chromiumcodereview.appspot.com/17560002/diff/6001/tools/telemetry/bui...
tools/telemetry/build/update_docs.py:34: os.remove(py_path)
SHouldn't this be pyc_path?

[dtu, 2013-06-24 22:14:37]

https://chromiumcodereview.appspot.com/17560002/diff/6001/tools/telemetry/bui...
File tools/telemetry/build/update_docs.py (right):

https://chromiumcodereview.appspot.com/17560002/diff/6001/tools/telemetry/bui...
tools/telemetry/build/update_docs.py:34: os.remove(py_path)
On 2013/06/24 22:05:06, achuith.bhandarkar wrote:
> SHouldn't this be pyc_path?

lol yes, you're right