On 7/29/06, Marcus Fritzsch <fritschy googlemail com> wrote:
Hi again, On 7/29/06, Jamie McCracken <jamiemcc blueyonder co uk> wrote: [...] > should do - feel free to apply. Ah yes - it does not do ;) There is this dir_only flag, which prevents my no_watch_dirs from being checked, as src/trackerd/trackerd.c:scan_directory calls src/trackerd/tracker-utils.c:tracker_get_files with that dir_only being FALSE. Also, doing !dir_only || (dir_only && ... is not that beautiful ;) But, what is dir_only for? It should be TRUE, else no_watch_dirs will not be checked... > > Currently they are quite happy indexed, so I ended up with this patch > > ;) > > yeah sorry about that - I have a number of outstanding patches to apply > (and convert to new cvs version) Arrgh - for every thing i posted I did a search to the mailing list (posts before my time being here) --- but did not found anything so far, well, this time did not searched for :) > Just been so busy this month hence the delay in applying them all. > > thanks anyway for your efforts :) Nah - nothing compared to tracker! _______________________________________________ tracker-list mailing list tracker-list gnome org http://mail.gnome.org/mailman/listinfo/tracker-list
Sorry about my email, stupid Gmails is being weird.
Hey there,
Yea, I realized the problem with dir_only after I sent in the patch,
but I've been a bit busy. The patch for current cvs would be this:
change line 1255 to:
if ((!dir_only || tracker_is_directory (mystr)) &&
g_slist_find_custom(no_watch_directory_list, mystr,
(GCompareFunc)strcmp) == NULL) {
I've attached another patch to CVS :-)
Cheers,
Samuel
Attachment:
tracker-no-watch.patch
Description: Text Data