pantsbuild #general

as a heads up: we just updated to the latest version of zinc, and there were some regressions in zinc's analysis accuracy. so the unused deps warning will likely be disabled by default in the next releases

witty-crayon-22786

11/18/2016, 4:29 PM

https://github.com/pantsbuild/pants/issues/4072

stale-chef-81972

11/18/2016, 6:45 PM

hey i've got a little issue maybe someone can help me with... i'm having some trouble using a python package (https://github.com/Azure/azure-storage-python) that seems pants-specific

stale-chef-81972

11/18/2016, 6:45 PM

if i

pip install azure-storage

, i can hop in a repl and

from azure.storage.blob import x

just fine

stale-chef-81972

11/18/2016, 6:46 PM

but in the pants repl for my project, i can only

import azure

, importing from azure.storage yields

No module named storage

witty-crayon-22786

11/18/2016, 6:46 PM

does

./pants dependencies $target

show the relevant dep?

stale-chef-81972

11/18/2016, 6:47 PM

yup

witty-crayon-22786

11/18/2016, 6:47 PM

./pants dependencies $target

shows it, then

./pants repl $target

should include it

stale-chef-81972

11/18/2016, 6:47 PM

yup, and i can import azure in the repl context

stale-chef-81972

11/18/2016, 6:47 PM

azure-storage is the only azure-specific dependency i declared

witty-crayon-22786

11/18/2016, 6:48 PM

does your code have package collisions with the thing you're importing?

stale-chef-81972

11/18/2016, 6:48 PM

i'm also building a binary and i've poked around under .deps to make sure storage & blob packages are there, which they are

stale-chef-81972

11/18/2016, 6:48 PM

it shouldn't..

stale-chef-81972

11/18/2016, 6:49 PM

if i pip install everything from the requirements.txt that's also powering the pants repl, there's no issue

witty-crayon-22786

11/18/2016, 6:49 PM

the package/module of your code will be everything relative to the nearest source_root: http://www.pantsbuild.org/setup_repo.html#source-roots

stale-chef-81972

11/18/2016, 6:51 PM

there's nothing in our code that should conflict

witty-crayon-22786

11/18/2016, 6:53 PM

@stale-chef-81972 : what does the BUILD file that defines the requirement look like?.

stale-chef-81972

11/18/2016, 6:55 PM

i'm using

python_requirements()

with a requirements.txt file

stale-chef-81972

11/18/2016, 6:55 PM

should i try defining it explicitly as a python_requirement_library?

witty-crayon-22786

11/18/2016, 6:56 PM

no, i just repro'd with

python_requirement_library

stale-chef-81972

11/18/2016, 6:56 PM

interestingly, other people have had this issue with this package but always seem to have resolved by uninstalling & reinstalling with pip or something of the like

stale-chef-81972

11/18/2016, 6:57 PM

according to SO

witty-crayon-22786

11/18/2016, 7:03 PM

hm, yea, malformed python packages are a bit above my pay grade =/

witty-crayon-22786

11/18/2016, 7:04 PM

.@kwlzn may know

limited-country-20626

11/18/2016, 7:15 PM

you can try installing and reinstalling in the pantsenv. In pants master it is

./build-support/pants_dev_deps.venv/bin/pip

but for your repo it may be under ~/.cache

witty-crayon-22786

11/18/2016, 7:16 PM

my first attempt to use it repro'd the problem, so i suspect an issue with the package... possibly that it has undeclared deps?

limited-country-20626

11/18/2016, 7:17 PM

or missing

__init__.py

file at that level