<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jan 30, 2015 at 5:58 PM, René J.V. <span dir="ltr"><<a href="mailto:rjvbertin@gmail.com" target="_blank">rjvbertin@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Friday January 30 2015 16:34:51 Jeremy Whiting wrote:<br>
<br>
Hi Jeremy,<br>
<span class=""><br>
> Ok, I took a deep look inside kbuildsycoca5 today. I ran Instruments on it<br>
> and found as Marko already pointed out most of the time spent in there is<br>
> delving deep into /Applications subfolders. I'm not sure exactly what it's<br>
> looking for, mostly .desktop files from what I saw, but it's not finding<br>
<br>
</span>No leaves? ;)<br>
<br>
Have you tried to figure out *why* it's looking in there? Is that a regression introduced by our QSP patch, or is it because we keep standard locations where it shouldn't be searching?<br>
Do we know why this isn't an issue with kbuildsycoca4?<br></blockquote><div><br></div><div>I believe it's because kbuildsycoca4 looks using kstandarddirs, not QSP, QSP ApplicationsPath is /Applications on osx, while it's /usr/share/applications, /usr/local/share/applications and ~/.local/share/applications on linux. Maybe we should make it look in $PREFIX/share/applications since .desktop files aren't installed into /Applications anyway. macports is putting those into /opt/local/share/applications, kdesrc-build with /usr/local as prefix is doing likewise into that prefix. I.e. having kbuildsycoca5 look in QSP ApplicationsPath on osx is ridiculous, it will never contain .desktop files. If we do that the check for .isBundle() isn't as important since macports/homebrew/fink and kdesrc-build aren't putting .app files into $PREFIX/share/applications anyway... So what's a good way to get $PREFIX/share/applications on osx, QSP isn't it :/</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
> any. Traversing the whole tree is time consuming though, and we know it's<br>
> not likely to find .desktop files inside any .app bundles, so I added the<br>
> attached patch and that sped up both kbuildsycoca5 itself (running from the<br>
> terminal manually) as well as makes kded5 responsive from what I can tell.<br>
<br>
</span>Sped up how much?<br>
Regardless of where it looks, if the time spent searching remains an issue we could consider tapping into SpotLight to find the files.<br></blockquote><div><br></div><div>Using spotlight isn't a bad idea tbh, but it's orders of magnitude faster to skip bundles. KBuildSycoca5 took minutes before this change, seconds after this change (but probably isn't very useful since it's not actually finding the .desktop files. </div><div><br></div><div>thanks,</div><div>Jeremy</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
R.<br>
_______________________________________________<br>
<a href="mailto:kde-mac@kde.org">kde-mac@kde.org</a><br>
List Information: <a href="https://mail.kde.org/mailman/listinfo/kde-mac
KDE/Mac" target="_blank">https://mail.kde.org/mailman/listinfo/kde-mac<br>
KDE/Mac</a> Information: <a href="http://community.kde.org/Mac" target="_blank">http://community.kde.org/Mac</a><br>
</div></div></blockquote></div><br></div></div>