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

Humble: Suppress mcap warnings. #1854

Merged
merged 1 commit into from
Nov 6, 2024

Conversation

clalancette
Copy link
Contributor

When building humble with Conda, we get lots of warnings about dll-import problems. However, we know that these are safe to ignore for mcap, as this library is strictly used internally and we suppress this on Rolling. Do the same suppression here.

(to be perfectly frank, I don't understand why this is a problem with conda and not with the current setup, but this is an easy enough solution)

When building humble with Conda, we get lots of warnings
about dll-import problems.  However, we know that these
are safe to ignore for mcap, as this library is strictly
used internally and we suppress this on Rolling.  Do
the same suppression here.

(to be perfectly frank, I don't understand why this is a
problem with conda and not with the current setup, but this
is an easy enough solution)

Signed-off-by: Chris Lalancette <[email protected]>
@clalancette clalancette requested a review from a team as a code owner November 6, 2024 14:02
@clalancette clalancette requested review from emersonknapp and jhdcs and removed request for a team November 6, 2024 14:02
@clalancette
Copy link
Contributor Author

Pulls: #1854
Gist: https://gist.githubusercontent.com/clalancette/d059559588e3007f6af19394e3f75bb0/raw/36b1b959d06ff2d669b8fab0b26b57e84b299562/ros2.repos
BUILD args: --packages-above-and-dependencies rosbag2_storage_mcap
TEST args: --packages-above rosbag2_storage_mcap
ROS Distro: humble
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/14791

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

@clalancette
Copy link
Contributor Author

The RHEL warnings are unrelated, so going ahead and merging this one.

@clalancette clalancette merged commit 2e07450 into humble Nov 6, 2024
14 checks passed
@clalancette clalancette deleted the clalancette/suppress-mcap-warnings branch November 6, 2024 16:33
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