> On April 12, 2014, 1:57 a.m., Kevin Krammer wrote: > > src/file/kcm/folderselectionwidget.cpp, line 198 > > <https://git.reviewboard.kde.org/r/117436/diff/1/?file=263434#file263434line198> > > > > maybe QFileInfo(QDir::homePath()).absolutPath() > >
Much neater and less kludgy, thanks. - Lindsay ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117436/#review55461 ----------------------------------------------------------- On April 12, 2014, 7:56 a.m., Lindsay Mathieson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/117436/ > ----------------------------------------------------------- > > (Updated April 12, 2014, 7:56 a.m.) > > > Review request for Baloo. > > > Repository: baloo > > > Description > ------- > > Part of the KCM code assumes the users home dir is under "/home", which is > not always the cause in linux - e.g OpenSUSE with AD Authentication uses a > /home/DOMAIN hierarchy. Other systems use /opt. Not to mention windows etc. > > The code adds a function homeRoot() which returns the parent directory of the > uses home directory. > > > Diffs > ----- > > src/file/kcm/folderselectionwidget.cpp bc87a33 > > Diff: https://git.reviewboard.kde.org/r/117436/diff/ > > > Testing > ------- > > Built, installed. Adding paths to the exclude lists still recognise the home > root. > > > Thanks, > > Lindsay Mathieson > >
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<
