-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112907/
-----------------------------------------------------------

Review request for KDE Frameworks.


Description
-------

Move KEmoticons framework to tier3

Done:
-Adjust the CMakeLists.txt to the new location.
-Substitute kde_add_plugin to add_library.
-Substitute Qt5::Xml to Qt5Xml in target_link_libraries
-Substitute Qt5::Test to Qt5Test in target_link_libraries

TODO:
Modify API to make it more coherent


Diffs
-----

  staging/CMakeLists.txt 5c52cbe 
  staging/kemoticons/CMakeLists.txt 0e5d935 
  staging/kemoticons/KEmoticonsConfig.cmake.in f394979 
  staging/kemoticons/autotests/CMakeLists.txt b7e890c 
  staging/kemoticons/autotests/emoticon-parser-testcases/broken-1.input 795d3c7 
  staging/kemoticons/autotests/emoticon-parser-testcases/broken-1.output 
795d3c7 
  staging/kemoticons/autotests/emoticon-parser-testcases/broken-10.input 
6ddd0c7 
  staging/kemoticons/autotests/emoticon-parser-testcases/broken-10.output 
6ddd0c7 
  staging/kemoticons/autotests/emoticon-parser-testcases/broken-2.input 2571b16 
  staging/kemoticons/autotests/emoticon-parser-testcases/broken-2.output 
2571b16 
  staging/kemoticons/autotests/emoticon-parser-testcases/broken-4.input f5d8887 
  staging/kemoticons/autotests/emoticon-parser-testcases/broken-4.output 
0d94eb9 
  staging/kemoticons/autotests/emoticon-parser-testcases/broken-5.input 5b39691 
  staging/kemoticons/autotests/emoticon-parser-testcases/broken-5.output 
5b39691 
  staging/kemoticons/autotests/emoticon-parser-testcases/broken-6.input 379e01a 
  staging/kemoticons/autotests/emoticon-parser-testcases/broken-6.output 
379e01a 
  staging/kemoticons/autotests/emoticon-parser-testcases/broken-8.input a373402 
  staging/kemoticons/autotests/emoticon-parser-testcases/broken-8.output 
a373402 
  staging/kemoticons/autotests/emoticon-parser-testcases/broken-9.input 538c5b0 
  staging/kemoticons/autotests/emoticon-parser-testcases/broken-9.output 
538c5b0 
  staging/kemoticons/autotests/emoticon-parser-testcases/working-1.input 
a5440d6 
  staging/kemoticons/autotests/emoticon-parser-testcases/working-1.output 
1f6d4aa 
  staging/kemoticons/autotests/emoticon-parser-testcases/working-2.input 
223ce5b 
  staging/kemoticons/autotests/emoticon-parser-testcases/working-2.output 
223ce5b 
  staging/kemoticons/autotests/emoticon-parser-testcases/working-3.input 
d685c09 
  staging/kemoticons/autotests/emoticon-parser-testcases/working-3.output 
e1b37c9 
  staging/kemoticons/autotests/emoticon-parser-testcases/working-4.input 
093690c 
  staging/kemoticons/autotests/emoticon-parser-testcases/working-4.output 
093690c 
  staging/kemoticons/autotests/emoticon-parser-testcases/working-5.input 
3994bf3 
  staging/kemoticons/autotests/emoticon-parser-testcases/working-5.output 
cba121a 
  staging/kemoticons/autotests/emoticon-parser-testcases/working-6.input 
2319ced 
  staging/kemoticons/autotests/emoticon-parser-testcases/working-6.output 
2319ced 
  staging/kemoticons/autotests/emoticon-parser-testcases/working-7.input 
d6e7e6c 
  staging/kemoticons/autotests/emoticon-parser-testcases/working-7.output 
