Excerpts from Rich Lane's message of Sun Aug 30 15:28:55 -0500 2009:
> ---
>  lib/sup/xapian_index.rb |    4 +++-
>  1 files changed, 3 insertions(+), 1 deletions(-)
> 
> diff --git a/lib/sup/xapian_index.rb b/lib/sup/xapian_index.rb
> index c260728..1395601 100644
> --- a/lib/sup/xapian_index.rb
> +++ b/lib/sup/xapian_index.rb
> @@ -362,7 +362,9 @@ class XapianIndex < BaseIndex
>    end
>  
>    def find_docid id
> -    term_docids(mkterm(:msgid,id)).tap { |x| fail unless x.size <= 1 }.first
> +    docids = term_docids(mkterm(:msgid,id))
> +    fail unless docids.size <= 1
> +    docids.first
>    end
>  
>    def find_doc id

After applying this on next in hopes it lets me run sup on ruby 1.8.6, it
doesn't work.

It crashed with this log:
$ SUP_INDEX=xapian ruby -Ilib bin/sup-sync --all --all-sources --restore 
~/dumpfile
Loading state dump from /Users/blake/dumpfile...
Read 6790 entries from dump file.
Scanning maildir:/Users/blake/Mail/bburky/INBOX...
[Sun Aug 30 15:38:57 -0500 2009] hook: read 'before-add-message' from 
/Users/blake/.sup/hooks/before-add-message.rb
[Sun Aug 30 15:38:57 -0500 2009] hook[before-add-message]: Marking message 
1234774373-sup-1...@zo as suptalk, subject is 'Re: [sup-talk] (no subject)'
./lib/sup/label.rb:64:in `<<': expecting a symbol (ArgumentError)
        from ./lib/sup/util.rb:520:in `send'
        from ./lib/sup/util.rb:520:in `method_missing'
        from ./lib/sup/xapian_index.rb:114:in `sync_message'
        from 
/System/Library/Frameworks/Ruby.framework/Versions/1.8/usr/lib/ruby/1.8/set.rb:189:in
 `each'
        from 
/System/Library/Frameworks/Ruby.framework/Versions/1.8/usr/lib/ruby/1.8/set.rb:189:in
 `each_key'
        from 
/System/Library/Frameworks/Ruby.framework/Versions/1.8/usr/lib/ruby/1.8/set.rb:189:in
 `each'
        from ./lib/sup/xapian_index.rb:114:in `sync_message'
        from ./lib/sup/xapian_index.rb:87:in `add_message'
         ... 10 levels...
        from ./lib/sup/util.rb:520:in `method_missing'
        from bin/sup-sync:146
        from bin/sup-sync:141:in `each'
        from bin/sup-sync:141

However, I'm having other odd problems too with labels anyway. On the ferret
version of sup, I'm not able to apply labels with the 'l' command. However, I
have a hook for applying a label from the List-ID that does still work. So
this patch may or may not work, I think I'm possibly having problems of my
own.

Actually, I haven't tried to restore from a dumpfile for ferret. I'll do that
later and see if it works.

-- Blake Burkhart
_______________________________________________
sup-talk mailing list
[email protected]
http://rubyforge.org/mailman/listinfo/sup-talk

Reply via email to