Review Request 127741: libkcddb port complete to KF5

[Aleix Pol Gonzalez]

> On April 25, 2016, 10:37 p.m., Albert Astals Cid wrote:
> > Why did you copy MacroLogFeature?
> 
> Angelo Scarna wrote:
>     Because it gave me error
> 
> Angelo Scarna wrote:
>     Why it was present in cmake modules for kde4

Use upstream cmake `include(FeatureSummary)`. See its documentation or any other ported KDE project to see how it's used.


- Aleix


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127741/#review94841
-----------------------------------------------------------


On April 25, 2016, 11:37 a.m., Angelo Scarna wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127741/
> -----------------------------------------------------------
> 
> (Updated April 25, 2016, 11:37 a.m.)
> 
> 
> Review request for KDE Multimedia and Harald Sitter.
> 
> 
> Repository: libkcddb
> 
> 
> Description
> -------
> 
> Port complete to KF5
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt a92ff28 
>   cmake/MacroLogFeature.cmake PRE-CREATION 
>   kcmcddb/CMakeLists.txt 6cd7822 
>   kcmcddb/cddbconfigwidget.h 1cc9a8b 
>   kcmcddb/cddbconfigwidget.cpp e7e3758 
>   kcmcddb/cddbconfigwidget.ui 16fc911 
>   kcmcddb/doc/CMakeLists.txt a565b0b 
>   kcmcddb/kcmcddb.h 5a9a712 
>   kcmcddb/kcmcddb.cpp 00c8b9f 
>   kcmcddb/libkcddb.desktop d9224bf 
>   libkcddb/CMakeLists.txt 8927d1a 
>   libkcddb/asynccddbplookup.h 0b2aa2a 
>   libkcddb/asynccddbplookup.cpp 8cc7d23 
>   libkcddb/asynchttplookup.h 6f3ee4a 
>   libkcddb/asynchttplookup.cpp bba91df 
>   libkcddb/asynchttpsubmit.h 9dac9d6 
>   libkcddb/asynchttpsubmit.cpp acc6205 
>   libkcddb/asyncsmtpsubmit.h 6eaeef8 
>   libkcddb/asyncsmtpsubmit.cpp 3ea9702 
>   libkcddb/cache.h 0f67c10 
>   libkcddb/cache.cpp 96c9560 
>   libkcddb/categories.h c2d28fa 
>   libkcddb/categories.cpp 222df47 
>   libkcddb/cddb.h 202f1d9 
>   libkcddb/cddb.cpp 9518f87 
>   libkcddb/cddbplookup.h 6c4244c 
>   libkcddb/cddbplookup.cpp 2f3c772 
>   libkcddb/cdinfo.h 6322f59 
>   libkcddb/cdinfo.cpp ee1cd39 
>   libkcddb/cdinfodialog.h b52773e 
>   libkcddb/cdinfodialog.cpp e7f7839 
>   libkcddb/cdinfodialog.ui 9ca3400 
>   libkcddb/cdinfoencodingwidget.h 9c30950 
>   libkcddb/cdinfoencodingwidget.cpp 97f6071 
>   libkcddb/cdinfoencodingwidget.ui 0f92cf7 
>   libkcddb/client.h 136cab7 
>   libkcddb/client.cpp 6ceb0f5 
>   libkcddb/genres.h d0fe2bb 
>   libkcddb/genres.cpp 3af2558 
>   libkcddb/httplookup.h c4861ac 
>   libkcddb/httplookup.cpp 9a43712 
>   libkcddb/httpsubmit.h a37f773 
>   libkcddb/httpsubmit.cpp 402c98d 
>   libkcddb/kcddb.h fca9fd8 
>   libkcddb/kcddb.cpp b90873f 
>   libkcddb/kcddb_export.h 6ac6d57 
>   libkcddb/kcddbconfig.h 7df5d4e 
>   libkcddb/kcddbconfig.cpp 27f2a34 
>   libkcddb/lookup.h 5b5556b 
>   libkcddb/lookup.cpp d3ec89a 
>   libkcddb/musicbrainz/asyncmusicbrainzlookup.h 373fb16 
>   libkcddb/musicbrainz/asyncmusicbrainzlookup.cpp c13c94f 
>   libkcddb/musicbrainz/musicbrainzlookup.h 5f0254f 
>   libkcddb/musicbrainz/musicbrainzlookup.cpp 3364897 
>   libkcddb/sites.h 3a4121b 
>   libkcddb/sites.cpp 65a2e48 
>   libkcddb/smtpsubmit.h 6a082b0 
>   libkcddb/smtpsubmit.cpp 3e74ce9 
>   libkcddb/submit.h e46fedb 
>   libkcddb/submit.cpp 99661cc 
>   libkcddb/synccddbplookup.h f13bcce 
>   libkcddb/synccddbplookup.cpp a4d9625 
>   libkcddb/synchttplookup.h 3c89078 
>   libkcddb/synchttplookup.cpp bfe79d2 
>   libkcddb/synchttpsubmit.h a88b289 
>   libkcddb/synchttpsubmit.cpp 7e13aa7 
>   libkcddb/syncsmtpsubmit.h b2ff98e 
>   libkcddb/syncsmtpsubmit.cpp 19cc941 
>   test/CMakeLists.txt e2e2aab 
>   test/asynccddblookuptest.h 5ce40fb 
>   test/asynccddblookuptest.cpp 03f79fe 
>   test/asynchttplookuptest.h bcddc58 
>   test/asynchttplookuptest.cpp 3377987 
>   test/asynchttpsubmittest.h 0f31f10 
>   test/asynchttpsubmittest.cpp 155ad98 
>   test/asyncmusicbrainztest.h b56fdc2 
>   test/asyncmusicbrainztest.cpp 4aea67e 
>   test/asyncsmtpsubmittest.h d053267 
>   test/asyncsmtpsubmittest.cpp 5ade7c9 
>   test/cachetest.h 6f8820d 
>   test/cachetest.cpp fd5bcb7 
>   test/cdinfotest.h 12d4749 
>   test/cdinfotest.cpp 69f98a8 
>   test/musicbrainztest-fulldate.h 8ab56c1 
>   test/musicbrainztest-fulldate.cpp d1c4ad4 
>   test/musicbrainztest-severaldiscs.h 87655f6 
>   test/musicbrainztest-severaldiscs.cpp e476a8b 
>   test/musicbrainztest.h df9d2b3 
>   test/musicbrainztest.cpp 95bef0b 
>   test/sitestest.cpp dd028fa 
>   test/synccddblookuptest.h 01a750f 
>   test/synccddblookuptest.cpp 6b6b28b 
>   test/synchttplookuptest.h 72e3ddc 
>   test/synchttplookuptest.cpp c09f324 
>   test/synchttpsubmittest.cpp a107b9d 
>   test/syncsmtpsubmittest.cpp c957c6d 
>   test/utf8test.h 631dafd 
>   test/utf8test.cpp 83f7b3a 
> 
> Diff: https://git.reviewboard.kde.org/r/127741/diff/
> 
> 
> Testing
> -------
> 
> It has been successfully tested . Now I propose another passage , to bring k3b to KF5
> 
> 
> Thanks,
> 
> Angelo Scarna
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-multimedia/attachments/20160425/3f435f06/attachment.htm>
-------------- next part --------------
_______________________________________________
kde-multimedia mailing list
kde-multimedia at kde.org
https://mail.kde.org/mailman/listinfo/kde-multimedia