Skip to content

pylint 1.8.1: test failures during build #1847

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
sandrotosi opened this issue Jan 22, 2018 · 2 comments
Closed

pylint 1.8.1: test failures during build #1847

sandrotosi opened this issue Jan 22, 2018 · 2 comments
Labels
Bug 🪲 Needs reproduction 🔍 Need a way to reproduce it locally on a maintainer's machine

Comments

@sandrotosi
Copy link
Contributor

Hello,
during debian package building we run the unittests, but with 1.8.1 there are quite some failures, here i'm pasting them below:

(how we run tests)

for python in python2.7 python3.6 ; do \
        LIB=$($python -c "from distutils.command.build import build ; from distutils.core import Distribution ; b = build(Distribution()) ; b.finalize_options() ; print(b.build_purelib)") ; \
        PYTHONPATH=/home/morph/deb/build-area/pylint-1.8.1/$LIB $python setup.py pytest --addopts "-k 'not test_good_comprehension_checks'" ; \
done

i uploaded the full build log at https://people.debian.org/~morph/pylint_1.8.1-1_amd64.build.txt (since the cut & past of the whole test failures is too big for GH issue text size limit) you can see the 2 test runs (one for 2.7 and one for 3.6) looking for the string "test session starts"

@PCManticore
Copy link
Contributor

Thanks for the report! At least the ImportFrom error seems something that should already be solved either in master or in the 1.8 branch. Will take a look at the rest to see what's happening.

@PCManticore PCManticore added Bug 🪲 Needs reproduction 🔍 Need a way to reproduce it locally on a maintainer's machine labels Jan 23, 2018
simobasso added a commit to simobasso/codacy-pylint that referenced this issue Jan 23, 2018
1.8.1 seems broken in some way
pylint-dev/pylint#1847
@PCManticore
Copy link
Contributor

Closing this one in favour of #1981 Thanks for creating these issues @sandrotosi !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug 🪲 Needs reproduction 🔍 Need a way to reproduce it locally on a maintainer's machine
Projects
None yet
Development

No branches or pull requests

2 participants