search

nv-morpheus / Morpheus

Morpheus SDK
Apache License 2.0
309 stars 119 forks source link

[BUG]: Install target installs headers into `/usr/local/include` rather than the conda env #1776

Open dagardner-nv opened 6 days ago

dagardner-nv commented 6 days ago

Version

24.06

Which installation method(s) does this occur on?

Docker

Describe the bug.

This happens in the dev container, not sure about other envs.

Minimum reproducible example

docker/run_container_dev.sh
CMAKE_CONFIGURE_EXTRA_ARGS="-DMORPHEUS_PYTHON_INPLACE_BUILD=OFF" ./scripts/compile.sh --target install


### Relevant log output

<details><summary>Click here to see error details</summary><pre>

 [Paste the error here, it will be hidden by default]

</pre></details>

### Full env printout

<details><summary>Click here to see environment details</summary><pre>

 [Paste the results of print_env.sh here, it will be hidden by default]

</pre></details>

### Other/Misc.

_No response_

### Code of Conduct

- [X] I agree to follow Morpheus' Code of Conduct
- [X] I have searched the [open bugs](https://github.com/nv-morpheus/Morpheus/issues?q=is%3Aopen+is%3Aissue+label%3Abug) and have found no duplicates for this bug report
mdemoret-nv commented 6 days ago

Are you recommending that we install to /usr/local/include or that it is an error that we install there?

dagardner-nv commented 6 days ago

It currently installs headers into /usr/local/include when it should be installing into the conda env.