catfish always uses 'locate' to search

Bug #1119718 reported by Florian Diesch
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Catfish
Invalid
Undecided
Unassigned
catfish (Ubuntu)
Invalid
Undecided
Unassigned

Bug Description

catfish always uses "locate" as search method even if you call it with a '--method'' command line option, like
   catfish --method find

This is caused by the function 'catfish.on_entry_find_text_activate()' calling 'self.find()' without any option in line 1259 of /usr/share/catfish/catfish.py

In line 537 - 551 'catfish.__init__()' contains some code to find the best search method but the result is never used.

ProblemType: Bug
DistroRelease: Ubuntu 12.10
Package: catfish 0.4.0.2-0ubuntu1
ProcVersionSignature: Ubuntu 3.5.0-23.35-generic 3.5.7.2
Uname: Linux 3.5.0-23-generic i686
NonfreeKernelModules: nvidia
ApportVersion: 2.6.1-0ubuntu10
Architecture: i386
Date: Fri Feb 8 20:56:17 2013
InstallationDate: Installed on 2012-09-25 (136 days ago)
InstallationMedia: Ubuntu 12.04 LTS "Precise Pangolin" - Release i386 (20120423)
MarkForUpload: True
PackageArchitecture: all
SourcePackage: catfish
UpgradeStatus: Upgraded to quantal on 2012-10-19 (112 days ago)

Revision history for this message
Florian Diesch (diesch) wrote :
Florian Diesch (diesch)
summary: - catfish always users 'locate' to search
+ catfish always uses 'locate' to search
Revision history for this message
Daniel Letzeisen (dtl131) wrote :

Thanks for reporting, but catfish no longer uses the --method option and depends explicitly on locate. To move forward with this bug report, please test the latest version of Catfish on Ubuntu Saucy/13.10 and verify it terminates the locate command properly when doing a long search.

Changed in catfish (Ubuntu):
status: New → Incomplete
Revision history for this message
Launchpad Janitor (janitor) wrote :

[Expired for catfish (Ubuntu) because there has been no activity for 60 days.]

Changed in catfish (Ubuntu):
status: Incomplete → Expired
Sean Davis (bluesabre)
Changed in catfish (Ubuntu):
status: Expired → Confirmed
Changed in catfish-search:
status: New → Confirmed
status: Confirmed → Invalid
Changed in catfish (Ubuntu):
status: Confirmed → Invalid
Revision history for this message
apport hater (g112) wrote :

Actually that's not the case.
Catfish does allow to be installed without locate
and is able to use os.walk (formely find) without any trouble.

Any option to choose preferred backend is thus a good idea to have.

Revision history for this message
apport hater (g112) wrote :
apport hater (g112)
Changed in catfish-search:
status: Invalid → Confirmed
Changed in catfish (Ubuntu):
status: Invalid → Confirmed
Revision history for this message
Sean Davis (bluesabre) wrote :

Catfish will no longer fail when locate cannot be found. Resolved with https://bugs.launchpad.net/ubuntu/+source/catfish/+bug/1482919

There are plans to manually choose search methods from the interface. I'll be incorporated that during the 1.3.x cycle. I'll be releasing 1.3.1 by Wednesday with the above fix.

Revision history for this message
Sean Davis (bluesabre) wrote :

Marking as Invalid since the --method feature is no longer available.

Changed in catfish-search:
status: Confirmed → Invalid
Changed in catfish (Ubuntu):
status: Confirmed → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.