New deb source arpy dependency is too hard
Sources by themselves are not supposed to expand on BuildStream's hard dependencies, we mostly use preflight for this.
Two things need to be done for this:
-
The
debsource plugin must notimport arpyat the toplevel- It should
try / exceptthe import atpreflight()time and report a sensible SourceError - And
import arpyseparately in the one function where it's used.
- It should
-
The test suite needs to also conditionalize this, so that
./setup.py testdoes not bail out in the absence ofarpy- Add
HAVE_ARPYvariable in the usual way totests/testutils/site.py - Conditionalize the tests which use
arpyin the usual way - Because the import statement is what causes test to bail, the
import arpystatement in the test case probably needs to be moved into the individual test functions, too
- Add
Edited by Tristan Van Berkom