patch for current version about text-sherlock HOT 6 CLOSED

right, here's another patch for the is_recursive option:

diff --git a/core/sherlock/indexer.py b/core/sherlock/indexer.py
index 4104234..07889c9 100644
--- a/core/sherlock/indexer.py
+++ b/core/sherlock/indexer.py
@@ -200,6 +200,8 @@ class Indexer(object):
                     if can_index:
                         path = os.path.join(dirpath, name)
                         self.__index_file(path)
+                if dirpath != dpath:
+                    self.__index_dir(dirpath) 
         pass

     def __index_file(self, filepath):

regards :D

from text-sherlock.

Thanks! I'll review the changes and get this merged.

from text-sherlock.

great!

i think text-sherlock is really great project. It certainly beats
installing tomcat :D however, i would like to extend this project a bit. is
that ok with you ? should i fork the project or sth?

i would like to add such features:

• when you click on the results, the display page should link to specific
  line (right now it jumps to the beggining of file)
• indexing directly from scm like git/svn/etc
• indexing more than one branch
• project's browser

also, currently when my query includes a slash (/) i don't find the
results. i don't know whether that is a bug of sherlock or xapian.

best regards,
toudi.

2012/11/14 C. Bess [email protected]

Thanks! I'll review the changes and get this merged.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/11#issuecomment-10368308.

pozdrawiam serdecznie,
Mateusz Mikołajczyk, a.k.a. toudi

from text-sherlock.

@toudi Thanks, yeh, I got fed up with Tomcat/OpenGrok setup and created text sherlock.

Line numbers was my next planned feature addition, so if you handled it that would be great. I'll make an issue for it.

As long as you make a unit test for the features and the other tests pass, I'll merge it.

I'm not sure about that '/' issue, separate issue?

Thanks for your contributions.

from text-sherlock.

@toudi Made an issue for line number: #13

I'm in the process of updating the db logic to support peewee 2.x. I tagged the current working package info.

from text-sherlock.

Fixed in 424cdd5. I found no recursive indexing issue.

from text-sherlock.