bcooksley requested changes to this revision.
bcooksley added a comment.

  Christophe is correct here, it is worth warning developers about these issues 
regardless of the platform, so they can get the code ready for those platforms 
and test everything in their local environment as much as possible.
  I know for certain that there are developers who rely on our CI system and 
the Binary Factory to test and validate their applications (because they 
themselves do not have access to a development environment on those platforms).
  
  If this warning were to occur only on the platform(s) which it impacts then 
it would become much harder for people to fix and debug.
  
  Of course, if you aren't targeting those platforms, you can just do the 
necessary if() to not hit this path.

REPOSITORY
  R240 Extra CMake Modules

REVISION DETAIL
  https://phabricator.kde.org/D26752

To: patrickelectric, apol, tcanabrava, cgiboudeaux, bcooksley
Cc: bcooksley, patrickelectric, apol, cgiboudeaux, kde-frameworks-devel, 
kde-buildsystem, LeGast00n, GB_2, bencreasy, michaelh, ngraham, bruns

Reply via email to