Project

General

Profile

Idea #22175

Optionally? use SYSTEM option for include_directories in FindUpsPackage.cmake

Added by Eric Flumerfelt 6 months ago. Updated 6 months ago.

Status:
Feedback
Priority:
Normal
Assignee:
-
Target version:
-
Start date:
03/20/2019
Due date:
% Done:

0%

Estimated time:
Duration:

Description

In many cases, once a product is being included from a UPS package, the users of cetbuildtools no longer care about warnings from header files in that product. Therefore, it may be desirable for FindUpsPackage.cmake to include the SYSTEM option in its include_directories call. This will cause the compiler to ignore warnings (except #warning) and clang-tidy to skip checks on header files that the user most likely can't modify.

History

#1 Updated by Kyle Knoepfel 6 months ago

  • Status changed from New to Feedback

We would prefer not to go this route--our experience has been that doing what you propose hides problems that bite us later. We acknowledge that the emitted warnings can be distracting. We're happy to talk about this further if you like.



Also available in: Atom PDF