Skip to content

lyrical: plotjuggler needs python in host - #198

Open
EzraBrooks wants to merge 1 commit into
RoboStack:mainfrom
EzraBrooks:lyrical-plotjuggler-python
Open

EzraBrooks wants to merge 1 commit into
RoboStack:mainfrom
EzraBrooks:lyrical-plotjuggler-python

Conversation

@EzraBrooks

Copy link
Copy Markdown

Split out from #193

plotjuggler embeds the Python interpreter (plotjuggler_app links Python3::Python for its Python custom transforms) without declaring a Python dependency in package.xml. Today every lyrical package has python in host anyway, so this changes nothing yet; it keeps plotjuggler built per Python once packages without Python content stop getting python (vinca's conditional_python_dependency).

plotjuggler embeds the Python interpreter (plotjuggler_app links
Python3::Python for its Python custom transforms) without declaring a
Python dependency in package.xml. Today every lyrical package has python
in host anyway, so this changes nothing yet; it keeps plotjuggler built
per Python once packages without Python content stop getting python
(vinca's conditional_python_dependency).
@EzraBrooks

Copy link
Copy Markdown
Author

I think the CI run here got bit by RoboStack/vinca#191 while other PRs had merged ahead of it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant