2009/5/15 Raphael Kubo da Costa <kub...@gmail.com>: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviewboard.kde.org/r/701/ > ----------------------------------------------------------- > > Review request for Kopete. > > > Summary > ------- > > The attached patch includes a kconf_update script that changes styleName=Gaim > to styleName=Pidgin in the Appearance section of kopeterc. > > I had never coded in Perl before, but I hope the script is small enough not > to have allowed me to screw things up -- besides, I don't know if the regular > expression will match if it's being run on Windows (I don't know if kopeterc > is separated by \r\n there). > > This is the first time I've written a kconf_update file too, so I don't know > if that's the best ID or those are the correct options. > > > This addresses bug 187192. > https://bugs.kde.org/show_bug.cgi?id=187192 > > > Diffs > ----- > > /trunk/KDE/kdenetwork/kopete/kopete/kconf_update/CMakeLists.txt 967791 > /trunk/KDE/kdenetwork/kopete/kopete/kconf_update/kopete-gaim_to_pidgin_style.pl > PRE-CREATION > /trunk/KDE/kdenetwork/kopete/kopete/kconf_update/kopete-gaim_to_pidgin_style.upd > PRE-CREATION > > Diff: http://reviewboard.kde.org/r/701/diff > > > Testing > ------- > > Running the script with a kopeterc with styleName=Gaim correctly renames the > entry. > > > Thanks, > > Raphael > >
Did anyone have time to look at this? This change is pretty harmless and solves a critical bug. I can commit it if that's OK. _______________________________________________ kopete-devel mailing list kopete-devel@kde.org https://mail.kde.org/mailman/listinfo/kopete-devel