d6e7e6c 
  staging/kemoticons/autotests/kemoticontest.h c79c193 
  staging/kemoticons/autotests/kemoticontest.cpp 083a804 
  staging/kemoticons/src/CMakeLists.txt 7b10087 
  staging/kemoticons/src/core/CMakeLists.txt f7fb463 
  staging/kemoticons/src/core/kemoticons.h 12b8308 
  staging/kemoticons/src/core/kemoticons.cpp 43dac65 
  staging/kemoticons/src/core/kemoticonsTheme.desktop 848ef2e 
  staging/kemoticons/src/core/kemoticonsprovider.h 85fc7ef 
  staging/kemoticons/src/core/kemoticonsprovider.cpp d04c76e 
  staging/kemoticons/src/core/kemoticonstheme.h b8b25f7 
  staging/kemoticons/src/core/kemoticonstheme.cpp ed3407c 
  staging/kemoticons/src/providers/CMakeLists.txt 445f721 
  staging/kemoticons/src/providers/adium/CMakeLists.txt c94c0be 
  staging/kemoticons/src/providers/adium/adium_emoticons.h 039a267 
  staging/kemoticons/src/providers/adium/adium_emoticons.cpp a3aaa0f 
  staging/kemoticons/src/providers/adium/emoticonstheme_adium.desktop 181f16a 
  staging/kemoticons/src/providers/kde/CMakeLists.txt e6d4243 
  staging/kemoticons/src/providers/kde/emoticonstheme_kde.desktop 144ee5c 
  staging/kemoticons/src/providers/kde/kde_emoticons.h 8cba6b1 
  staging/kemoticons/src/providers/kde/kde_emoticons.cpp 5b5114a 
  staging/kemoticons/src/providers/pidgin/CMakeLists.txt 44ac2dd 
  staging/kemoticons/src/providers/pidgin/emoticonstheme_pidgin.desktop e334d3f 
  staging/kemoticons/src/providers/pidgin/pidgin_emoticons.h 70bafa3 
  staging/kemoticons/src/providers/pidgin/pidgin_emoticons.cpp e9f89ee 
  staging/kemoticons/src/providers/xmpp/CMakeLists.txt f034de0 
  staging/kemoticons/src/providers/xmpp/emoticonstheme_xmpp.desktop a6927e4 
  staging/kemoticons/src/providers/xmpp/xmpp_emoticons.h 0873a63 
  staging/kemoticons/src/providers/xmpp/xmpp_emoticons.cpp 0dc92ed 
  staging/kemoticons/tests/CMakeLists.txt f7ec91f 
  staging/kemoticons/tests/main.cpp 563d760 
  tier3/CMakeLists.txt fb4de8f 
  tier3/kemoticons/CMakeLists.txt PRE-CREATION 
  tier3/kemoticons/KEmoticonsConfig.cmake.in PRE-CREATION 
  tier3/kemoticons/autotests/CMakeLists.txt PRE-CREATION 
  tier3/kemoticons/autotests/emoticon-parser-testcases/broken-1.input 
PRE-CREATION 
  tier3/kemoticons/autotests/emoticon-parser-testcases/broken-1.output 
PRE-CREATION 
  tier3/kemoticons/autotests/emoticon-parser-testcases/broken-10.input 
PRE-CREATION 
  tier3/kemoticons/autotests/emoticon-parser-testcases/broken-10.output 
PRE-CREATION 
  tier3/kemoticons/autotests/emoticon-parser-testcases/broken-2.input 
PRE-CREATION 
  tier3/kemoticons/autotests/emoticon-parser-testcases/broken-2.output 
PRE-CREATION 
  tier3/kemoticons/autotests/emoticon-parser-testcases/broken-4.input 
PRE-CREATION 
  tier3/kemoticons/autotests/emoticon-parser-testcases/broken-4.output 
PRE-CREATION 
  tier3/kemoticons/autotests/emoticon-parser-testcases/broken-5.input 
PRE-CREATION 
  tier3/kemoticons/autotests/emoticon-parser-testcases/broken-5.output 
PRE-CREATION 
  tier3/kemoticons/autotests/emoticon-parser-testcases/broken-6.input 
PRE-CREATION 
  tier3/kemoticons/autotests/emoticon-parser-testcases/broken-6.output 
PRE-CREATION 
  tier3/kemoticons/autotests/emoticon-parser-testcases/broken-8.input 
PRE-CREATION 
  tier3/kemoticons/autotests/emoticon-parser-testcases/broken-8.output 
PRE-CREATION 
  tier3/kemoticons/autotests/emoticon-parser-testcases/broken-9.input 
PRE-CREATION 
  tier3/kemoticons/autotests/emoticon-parser-testcases/broken-9.output 
PRE-CREATION 
  tier3/kemoticons/autotests/emoticon-parser-testcases/working-1.input 
PRE-CREATION 
  tier3/kemoticons/autotests/emoticon-parser-testcases/working-1.output 
PRE-CREATION 
  tier3/kemoticons/autotests/emoticon-parser-testcases/working-2.input 
