cgiboudeaux added a comment.

  -1. They don't match the ECM coding style and code quality (doc, license, 
endif(), pkgconfig...)
  
  And :
  
  > kdelibs4support/cmake/modules/FindGLIB2.cmake
  >  ecm/attic/modules/FindGLIB2.cmake
  
  These two are there for legacy purpose.
  
  > phonon/cmake/FindGLIB2.cmake
  >  polkit-qt/cmake/modules/FindGLIB2.cmake
  
  these modules don't use ECM to build.
  
  > kdelibs4support/cmake/modules/FindPulseAudio.cmake
  >  ecm/attic/modules/FindPulseAudio.cmake
  >  phonon/cmake/FindPulseAudio.cmake
  
  same thing here.

REPOSITORY
  R240 Extra CMake Modules

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

To: marten, #frameworks, #build_system
Cc: cgiboudeaux, cfeck, heikobecker

Reply via email to