pip install add tests folder to site-packages about aiogoogle HOT 9 CLOSED

Hey Matresh, are you installing directly from the git repo or from Pypi (via pip)?

Because in the manifest.in file, I am excluding the tests folder. And on my machine, the tests directory is in fact not being included.

I think it would be better if we exclude in one place i.e. manifest.in. As per this, wouldn't adding recursive-exclude tests* to the manifest.in be a better solution?

from aiogoogle.

Yeah, Using Pypi:
pip install aiogoogle

Using manifest.in will be better i think.

from aiogoogle.

I never worked with manifest.in but if I'm correctly understand prune must be similar to recursive-exclude tests* but prune currently not working

from aiogoogle.

Hmm, weird. Can you try and see if recursive-exclude tests* works for you instead of prune tests. If it does, I can merge that.

from aiogoogle.

Im not sure how can i test it, directly pip install from git repo is representative?
Or need to upload into PyPi ?

from aiogoogle.

I have 2 packages aiogoogle and test in top_level.txt

And 2 packages in -site-packages

from aiogoogle.

This is weird.

Im not sure how can i test it, directly pip install from git repo is representative?

Yes. Clone it, then make your changes. Then, in the CLI, write: python setup.py sdist. Then , you'll find the new package in the dist directory.

I did just that right now, and added the line that you wanted to add to the setup.py file and it didn't change anything. I also tried replacing prune with global-exclude and replacing tests/ with **/*tests and also nothing.

All of the methods I tried didn't include the tests directory in the .tar package. Also, all of them included the line tests in the top_level.txt file.

Are you installing directly from Github? Maybe that's why you're getting this error.

If adding exclude=["tests"] is the only thing that works for you, I don't mind merging it.

from aiogoogle.

Thanks for help, I've tested it for myself and any type of exclude from MANIFEST.in isn't working for me. exclude=["tests"] is not working too, but exclude=['tests*'] is working fine, i will create pull request with that changes soon

from aiogoogle.

Alrighty. Thank you Matresh! I'll release a new version now.

from aiogoogle.