<div dir="ltr"><div class="" id="post-body-6407006783170581936" itemprop="description articleBody">
<div dir="ltr" style="text-align:left">
The second week of GSoC coding period has come to an end. I didn't want to create another thread for my weekly updates hence I have emailed in this thread itself.<br>
<br>
So this week some of the changes that I made are as follows:<br>
<ul style="text-align:left"><li>Changed KIcon to QIcon. If anyone is using the porting script in 
kde-dev-scripts for this, then be careful that the script doesn't affect
 the return types of the functions which previously returned an instance
 of KIcon. This means that if the return type of a function is KIcon 
then you have to change it to QIcon by yourself. Not a major inconvenience, if you ask me.</li><li>Added include directory of phonon (I added PHONON_INCLUDES instead 
of PHONON_INCLUDE_DIR) to the cmake path in which header files are 
searched. Adding this wasn't important before but now with Phonon4Qt5 it
 is important. More information can be found <a href="http://api.kde.org/kdesupport-api/phonon-apidocs/phonon/html/phonon4qt5.html" target="_blank">here</a>.</li><li>I will be moving Amarok to KF5::Plasma after it has been ported to 
use the other KF5 components because I think that porting the rest of 
the code demands more attention.</li><li>I have marked some code with "#TODO KF5" so that the work there wont be forgotten and especially to mark temporary solutions.</li></ul>
<br>
Though KDialog is in KDELibs4Support but the classes that previously 
inherited from this have changed their base class. As the usage of 
functions(ex setButtons of KDialog) from these inherited classes cause errors now so I
 am currently porting away from KDialog. I have to review (many)changes 
made by the porting script for which I have to understand their API 
first. So I believe this is going to take some time.<br>
<br>
I will be pushing the commits that concern the above changes soon. <br>
<br>
Cheers!!<br>
<br>
P.S Happy Birthday Mamarok!</div>

</div>

<span class=""></span></div><div class="gmail_extra"><br><div class="gmail_quote">On 1 June 2015 at 20:10, Aroonav Mishra <span dir="ltr"><<a href="mailto:aroonav11@gmail.com" target="_blank">aroonav11@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hello all,<br><br>The major changes that I have introduced in the Amarok code-base till now are <br>as follows:<br><br>* I have changed the CMake files and I have added ECM (with a minimum version<br>of 1.7.0) in it. The KF5 libraries have been remodelled into a set of<br>independent modules enabling the developers to use only the specific parts<br>which they need and avoid pulling unwanted dependencies. For this reason, for<br>now I have added a few key components of KF5 and Qt5 for the cmake to find.<br>I will be adding more components as and when required. I have added the list<br>of the new dependencies in the amarok porting page().<br><br>* A major development included a change in MySQLAmarok.cmake module. As<br>feature_summary has been used with FATAL_ON_MISSING_REQUIRED_PACKAGES,<br>MySQLAmarok was turning out to be missing as MYSQLAMAROK_FOUND wasn't yet<br>defined. So I have defined MYSQLAMAROK_FOUND as TRUE when both mysql and<br>its embedded libraries have been found. More details on this have been given<br>in the commit and in the cmake module itself.<br><br>* The porting scripts in kde-dev-scripts have been a lot of help to me<br>(kudos to Laurent Montel, Kevin Funk, David Faure and all other contributors).<br>I have used them and they really eased the conversion of KUrl to QUrl and in<br>many many more changes (like the next one).<br><br>* The icons have been renamed according to the new pattern and the<br>ecm_install_icons calls have been changed similarly which previously created<br>a lot of warnings.<br><br>* CMake version was increased to 2.8.12 as it's the minimum required to use<br>FindKF5.cmake.<br><br>* To port to CMAKE_AUTOMOC, the moc include files were no longer needed(except<br>a few places) and they have been removed as they too created a lot of warnings.<br><br>* QT_NO_URL_CAST_FROM_STRING has been added to make the changes from QUrl from<br>QString explicit.<br><br>* I have also added the definitions to disable depreciation warnings<br><br>One of the major problems that I have faced this week was in porting to<br>KF5::ThreadWeaver. The documentation of the new API was surprisingly lacking in<br>very important details. I faced problems in usage of the QObjectDecorator class<br>where some signals have been moved into. I have thought of two solutions for<br>this which lets the files compile and which I am planning to test in future.<br>For this I spent some time on learning about the fine details of the working<br>of signals and slots systems.<br><br>It has been a very nice first week and I have learnt the details of quite a few<br>things like the above said signals and slots system and I also spent some time<br>to learn about the cmake's AUTOMOC system too.<br><br>Cheers<span class="HOEnZb"><font color="#888888"><br>Aroonav<br></font></span></div>
</blockquote></div><br></div>