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

ROS2TransportをWindowsでビルドする場合の警告抑制、エラー修正 #1162

Merged
merged 8 commits into from
Sep 24, 2024

Conversation

Nobu19800
Copy link

Identify the Bug

ros2-jazzy-20240705-windows-release-amd64を使う場合にヘッダファイルで警告、またビルドのコマンドが長すぎるためエラーが発生する。

Description of the Change

  • C4996とC4819の警告はROS2のヘッダーファイルからの発生のため抑制した。
  • コマンドを短くするため、リンクするライブラリを一部削除した。

Verification

  • Did you succeed the build?
  • No warnings for the build?
  • Have you passed the unit tests?

@Nobu19800 Nobu19800 added the bug Something isn't working label Sep 20, 2024
@Nobu19800 Nobu19800 self-assigned this Sep 20, 2024
n-ando
n-ando previously approved these changes Sep 24, 2024
@Nobu19800 Nobu19800 dismissed n-ando’s stale review September 24, 2024 00:37

The merge-base changed after approval.

@n-ando n-ando merged commit 67a49ed into OpenRTM:master Sep 24, 2024
4 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants