Fix windows warning for exception c++ code

[Blast545]

Signed-off-by: Jorge Perez [email protected]

🦟 Bug fix

Fixes #211

Summary

Include the missing flag for Windows builds. This will be forward ported up to ign-fuel-tools-7

Checklist

• Signed all commits for DCO
• Added tests
• Updated documentation (as needed)
• Updated migration guide (as needed)
• codecheck passed (See contributing)
• All tests passed (See test coverage)
• While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge

[codecov]

Codecov Report

Merging #212 (8250612) into ign-fuel-tools4 (d078bd0) will not change coverage.
The diff coverage is n/a.

❗ Current head 8250612 differs from pull request most recent head 91eed29. Consider uploading reports for the commit 91eed29 to get more accurate results

@@               Coverage Diff                @@
##           ign-fuel-tools4     #212   +/-   ##
================================================
  Coverage            78.13%   78.13%           
================================================
  Files                   19       19           
  Lines                 2643     2643           
================================================
  Hits                  2065     2065           
  Misses                 578      578           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d078bd0...91eed29. Read the comment docs.

[azeey]

I think the warning is coming from ign-tools, not from ign-fuel-tools. Should hopefully be fixed by gazebosim/gz-tools#68

[Blast545]

@azeey The error comes from ign-tools indeed, but I am not sure setting the variable there would fix the issue here. If the other PR removes the warnings I'd close this PR, I'd take a look after its merged.

[azeey]

@azeey The error comes from ign-tools indeed, but I am not sure setting the variable there would fix the issue here. If the other PR removes the warnings I'd close this PR, I'd take a look after its merged.

It's merged. Can you give it a try?

[Blast545]

@azeey Sure! I'll just retrigger a CI for ign-fuel_tools4:

• ign-fuel_tools4

[Blast545]

It was fixed, thank you! @azeey !