storm-0.20 installs tests directory in site-packages

Bug #1199578 reported by Mike Gilbert on 2013-07-10
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Storm
High
Colin Watson

Bug Description

There is no reason to install the tests directory, and doing so can actually cause package manager file conflicts if multiple packages do so.

See patch for an easy solution.

Related branches

Mike Gilbert (floppym) wrote :
Aurélien Bompard (abompard) wrote :

Actually you need to exclude the submodules of the tests module too, so the line should be:

packages=find_packages(exclude=['tests', 'tests.*'])

Colin Watson (cjwatson) wrote :

I agree that the current behaviour is annoying. Installing tests (in a better place!) is often handy so that you can do as-installed tests, though, and many Python packages do so. IMO the best thing to do here would instead be to move the tests to a storm.tests package so that they won't conflict with anything else.

Changed in storm:
assignee: nobody → Colin Watson (cjwatson)
importance: Undecided → High
status: New → In Progress
Colin Watson (cjwatson) on 2019-08-12
Changed in storm:
status: In Progress → Fix Committed
Colin Watson (cjwatson) on 2019-09-20
Changed in storm:
milestone: none → 0.21
Colin Watson (cjwatson) on 2019-09-20
Changed in storm:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers