----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125570/#review94004 -----------------------------------------------------------
The reviewboard upgrade must have messed up something, the current patch is just whitespace. Please re-upload. - David Faure On Oct. 22, 2015, 1:30 p.m., R.Harish Navnit wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/125570/ > ----------------------------------------------------------- > > (Updated Oct. 22, 2015, 1:30 p.m.) > > > Review request for KDE Frameworks, KDEPIM, Aleix Pol Gonzalez, and Laurent > Montel. > > > 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 > ----- > > 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
