Opened 6 years ago
Closed 4 years ago
#1183 closed enhancement (fixed)
Make Logs Searchable
| Reported by: | marcel@… | Owned by: | pacopablo |
|---|---|---|---|
| Priority: | normal | Component: | IrcLogsPlugin |
| Severity: | normal | Keywords: | pyndexter search irc logs |
| Cc: | mitsuhiko | Trac Release: | 0.11 |
Description
integrated like the other things i guess
Attachments (1)
Change History (9)
comment:1 Changed 6 years ago by pacopablo
- Cc mitsuhiko added
- Keywords pyndexter search irc logs added
- Status changed from new to assigned
- Summary changed from make the logs searchable to Make Logs Searchable
comment:2 Changed 6 years ago by marcel@…
any news on this?
comment:3 Changed 6 years ago by pacopablo
OK, so this is done. Much thanks go to alect and his pyndexter stuff. However, I haven't actualyl committed or merged the code. It's working at http://pacopablo.com, but that's based ona slightly modified 0.10 version of the logs plugin. I hope to get it committed soon.
comment:4 Changed 6 years ago by marcel@…
Anxious for something to test. ;-)
comment:5 Changed 6 years ago by pacopablo
source:irclogsplugin/0.10 should have the initial import. Some paths are hard coded still, but it's something to test.
comment:6 Changed 5 years ago by thomas.moschny@…
Here is my attempt to port the search functionality to 0.11. Tested with Trac 0.11dev-r7092 and pyndexter 0.2. The patch is against source:irclogsplugin/0.11@2591.
comment:7 Changed 5 years ago by marcel@…
Excellent! Applied the patch and due to our setup of irc log the line
'file = hit.uri.path.lstrip(self.path)'
had to be replaced with
'file = os.path.basename(hit.uri.path)'
lstrip would not be correct here i think. After that change (and adjusting the indexer script a bit) it works.
comment:8 Changed 4 years ago by doki_pen
- Resolution set to fixed
- Status changed from assigned to closed
This is done in irclogsplugin/0.11
Already working on it ;)
I'm planning on using pyndexter for the searching. If you see this as a horrible impediment, let me know.