----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119662/#review64107 -----------------------------------------------------------
src/file/main.cpp <https://git.reviewboard.kde.org/r/119662/#comment44750> Plus, we're actually creating the directory in Database::setPath as well. Perhaps you just want to move the code to the init code of Database? - Vishesh Handa On Aug. 8, 2014, 11:49 a.m., Rohan Garg wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/119662/ > ----------------------------------------------------------- > > (Updated Aug. 8, 2014, 11:49 a.m.) > > > Review request for Baloo. > > > Repository: baloo > > > Description > ------- > > Testing from the Neon 5 ISO reveals that baloo_file gives a permission error > when the database folder does not exist. > > > Diffs > ----- > > src/file/main.cpp 3b0c240 > > Diff: https://git.reviewboard.kde.org/r/119662/diff/ > > > Testing > ------- > > Without patch baloo_file aborts with : > /home/USER/.project-neon5-kde//local/share/baloo/file/ does not have write > permissions. Aborting > > With patch : baloo_file starts fine > > > Thanks, > > Rohan Garg > >
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<
