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

Issue 859975 link

Starred by 1 user

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

CheckDeps error message is not actionable

Project Member Reported by khorimoto@chromium.org, Jul 3

Issue description

I've been helping several people who are new to Chromium, and they've all been bitten by error messages like:

      Illegal include: "path/to/file.h"
    Because of no rule applying.

They've been pretty confused about what this really means and why the include is illegal. I think we should change the error messages in checkdeps_test.py [1] to make it clear that the developer needs to add/update the appropriate DEPS file.

[1] https://cs.chromium.org/chromium/src/buildtools/checkdeps/checkdeps_test.py
 
Components: Infra>Client>Chrome
Owner: ----
Status: Available (was: Assigned)

Sign in to add a comment