iotile / iotile/coretools

iotile-build should not fail if iotile-sensorgraph is not installed

Open
#797 1 comment 0 reactions 0 assignees View on GitHub
class:self-contained iotile-build type:bug
Dominant language
Python
Stars
14
Forks
7
PR merge metrics
No merged PRs in 30d

Description

There is a global import for an `iotile-sensorgraph` provided feature in [`autobuild.py:29`](https://github.com/iotile/coretools/blob/master/iotilebuild/iotile/build/config/site_scons/autobuild.py#L29)

This function is only used in `autobuild_trub_script` and `install_requires` for `iotile-build` does not list `iotile-sensorgraph`.

The import should be moved within the function that uses it and wrapped in a `try/except` so that a nice error message can be given saying that this feature requires `iotile-sensorgraph`.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.