mgallien accepted this revision.
mgallien added a comment.
This revision is now accepted and ready to land.


  Please feel free to land this after solving my comment.

INLINE COMMENTS

> CMakeLists.txt:57
> -set_package_properties(Exiv2 PROPERTIES DESCRIPTION "Image Tag reader"
> -                       URL "http://www.exiv2.org"; TYPE OPTIONAL
>                         PURPOSE "Support for image metadata")

Maybe, you may not remove the URL. That would be consistent with other 
dependencies.

REPOSITORY
  R286 KFileMetaData

BRANCH
  master

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

To: cgiboudeaux, mgallien
Cc: mgallien, kde-frameworks-devel, #baloo, ashaposhnikov, michaelh, astippich, 
spoorun, ngraham, bruns, abrahams

Reply via email to