Skip to content

Parametrization of parameter_bridge quality of service [Port of the commits (ec44770) and (86b4245) to foxy branch]#401

Merged
quarkytale merged 3 commits intoros2:foxyfrom
lucyannofrota:foxy
May 24, 2023
Merged

Parametrization of parameter_bridge quality of service [Port of the commits (ec44770) and (86b4245) to foxy branch]#401
quarkytale merged 3 commits intoros2:foxyfrom
lucyannofrota:foxy

Conversation

@lucyannofrota
Copy link
Copy Markdown

@lucyannofrota lucyannofrota commented May 4, 2023

Reproduces the changes made in commit #331 in ros foxy.

Parametrize Quality of Service in parameter_bridge

Signed-off-by: Lucyanno Frota lucyannofrota@gmail.com

@lucyannofrota lucyannofrota changed the title Port of the commit (ec44770) to foxy branch Parametrize Quality of Service in parameter_bridge [Port of the commit (ec44770) to foxy branch] May 4, 2023
@quarkytale
Copy link
Copy Markdown
Contributor

Should this include xmlrpcpp only when ROS 1 is found as in #371 ?

@lucyannofrota
Copy link
Copy Markdown
Author

Sure! Great suggestion @quarkytale. I already made a commit to address this port!

@lucyannofrota lucyannofrota changed the title Parametrize Quality of Service in parameter_bridge [Port of the commit (ec44770) to foxy branch] Parametrization of parameter_bridge quality of service [Port of the commits (ec44770) and (86b4245) to foxy branch] May 15, 2023
@quarkytale quarkytale self-assigned this May 15, 2023
@quarkytale
Copy link
Copy Markdown
Contributor

CI:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Windows Build Status

@quarkytale
Copy link
Copy Markdown
Contributor

Correct CI
Linux Packaging: Build Status

@quarkytale
Copy link
Copy Markdown
Contributor

Thanks for the fix. running again:
Build Status

Just reproduces the changes made in ros2#331 in ros foxy.

Parametrize Quality of Service in `parameter_bridge`

Signed-off-by: Lucyanno Frota <lucyannofrota@gmail.com>
Reproduces the changes made in ros2#371 in ros foxy.

Move xmlrpcpp find_package so it only searches if ROS 1 is found

Signed-off-by: Lucyanno Frota <lucyannofrota@gmail.com>
White spaces removed
Signed-off-by: Lucyanno Frota <lucyannofrota@gmail.com>
@quarkytale
Copy link
Copy Markdown
Contributor

Final CI: Build Status

@quarkytale quarkytale merged commit 2c24a8d into ros2:foxy May 24, 2023
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.

2 participants