Fwd: Reminder: use KF5::foo instead of ${foo_LIBRARIES} in CMakeLists

Aurélien Gâteau agateau at kde.org
Thu Sep 26 12:38:55 UTC 2013


On Wednesday 25 September 2013 13:52:01 Nicolás Alvarez wrote:
> 2013/9/25 Aurélien Gâteau <agateau at kde.org>:
> > The alternative syntax: ${KConfigCore_LIBRARIES} would work in both cases,
> > but it is more error prone: any typo in the variable name causes the
> > variable to be expanded to an empty string, making it more difficult to
> > debug, whereas using target names provide more explicit error messages.
> 
> Have you tried using cmake --warn-uninitialized?

I didn't know --warn-uninitialized. It's quite handy, but as Aleix said it's 
way too verbose right now with kdelibs, plus it's just a warning, whereas a 
missing will stop the build with an error.

Aurélien


More information about the Kde-frameworks-devel mailing list