New issue
Advanced search Search tips

Issue 632901 link

Starred by 3 users

Issue metadata

Status: Verified
Owner:
Closed: Aug 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Missing pylint dependencies from autotest utils/build_externals.py

Project Member Reported by ayatane@chromium.org, Jul 29 2016

Issue description

utils/build_externals.py does not install needed dependencies for pylint:

Astroid, logilab-common

 
Description: Show this description
Description: Show this description
Owner: ayatane@chromium.org
Status: Started (was: Untriaged)
I think you're working on this (https://chromium-review.googlesource.com/#/c/364881/)

Plus you should use your chromium account to upload cls instead of google.
Project Member

Comment 4 by bugdroid1@chromium.org, Aug 10 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/a9f2a2b966058047b734d987352d8df17d6f6c56

commit a9f2a2b966058047b734d987352d8df17d6f6c56
Author: Allen Li <ayatane@google.com>
Date: Wed Aug 03 20:44:50 2016

[autotest] Add missing pylint dependencies

astroid and logilab-common are pylint dependencies that aren't installed
automatically, so we need to install them ourselves.

This needs to do some egg-info metadata fiddling because Python
packaging is complicated.

BUG= chromium:632901 
TEST=Ran build_externals.py and run_pylint.py with clean state

Change-Id: I80c4188609541211ab1d495352328a89599e55d8
Reviewed-on: https://chromium-review.googlesource.com/364881
Commit-Ready: Allen Li <ayatane@chromium.org>
Tested-by: Allen Li <ayatane@chromium.org>
Reviewed-by: Simran Basi <sbasi@chromium.org>

[modify] https://crrev.com/a9f2a2b966058047b734d987352d8df17d6f6c56/utils/external_packages.py

Status: Verified (was: Started)

Sign in to add a comment