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

Shush warnings from third_party code? #10345

Closed
3 tasks
jmarrec opened this issue Dec 22, 2023 · 1 comment · Fixed by #10346
Closed
3 tasks

Shush warnings from third_party code? #10345

jmarrec opened this issue Dec 22, 2023 · 1 comment · Fixed by #10346
Labels
Developer Issue Related to cmake, packaging, installers, or developer tooling (CI, etc)

Comments

@jmarrec
Copy link
Contributor

jmarrec commented Dec 22, 2023

Issue overview

Consider just ignoring every warning coming from the third_party folder.

I'm getting annoyed by lots and lots of warnings, especially on newer compilers/oses... I just tried clang 17 on an up-to-date mac 14.2.1 and the list is super duper long.

One random example:

/Users/julien/Software/Others/EnergyPlus/third_party/kiva/vendor/boost-1.77.0/boost/type_traits/has_trivial_copy.hpp:34:4: warning: builtin __has_trivial_copy is deprecated; use __is_trivially_copyable instead [-Wdeprecated-builtins]
   34 |    BOOST_HAS_TRIVIAL_COPY(T) BOOST_TT_TRIVIAL_CONSTRUCT_FIX

Possible solution is to just bump to cmake 3.25, and change this line:

add_subdirectory(third_party)

to

add_subdirectory(third_party SYSTEM)

https://cmake.org/cmake/help/latest/command/add_subdirectory.html

Details

Some additional details for this issue (if relevant):

  • Platform (Operating system, version)
  • Version of EnergyPlus (if using an intermediate build, include SHA)
  • Unmethours link or helpdesk ticket number

Checklist

Add to this list or remove from it as applicable. This is a simple templated set of guidelines.

  • Defect file added (list location of defect file here)
  • Ticket added to Pivotal for defect (development team task)
  • Pull request created (the pull request will have additional tasks related to reviewing changes that fix this defect)
@jmarrec jmarrec added the Developer Issue Related to cmake, packaging, installers, or developer tooling (CI, etc) label Dec 22, 2023
@jmarrec
Copy link
Contributor Author

jmarrec commented Dec 22, 2023

Actualy SYSTEM doesn't cut it. But we can add this line here for eg:

set_target_properties(libkiva PROPERTIES FOLDER ThirdParty/Kiva)

target_compile_options(libkiva PRIVATE "-w")

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Developer Issue Related to cmake, packaging, installers, or developer tooling (CI, etc)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant