On Sunday 07 of October 2012 22:01:20 Darrell Anderson wrote: > > I have another suggestion for change. During the search in > > the profile is treated, they are not searched folders in kmail/mail. But > > the same is necessary for folders dimap, imap and also autosave. On the > > folder kmail/search I'm not sure. Proposed patch attached. > > I have no problem with excluding imap folders but I can't test because I > don't use imap. > > > Next, I wanted to ask whether we can assume the presence of > > xargs on the users machines? It can in the replacement lower overhead of > > repeated execution sed and thus increase the speed. For example: > > > > find $HOME/.trinity \ > > -path $HOME/.trinity/share/apps/kmail/autosave -prune -o \ > > -path $HOME/.trinity/share/apps/kmail/dimap -prune -o \ > > -path $HOME/.trinity/share/apps/kmail/imap -prune -o \ > > -path $HOME/.trinity/share/apps/kmail/mail -prune -o \ > > -path $HOME/.trinity/share/apps/kmail/search -prune -o \ > > -type f -print0 | \ > > xargs -r0 sed -i "s|/opt/kde/|${TDEDIR}/|g" > > > > Instead of launching many sed, how much will be files, can > > be run only one sed, which will process all files that are found. > > > > What is your opinion on this? > > If xargs on all systems is a safe presumption then I have no problem with > that either. I should make more use of xargs in my own habits. Just lazy. > :) > > If your testing of the imap and xarg changes work on your system, I say go > ahead and push the patch. I can test the xargs part here. > > Darrell > Apart from KMail imap folders I added several other folders that contain user data - see commit in GIT hash f88add0a. Change to use xargs I will send soon as a separate commit. Slavek --