After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 513486 - TrackerConfig implementation
TrackerConfig implementation
Status: RESOLVED FIXED
Product: tracker
Classification: Core
Component: General
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Jamie McCracken
Jamie McCracken
Depends on:
Blocks:
 
 
Reported: 2008-01-31 17:19 UTC by Martyn Russell
Modified: 2008-03-05 11:54 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Patch to fix the bug (175.61 KB, patch)
2008-01-31 17:22 UTC, Martyn Russell 		none Details | Review
updated patch (170.16 KB, patch)
2008-02-07 16:49 UTC, Carlos Garnacho 		committed Details | Review

Description Martyn Russell 2008-01-31 17:19:51 UTC 
Hi,

I have a patch which modularises the configuration and separates it from the main Tracker typedef struct.

This patch reworks some functions too which make use of glib calls which already do some of the work trackerd was doing.

Patch to follow...
Comment 1 Martyn Russell 2008-01-31 17:22:03 UTC 
Created attachment 104114 [details] [review]
Patch to fix the bug

I have spent a the last week testing this since I know it changes a lot of important code - it works well IMO.

This patch: 

- Adds tracker-config.[ch]
- Adds tracker-language.[ch]
- Removes tracker-stem.[ch] (puts them into tracker-language.[ch])
- Heavily modifies trackerd.c
- Modifies some other areas that use the tracker struct.
Comment 2 Carlos Garnacho 2008-02-07 16:49:28 UTC 
Created attachment 104650 [details] [review]
updated patch

This patch adds the missing ThreadStackSize option and uses the #define'd defaults when creating the configuration file from scratch, so there's only one instance of this data.
Comment 3 Martyn Russell 2008-03-05 11:54:38 UTC 
Committed, thanks for the patch Carlos update :)