Don't cache ENABLE_ variables
ClosedPublic

Authored by davidedmundson on Jun 22 2017, 12:49 PM.

Details

Summary

QtCurve has a specialist detection of whether a variable is explicitly
set or not. If unset it probes for packages quietly. If explicitly set
to ON it will try to load the packages and error if they're not
existing.

By using option we were always caching the variable meaning compiling
twice would treat it like we ENABLE_QT5 was set.

Test Plan

Ran on a machine without Qt5 twice, it continued gracefully saying Qt5 wasn't found
Set -DENABLE_QT5=ON it errored at the specific point.

Diff Detail

Repository
R626 QtCurve
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
This revision is now accepted and ready to land.Jun 22 2017, 12:51 PM
This revision was automatically updated to reflect the committed changes.