Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(ansible): add new task to install libqt5charts5-dev #4119

Merged
merged 1 commit into from
Jan 25, 2024

Conversation

satoshi-ota
Copy link
Collaborator

@satoshi-ota satoshi-ota commented Jan 25, 2024

Description

Add new task to install libqt5charts5-dev so that we can develop utility tools with qt charts.

Related PR: autowarefoundation/autoware.universe#6154 (I use the lib in this PR.)

NOTE: After merging ros/rosdistro#39705, I'll remove this task.

Tests performed

I confirmed that libqt5charts5-dev was installed correctly with ./setup-dev-env.sh.

Effects on system behavior

Nothing.

Pre-review checklist for the PR author

The PR author must check the checkboxes below when creating the PR.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

  • There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@mitsudome-r
Copy link
Member

Could you explain in the description why you want to add this package? Please link to the issue or PR that requires this library to be installed.

@satoshi-ota
Copy link
Collaborator Author

Could you explain in the description why you want to add this package? Please link to the issue or PR that requires this library to be installed.

@mitsudome-r Thanks your comment. I added the related PR in the description.

@mitsudome-r mitsudome-r merged commit 01feac7 into autowarefoundation:main Jan 25, 2024
19 checks passed
@satoshi-ota satoshi-ota deleted the feat/add-new-task branch January 25, 2024 08:57
@xmfcx
Copy link
Contributor

xmfcx commented Jan 29, 2024

@satoshi-ota -san could you also update the ansible/roles/autoware_universe/README.md file as well?

Those are used in the Manual Installation Instructions.

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.

3 participants