search

colcon / colcon-cmake

Extension for colcon to support CMake packages
http://colcon.readthedocs.io
Apache License 2.0
16 stars 25 forks source link

Exclude test directory from package search #77

Closed cottsay closed 4 years ago

cottsay commented 4 years ago

The files in the test directory are getting installed as an additional package.

The __init__.py was added in #57, so this bug has existed since 0.2.18.

dirk-thomas commented 4 years ago

I tried to remove the test/__init__.py file instead: c280d5b037dbef6fdd0865d25e44f89683747376. To make that work Travis CI needs to invoke python -m pytest instead of pytest: 4f2a86feb14e43f27c28035100150c7152c2834e.

Since that seems to work I would rather go that route. Does that make sense to you?

cottsay commented 4 years ago

That's how I'm invoking the tests already, so it works for me. Thanks!

dirk-thomas commented 4 years ago

Thanks for the fix.