Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214025
Summary: Review Request: Tracker - An object database, tag/metadata database, search tool and indexer Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: normal Priority: normal Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: dakingun@gmail.com QAContact: fedora-package-review@redhat.com
Spec URL: ftp://czar.eas.yorku.ca/pub/tracker/tracker.spec SRPM URL: ftp://czar.eas.yorku.ca/pub/tracker/tracker-0.5.0-1.src.rpm Description: Tracker is a powerful desktop-neutral first class object database, tag/metadata database, search tool and indexer.
Tracker is also extremely fast and super efficient with your systems memory when compared with some other competing frameworks and is by far the fastest and most memory efficient Nautilus search and Deskbar backends currently availble.
It consists of a common object database that allows entities to have an almost infinte number of properties, metadata (both embedded/harvested as well as user definable), a comprehensive database of keywords/tags and links to other entities.
It provides additional features for file based objects including context linking and audit trails for a file object.
It has the ability to index, store, harvest metadata. retrieve and search all types of files and other first class objects.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Tracker - An object database, tag/metadata database, search tool and indexer
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214025
michel.salim@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |michel.salim@gmail.com
------- Additional Comments From michel.salim@gmail.com 2006-11-04 17:17 EST ------- Where did you get tracker 0.5.0 from? Upstream has only 0.0.4, and the URL listed in the spec file does not work.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Tracker - An object database, tag/metadata database, search tool and indexer
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214025
michel.salim@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED AssignedTo|nobody@fedoraproject.org |michel.salim@gmail.com OtherBugsDependingO|163776 |163778 nThis| |
------- Additional Comments From michel.salim@gmail.com 2006-11-04 17:20 EST ------- Uh, nevermind, I typed in 0.0.5 by mistake. Jamie really should update the Tracker website as well.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Tracker - An object database, tag/metadata database, search tool and indexer
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214025
gauret@free.fr changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |gauret@free.fr
------- Additional Comments From gauret@free.fr 2006-11-04 17:58 EST ------- I had started to review it, but if you want to do it, here's what I found out so far:
Needs work: * Missing SMP flags. If it doesn't build with it, please add a comment (wiki: PackagingGuidelines#parallelmake) * Build failed in mock. Missing BR: pango-devel, dbus-devel, dbus-glib * Missing BR for metadata extractors : libgnomeui-devel, gnome-vfs2-devel, gnome-desktop-devel, poppler, libvorbis-devel, libpng-devel, libexif-devel, libgsf-devel
Possible improvements : * Have trackerd start on login with a desktop file in /usr/share/gnome/autostart/ (GNOME) and in /usr/share/autostart/ (KDE) Is there a freedesktop spec for this by the way ?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Tracker - An object database, tag/metadata database, search tool and indexer
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214025
------- Additional Comments From michel.salim@gmail.com 2006-11-04 18:14 EST ------- Created an attachment (id=140370) --> (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=140370&action=vie...) Diff to tracker.spec
Weird, the package I downloaded has libexif-devel, libgsf-devel etc. as BuildRequires. pango-devel is not required as it's pulled in by gnome-desktop-devel.
Missing BuildRequires: dbus-devel dbus-glib-devel gnome-utils-devel libpng-devel openssl-devel
Description has some misspelled words, and summary should not end in '.'; also, since libraries are included you need to call ldconfig on %post and %postun
Do we want to split the libraries into separate packages?
Actually, Aurelien, you can take over the review if you want. I'm less familiar with autostarting.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Tracker - An object database, tag/metadata database, search tool and indexer
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214025
------- Additional Comments From michel.salim@gmail.com 2006-11-04 18:15 EST ------- Created an attachment (id=140371) --> (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=140371&action=vie...) New tracker.spec
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Tracker - An object database, tag/metadata database, search tool and indexer
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214025
------- Additional Comments From dakingun@gmail.com 2006-11-04 18:25 EST ------- (In reply to comment #3)
I had started to review it, but if you want to do it, here's what I found out so far:
Needs work:
- Missing SMP flags. If it doesn't build with it, please add a comment (wiki: PackagingGuidelines#parallelmake)
- Build failed in mock. Missing BR: pango-devel, dbus-devel, dbus-glib
- Missing BR for metadata extractors : libgnomeui-devel, gnome-vfs2-devel,
gnome-desktop-devel, poppler, libvorbis-devel, libpng-devel, libexif-devel, libgsf-devel
Really me culpa. After I initially uploaded the src.rpm, I realized I had not tested it in mock, so I quickly updated the files (without bumping the version) thinking no one is reviewing yet. Updated files are here; Spec URL: ftp://czar.eas.yorku.ca/pub/tracker/tracker.spec SRPM URL: ftp://czar.eas.yorku.ca/pub/tracker/tracker-0.5.0-2.src.rpm
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Tracker - An object database, tag/metadata database, search tool and indexer
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214025
------- Additional Comments From dakingun@gmail.com 2006-11-04 19:04 EST ------- (In reply to comment #4)
Created an attachment (id=140370)
--> (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=140370&action=vie...) [edit]
Diff to tracker.spec
Weird, the package I downloaded has libexif-devel, libgsf-devel etc. as BuildRequires. pango-devel is not required as it's pulled in by gnome-desktop-devel.
Missing BuildRequires: dbus-devel dbus-glib-devel gnome-utils-devel libpng-devel openssl-devel
Most of these are already pulled in by other dependencies. But why is gnome-utils-devel and openssl-devel needed, I can't find anything in the configure log that that suggest they are
Description has some misspelled words, and summary should not end in '.'; also,
Gone thru and corrected a bunch (I had copied it directly from the README).
since libraries are included you need to call ldconfig on %post and %postun
Done.
Do we want to split the libraries into separate packages?
Actually, Aurelien, you can take over the review if you want. I'm less familiar with autostarting.
I've also commented the build fails with the smp flags. I'll work on the autostarting stuff tomorrow.
Thanks for reviewing.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Tracker - An object database, tag/metadata database, search tool and indexer
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214025
------- Additional Comments From michel.salim@gmail.com 2006-11-04 20:57 EST ------- I found them by calling ./configure --help and looking at the options that can be specified. It's probably not accurate, since it missed certain things that configure actually checks for. Will look at it again tomorrow.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Tracker - An object database, tag/metadata database, search tool and indexer
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214025
------- Additional Comments From gauret@free.fr 2006-11-05 10:01 EST ------- According to http://standards.freedesktop.org/autostart-spec/autostart-spec-latest.html the directory for autostarting applications is /etc/xdg/autostart. You have to put a desktop file in it.
On my computer, tracker starts up and indexes stuff. When I search using tracker-search, it gives some results, but sometimes it gives no results when I'm searching for something I know I have. Any idea why ?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Tracker - An object database, tag/metadata database, search tool and indexer
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214025
------- Additional Comments From rmy@pobox.com 2006-11-05 12:46 EST ------- mikmod obsoletes tracker. Has done since it was introduced in Red Hat 5.2.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Tracker - An object database, tag/metadata database, search tool and indexer
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214025
------- Additional Comments From gauret@free.fr 2006-11-05 13:02 EST ------- Good catch ! A bug should be filled against mikmod and the obsoletes removed IMHO.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Tracker - An object database, tag/metadata database, search tool and indexer
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214025
------- Additional Comments From gauret@free.fr 2006-11-05 13:43 EST ------- OK, I found this mail : http://mail.gnome.org/archives/tracker-list/2006-November/msg00001.html which explains my bug. FC-5 sqlite is not compiled with --enable-threadsafe, while FC-6 sqlite is.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Tracker - An object database, tag/metadata database, search tool and indexer
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214025
------- Additional Comments From mclasen@redhat.com 2006-11-05 13:47 EST ------- i'd leave the advertisement (the complete second paragraph) out of the deskcription.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Tracker - An object database, tag/metadata database, search tool and indexer
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214025
dakingun@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- BugsThisDependsOn| |214112
------- Additional Comments From dakingun@gmail.com 2006-11-05 15:41 EST ------- I've included an autostart desktop file and took care of comment #13; new src rpm are at: Spec URL: ftp://czar.eas.yorku.ca/pub/tracker/tracker.spec SRPM URL: ftp://czar.eas.yorku.ca/pub/tracker/tracker-0.5.0-4.src.rpm
PS: Also filled a bug against mikmod obsoleting tracker and making this depend on it.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Tracker - An object database, tag/metadata database, search tool and indexer
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214025
------- Additional Comments From gauret@free.fr 2006-11-05 18:27 EST ------- There is no gnome-utils-devel on FC5, could you replace its requirement with something like : %if "%fedora" >= "6" BuildRequires: gnome-utils-devel %else BuildRequires: dbus-devel, dbus-glib %endif
On the other hand, I'm not sure gnome-utils is of any use for building tracker. It only seem to appears in src/tracker-search-tool/tracker-search-tool.c
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Tracker - An object database, tag/metadata database, search tool and indexer
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214025
------- Additional Comments From dakingun@gmail.com 2006-11-05 20:00 EST ------- (In reply to comment #15)
There is no gnome-utils-devel on FC5, could you replace its requirement with
That's a mistake actually, I didn't intend to have BR on gnome-utils-devel and openssl-devel as I don't think they are needed. Corrected with these, Spec URL: ftp://czar.eas.yorku.ca/pub/tracker/tracker.spec SRPM URL: ftp://czar.eas.yorku.ca/pub/tracker/tracker-0.5.0-5.src.rpm
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Tracker - An object database, tag/metadata database, search tool and indexer
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214025
------- Additional Comments From gauret@free.fr 2006-11-06 03:57 EST ------- You need to BuildRequire dbus-devel and dbus-glib. They are checked by ./configure, and not pulled by other packages on FC-5
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Tracker - An object database, tag/metadata database, search tool and indexer
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214025
------- Additional Comments From dakingun@gmail.com 2006-11-06 11:18 EST ------- Spec URL: ftp://czar.eas.yorku.ca/pub/tracker/tracker.spec SRPM URL: ftp://czar.eas.yorku.ca/pub/tracker/tracker-0.5.0-6.src.rpm
With build requires for dbus-devel and dbus-glib
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Tracker - An object database, tag/metadata database, search tool and indexer
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214025
michel.salim@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|michel.salim@gmail.com |gauret@free.fr
------- Additional Comments From michel.salim@gmail.com 2006-11-06 13:30 EST ------- I added the gnome-utils-devel and openssl-devel BRs, since for some reason they appeared as options in the configure script. Deji, could you clarify this with upstream? If they don't actually do anything they should be trimmed out of the configure script.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Tracker - An object database, tag/metadata database, search tool and indexer
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214025
------- Additional Comments From gauret@free.fr 2006-11-06 14:48 EST ------- Needs work: * Make sure the lines in the %description field are not more than 76 chars long * Applications/System Tool is not a standard group * Remove the dot at the end of the summary for the -devel subpackage * As tracker-devel ships a pkgconfig file (.pc), is should have "Requires: pkgconfig"
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Tracker - An object database, tag/metadata database, search tool and indexer
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214025
------- Additional Comments From dakingun@gmail.com 2006-11-06 18:42 EST ------- No quick response yet from upstream on Michel concern about gnome-utils-devel and openssl-devel BRs; the needs work have all been addressed. I have used Applications/System for the RPM group as I think that's the closest. Spec URL: ftp://czar.eas.yorku.ca/pub/tracker/tracker.spec SRPM URL: ftp://czar.eas.yorku.ca/pub/tracker/tracker-0.5.0-7.src.rpm
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Tracker - An object database, tag/metadata database, search tool and indexer
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214025
------- Additional Comments From dakingun@gmail.com 2006-11-06 21:39 EST ------- Upstream have responded, and that with a new release. GNOME_UTILS that appears in the configure script is actually meant to be GNOME utilities (specifically libgnomeui and gtk+) as opposed to the package gnome-utils. Also they've statically linked in a threadsafe sqlite which builds by default, but also provided a configure option --enable-external-sqlite. This option is used for FC >=6. New spec and src rpms uploaded. Spec URL: ftp://czar.eas.yorku.ca/pub/tracker/tracker.spec SRPM URL: ftp://czar.eas.yorku.ca/pub/tracker/tracker-0.5.1-1.src.rpm
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Tracker - An object database, tag/metadata database, search tool and indexer
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214025
gauret@free.fr changed:
What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO|163778 |163779 nThis| |
------- Additional Comments From gauret@free.fr 2006-11-07 09:00 EST ------- * RPM name is OK * Source tracker-0.5.1.tar.gz is the same as upstream * Builds fine in mock * rpmlints look OK * File lists look OK * Works fine
Notes: * Please don't use tabs in the headers for -devel (Requires and Summary). Not a blocker of course.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Tracker - An object database, tag/metadata database, search tool and indexer
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214025
Bug 214025 depends on bug 214112, which changed state.
Bug 214112 Summary: Remove obsoletes on tracker https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214112
What |Old Value |New Value ---------------------------------------------------------------------------- Resolution| |RAWHIDE Status|NEW |CLOSED
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Tracker - An object database, tag/metadata database, search tool and indexer
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214025
------- Additional Comments From dakingun@gmail.com 2006-11-09 10:47 EST ------- Thanks for the reviews. Now that the mikmod issue had been resolved (presently only on rawhide), I'll go ahead and import it into Extras and make the correction of not using tabs there.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Tracker - An object database, tag/metadata database, search tool and indexer
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214025
dakingun@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Tracker - An object database, tag/metadata database, search tool and indexer
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214025
Bug 214025 depends on bug 214112, which changed state.
Bug 214112 Summary: Remove obsoletes on tracker https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214112
What |Old Value |New Value ---------------------------------------------------------------------------- Resolution|RAWHIDE | Status|CLOSED |NEW
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Tracker - An object database, tag/metadata database, search tool and indexer
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214025
Bug 214025 depends on bug 214112, which changed state.
Bug 214112 Summary: Remove obsoletes on tracker https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=214112
What |Old Value |New Value ---------------------------------------------------------------------------- Resolution| |NOTABUG Status|NEW |CLOSED
package-review@lists.fedoraproject.org