PRE-CREATION 
  tier3/kemoticons/autotests/emoticon-parser-testcases/working-2.output 
PRE-CREATION 
  tier3/kemoticons/autotests/emoticon-parser-testcases/working-3.input 
PRE-CREATION 
  tier3/kemoticons/autotests/emoticon-parser-testcases/working-3.output 
PRE-CREATION 
  tier3/kemoticons/autotests/emoticon-parser-testcases/working-4.input 
PRE-CREATION 
  tier3/kemoticons/autotests/emoticon-parser-testcases/working-4.output 
PRE-CREATION 
  tier3/kemoticons/autotests/emoticon-parser-testcases/working-5.input 
PRE-CREATION 
  tier3/kemoticons/autotests/emoticon-parser-testcases/working-5.output 
PRE-CREATION 
  tier3/kemoticons/autotests/emoticon-parser-testcases/working-6.input 
PRE-CREATION 
  tier3/kemoticons/autotests/emoticon-parser-testcases/working-6.output 
PRE-CREATION 
  tier3/kemoticons/autotests/emoticon-parser-testcases/working-7.input 
PRE-CREATION 
  tier3/kemoticons/autotests/emoticon-parser-testcases/working-7.output 
PRE-CREATION 
  tier3/kemoticons/autotests/kemoticontest.h PRE-CREATION 
  tier3/kemoticons/autotests/kemoticontest.cpp PRE-CREATION 
  tier3/kemoticons/src/CMakeLists.txt PRE-CREATION 
  tier3/kemoticons/src/core/CMakeLists.txt PRE-CREATION 
  tier3/kemoticons/src/core/kemoticons.h PRE-CREATION 
  tier3/kemoticons/src/core/kemoticons.cpp PRE-CREATION 
  tier3/kemoticons/src/core/kemoticonsTheme.desktop PRE-CREATION 
  tier3/kemoticons/src/core/kemoticonsprovider.h PRE-CREATION 
  tier3/kemoticons/src/core/kemoticonsprovider.cpp PRE-CREATION 
  tier3/kemoticons/src/core/kemoticonstheme.h PRE-CREATION 
  tier3/kemoticons/src/core/kemoticonstheme.cpp PRE-CREATION 
  tier3/kemoticons/src/providers/CMakeLists.txt PRE-CREATION 
  tier3/kemoticons/src/providers/adium/CMakeLists.txt PRE-CREATION 
  tier3/kemoticons/src/providers/adium/adium_emoticons.h PRE-CREATION 
  tier3/kemoticons/src/providers/adium/adium_emoticons.cpp PRE-CREATION 
  tier3/kemoticons/src/providers/adium/emoticonstheme_adium.desktop 
PRE-CREATION 
  tier3/kemoticons/src/providers/kde/CMakeLists.txt PRE-CREATION 
  tier3/kemoticons/src/providers/kde/emoticonstheme_kde.desktop PRE-CREATION 
  tier3/kemoticons/src/providers/kde/kde_emoticons.h PRE-CREATION 
  tier3/kemoticons/src/providers/kde/kde_emoticons.cpp PRE-CREATION 
  tier3/kemoticons/src/providers/pidgin/CMakeLists.txt PRE-CREATION 
  tier3/kemoticons/src/providers/pidgin/emoticonstheme_pidgin.desktop 
PRE-CREATION 
  tier3/kemoticons/src/providers/pidgin/pidgin_emoticons.h PRE-CREATION 
  tier3/kemoticons/src/providers/pidgin/pidgin_emoticons.cpp PRE-CREATION 
  tier3/kemoticons/src/providers/xmpp/CMakeLists.txt PRE-CREATION 
  tier3/kemoticons/src/providers/xmpp/emoticonstheme_xmpp.desktop PRE-CREATION 
  tier3/kemoticons/src/providers/xmpp/xmpp_emoticons.h PRE-CREATION 
  tier3/kemoticons/src/providers/xmpp/xmpp_emoticons.cpp PRE-CREATION 
  tier3/kemoticons/tests/CMakeLists.txt PRE-CREATION 
  tier3/kemoticons/tests/main.cpp PRE-CREATION 

Diff: http://git.reviewboard.kde.org/r/112907/diff/


Testing
-------

It compiles, tests pass.


Thanks,

David Gil Oliva

_______________________________________________
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Reply via email to