Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 575112 - =dev-python/mccabe-0.4.0: build failure due to attempt to fetch 'pytest-runner'
Summary: =dev-python/mccabe-0.4.0: build failure due to attempt to fetch 'pytest-runner'
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Development (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Python Gentoo Team
URL: https://github.com/gentoo/gentoo/pull...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-02-18 23:28 UTC by Coacher
Modified: 2016-02-19 21:10 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---

Attachments
emerge --info mccabe (info,6.73 KB, text/plain)
2016-02-18 23:29 UTC, Coacher 		Details
build.log (build.log,12.12 KB, text/plain)
2016-02-18 23:29 UTC, Coacher 		Details
mccabe-0.4.0.patch (mccabe-0.4.0.patch,6.69 KB, patch)
2016-02-19 21:08 UTC, Coacher 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Coacher 2016-02-18 23:28:51 UTC 
Hello.

`emerge -1v =dev-python/mccabe-0.4.0' results in a build failure.
This is most probably because 'pytest-runner' is listed in setup_requires in 0.4.0 (see [0]) and I don't have it installed. Thus setuptools tries to fetch it during installation and fails.

build.log is attached below. You can also find a trace of a deadlock there because coredumps are forbidden on my machine.

Please fix.

[0]: https://github.com/PyCQA/mccabe/compare/0.3.1...master#diff-2eeaed663bd0d25b7e608891384b7298R36
Comment 1 Coacher 2016-02-18 23:29:28 UTC 
Created attachment 425866 [details]
emerge --info mccabe
Comment 2 Coacher 2016-02-18 23:29:55 UTC 
Created attachment 425868 [details]
build.log
Comment 3 Coacher 2016-02-18 23:33:13 UTC 
FYI `emerge -1v =dev-python/mccabe-0.3.1' builds and installs fine.
Comment 4 Patrick Lauer gentoo-dev 2016-02-19 07:09:54 UTC 
Fixed.

I thought it was only a test dep and didn't notice at build time.
Comment 5 Coacher 2016-02-19 21:08:10 UTC 
Created attachment 425938 [details, diff]
mccabe-0.4.0.patch

(In reply to Patrick Lauer from comment #4)
> Fixed.
> 
> I thought it was only a test dep and didn't notice at build time.

Fixed, but I believe it could be fixed better.

pytest-runner is not needed at all. The only thing that is actually needed for tests is pytest and it's required only for tests, not the regular installation.

Here's a patch that fixes test dependencies and testsuite itself as well as a couple of other minor problems.