diff options
author | Eric Wong <e@80x24.org> | 2018-07-29 09:34:41 +0000 |
---|---|---|
committer | Eric Wong <e@80x24.org> | 2018-07-29 09:43:06 +0000 |
commit | 9015a8af2909b0071e54b332bfafc4e5b8d0f6c2 (patch) | |
tree | 7db2a5e86ef3cbf9cd18f948be20df52543c837c /t/data | |
parent | 31eda8c90633766692156c9c0af061dad6299c84 (diff) | |
download | public-inbox-9015a8af2909b0071e54b332bfafc4e5b8d0f6c2.tar.gz |
We must not clobber the original message string, as Email::MIME(*) still needs it for iterating through parts in SearchIdx (but not when handing it as a raw string to git-fast-import). I've noticed message bodies (especially dfpre/dpost) were not getting indexed when going through -mda (no problems with -watch). This also did not affect v1 repos, since indexing is a separate process for v1 and requires re-reading the data from git. (*) tested Email::MIME 1.937 on Debian stretch
Diffstat (limited to 't/data')
-rw-r--r-- | t/data/0001.patch | 46 |
1 files changed, 46 insertions, 0 deletions
diff --git a/t/data/0001.patch b/t/data/0001.patch new file mode 100644 index 00000000..b7964a2b --- /dev/null +++ b/t/data/0001.patch @@ -0,0 +1,46 @@ +From: Eric Wong <e@80x24.org> +Date: Fri, 20 Jul 2018 07:21:41 +0000 +To: test@example.com +Subject: [PATCH] search: use boolean prefix for filenames in diffs, too +Message-ID: <20180720072141.GA15957@example> + +Filenames within a project tend to be reasonably stable within a +project and I plan on having automated searches hit these. + +Also, using no term prefix at all (the default for searching) +still allows probabilistic searches on everything that's in a +"git diff", including the blob names which were just made +boolean. + +Note, attachment filenames ("n:" prefix) will stil use +probabilistic search, as they're hardly standardized. +--- + lib/PublicInbox/Search.pm | 6 +++--- + 1 file changed, 3 insertions(+), 3 deletions(-) + +diff --git a/lib/PublicInbox/Search.pm b/lib/PublicInbox/Search.pm +index 090d998b6c2c..6e006fd73b1d 100644 +--- a/lib/PublicInbox/Search.pm ++++ b/lib/PublicInbox/Search.pm +@@ -53,6 +53,9 @@ my %bool_pfx_external = ( + dfpre => 'XDFPRE', + dfpost => 'XDFPOST', + dfblob => 'XDFPRE XDFPOST', ++ dfn => 'XDFN', ++ dfa => 'XDFA', ++ dfb => 'XDFB', + ); + + my $non_quoted_body = 'XNQ XDFN XDFA XDFB XDFHH XDFCTX XDFPRE XDFPOST'; +@@ -72,9 +75,6 @@ my %prob_prefix = ( + + q => 'XQUOT', + nq => $non_quoted_body, +- dfn => 'XDFN', +- dfa => 'XDFA', +- dfb => 'XDFB', + dfhh => 'XDFHH', + dfctx => 'XDFCTX', + +-- +^_^ |