New issue
Advanced search Search tips

Issue 619329 link

Starred by 3 users

Issue metadata

Status: Archived
Owner: ----
Closed: Jun 2017
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

Lint doesn't re-run when lint.py is changed

Reported by mlopat...@yandex-team.ru, Jun 11 2016

Issue description

Steps to reproduce the problem:
1. Checkout sources, configure android build, build chrome_public_apk
2. Change something in build/android/gyp/lint.py, in _OnStaleMd5. 
3. Build chrome_public_apk again

What is the expected behavior?
lint.py is run and produces error reports according to changes

What went wrong?
Something runs but actual lint tool isn't invoked and there are no errors.

Did this work before? N/A 

Chrome version: N/A  Channel: canary
OS Version: 
Flash Version: N/A

Lint.py computes md5 digest of all its inputs and skips actual linting if content of the inputs is up-to-date. But the content of the script itself isn't added to the digest, only path to the script.
 
Fix landed but there is no corresponding comment here.

https://codereview.chromium.org/2061573002
Project Member

Comment 2 by sheriffbot@chromium.org, Jun 21 2017

Status: Archived (was: Unconfirmed)
Issue has not been modified or commented on in the last 365 days, please re-open or file a new bug if this is still an issue.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Sign in to add a comment