FS#42341 - [catfish] Annoying message regarding locate.db

Attached to Project: Community Packages
Opened by Pantelis Panayiotou (plp) - Sunday, 12 October 2014, 11:27 GMT
Last edited by Alexander F. Rødseth (xyproto) - Wednesday, 22 October 2014, 13:29 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To Alexander F. Rødseth (xyproto)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 2
Private No

Details

Description: Annoying message regarding locate.db


Additional info:
* package version(s): catfish-1.2.2-1


Steps to reproduce:

The main window complains that the locate database file is more than 7 days old, even when the database is properly updated.

The issue has been reported here: https://bbs.archlinux.org/viewtopic.php?id=187953

I looked into the source, and found out that there are 2 causes for this bug:

1. Function check_locate() in catfish/CatfishWindow.py tries to guess the location of locate.db, and fails. More specifically, in Arch Linux this file is usually /var/lib/mlocate/mlocate.db, while catfish wrongly deduces that it's located at /var/lib/locate/locate.db.

2. Users in Arch don't have permission to read /var/lib/mlocate/mlocate.db normally. This makes Catfish believe that the file has never been updated, even when it has.

I've made a small patch to fix this, which I'm submitting together with the modified PKGBUILD.

The patch works by allowing the packager specify the location of the locate database file, by setting __locate_db_path__ in catfish_lib/catfishconfig.py. It also fixes the permissions problem.

It appears to work correctly on my system.
This task depends upon

Closed by  Alexander F. Rødseth (xyproto)
Wednesday, 22 October 2014, 13:29 GMT
Reason for closing:  Fixed
Comment by Robert Orzanna (orschiro) - Monday, 13 October 2014, 22:04 GMT
I can confirm that the patch resolves the issue and allows for proper updating of the search index database.
Comment by Alexander F. Rødseth (xyproto) - Wednesday, 22 October 2014, 07:27 GMT
Thanks for reporting and thanks for the patch.

I will update the package to include the patch. Please also report this upstream to the catfish developers (if it has not already been reported) so that they can fix it.

Loading...