>
> On Thu, Jun 25, 2015 at 6:33 PM, Pali Rohár <pali.ro...@gmail.com> wrote:
>>
>>
>> Just a quick note about your patches which removing KDE3/Qt3 code:
>>
>> Look also at CMakeLists.txt files and remove coresponding KDE3/Qt3
>> defines ideally with linking with KDE3/Qt3 libraries...
>>
> I'm not really sure how this is done. What I did was however, was to
remove the add_definitions(...) and remove $(KDE4_KDE3SUPPORT_LIBS) from
the target_link_libraries in the CMakeLists.txt file.


> It is good that we do not need to include and use KDE3/Qt3 support
>> headers, but if we are still linking with KDE3/Qt3 libraries it does not
>> help us.
>>
>> This just create hidden dependency on KDE3/Qt3 support libraries
>> because we do not see any KDE3/Qt3 includes in code, but linker still
>> needs real libraries...
>>
> Yes, you're right. I'm trying to compile it now after removing these
libraries and it's taking much much longer and is also more noisy(too many
warnings, probably errors too).


Warm Regards.

Thanking You,
R.Harish Navnit
The Enigma <http://harishnavnit.wordpress.com/>
_______________________________________________
kopete-devel mailing list
kopete-devel@kde.org
https://mail.kde.org/mailman/listinfo/kopete-devel

Reply via email to