Skip to content

Revert "rosbag2_py reader and writer"#507

Closed
mabelzhang wants to merge 1 commit intomasterfrom
revert-308-rosbag2_py
Closed

Revert "rosbag2_py reader and writer"#507
mabelzhang wants to merge 1 commit intomasterfrom
revert-308-rosbag2_py

Conversation

@mabelzhang
Copy link
Copy Markdown
Contributor

Reverts #308 due to Nightly failures on Debug mode in Windows #504 and Linux with Clang #505

We have a solution for Linux with Clang that passes the compiler but the tests are failing.
Still need to look into Windows.

@mabelzhang mabelzhang requested a review from clalancette August 14, 2020 22:49
@jacobperron
Copy link
Copy Markdown
Member

What about disabling the package with an ignore file while we work to fix CI for the affected platforms?

@mabelzhang mabelzhang marked this pull request as draft August 14, 2020 22:56
@mabelzhang
Copy link
Copy Markdown
Contributor Author

Closing in favor of adding AMENT_IGNORE

@mabelzhang mabelzhang closed this Aug 14, 2020
@emersonknapp
Copy link
Copy Markdown
Collaborator

I would not revert based on Clang since it's not a tier 1 platform, I'd instead just make sure to prioritize looking for a fix - though I see you closed this so I guess we're good to go 👍

@clalancette
Copy link
Copy Markdown
Contributor

I would not revert based on Clang since it's not a tier 1 platform, I'd instead just make sure to prioritize looking for a fix - though I see you closed this so I guess we're good to go +1

That's true, but I think it is also causing problems on Windows Debug: https://ci.ros2.org/view/nightly/job/nightly_win_deb/1714/#showFailuresLink

In any case, I like the idea to AMENT_IGNORE it for now, then deal with it next week.

@mabelzhang mabelzhang deleted the revert-308-rosbag2_py branch August 15, 2020 00:58
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.

4 participants