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

(Updated Oct. 12, 2015, 12:24 a.m.)


Review request for KDE Frameworks and KDEPIM.


Changes
-------

Add case insensitive comparisons and keep references to removed classes.


Repository: kde-dev-scripts


Description
-------

A lot of KABC classes are deprecated in KF5. The porting script only replaces 
kabc occurences to kcontacts, but overlooks if the class being used still 
exists in kcontact or not. This leads to a lot of compile time #include<> 
errors. I feel this should and can be avoided by modifying the script. Was this 
overlooked on purpose or did we just miss it out ?

I've really no prior knowledge or experience of working with perl and I've 
tried to handle the above mentioned scenario, by manually checking for 
deprecated classnames and replacing them with a blank space or commenting them 
out.


Diffs (updated)
-----

  kf5/convert-kabc-to-kcontacts.pl 321fdf2c92745a3aece8e66085994458720f9dfd 

Diff: https://git.reviewboard.kde.org/r/125570/diff/


Testing
-------

Ran the script on the kopete codebase which uses deprecated classes like 
addressbook, stdaddressbook, resource etc. 
Works fine, but takes a bit of time to execute.


Thanks,

R.Harish  Navnit

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

Reply via email to