apol created this revision.
apol added a reviewer: Frameworks.

REVISION SUMMARY
  Fallback on trying the same filename as the actual file.
  There were some regressions due to the former matching expression:
  string(REGEX REPLACE "^(.*\n)?File=([^\n]+kcfg).*\n.*$" "\\2"  _kcfg_FILENAME 
"${_contents}")
  
  This one would allow for the File field to be the kcfgc file (which doesn't
  make sense).
  With this patch we're not reproducing this behaviour, but we're allowing for 
the
  things that aren't building to work. After all, most kcfg and kcfgc files 
share
  the same name.

BRANCH
  master

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

AFFECTED FILES
  KF5ConfigMacros.cmake

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: apol, #frameworks

Reply via email to