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

🧑‍🌾 Error C2065: 'M_PI': undeclared identifier in AdvancedLiftDrag.cc(618,64) #2229

Closed
Crola1702 opened this issue Nov 6, 2023 · 2 comments · Fixed by #2230
Closed
Assignees
Labels
bug Something isn't working

Comments

@Crola1702
Copy link
Contributor

Environment

  • OS Version: Windows CI
  • Source gz-sim build

Description

  • Expected behavior: Build should pass
  • Actual behavior: Build is failing because of an undeclared identifier

Steps to reproduce

  1. Run a job in gz-sim7 or 8
  2. See build fail

Output

Reference build: gz-sim8-win

Log output:

  gz-sim8-acoustic-comms-system.vcxproj -> C:\J\workspace\ign_gazebo-gz-8-win\ws\build\gz-sim8\bin\Release\gz-sim8-acoustic-comms-system.dll

  Building Custom Rule C:/J/workspace/ign_gazebo-gz-8-win/ws/gz-sim/src/systems/advanced_lift_drag/CMakeLists.txt

  AdvancedLiftDrag.cc

C:\J\workspace\ign_gazebo-gz-8-win\ws\gz-sim\src\systems\advanced_lift_drag\AdvancedLiftDrag.cc(618,64): error C2065: 'M_PI': undeclared identifier [C:\J\workspace\ign_gazebo-gz-8-win\ws\build\gz-sim8\src\systems\advanced_lift_drag\gz-sim8-advanced-lift-drag-system.vcxproj]

C:\J\workspace\ign_gazebo-gz-8-win\ws\gz-sim\src\systems\advanced_lift_drag\AdvancedLiftDrag.cc(674,46): error C2065: 'M_PI': undeclared identifier [C:\J\workspace\ign_gazebo-gz-8-win\ws\build\gz-sim8\src\systems\advanced_lift_drag\gz-sim8-advanced-lift-drag-system.vcxproj]

M_PI mentions: https://github.com/gazebosim/gz-sim/blob/gz-sim8/src/systems/advanced_lift_drag/AdvancedLiftDrag.cc#L618 and https://github.com/gazebosim/gz-sim/blob/gz-sim8/src/systems/advanced_lift_drag/AdvancedLiftDrag.cc#L674.

This is only failing in Windows

@Crola1702 Crola1702 added the bug Something isn't working label Nov 6, 2023
@scpeters
Copy link
Member

scpeters commented Nov 6, 2023

it should be GZ_PI

@scpeters
Copy link
Member

scpeters commented Nov 6, 2023

@azeey azeey self-assigned this Nov 6, 2023
azeey added a commit to azeey/gz-sim that referenced this issue Nov 6, 2023
azeey added a commit that referenced this issue Nov 6, 2023
@github-project-automation github-project-automation bot moved this from Inbox to Done in Core development Nov 6, 2023
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
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants