[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [sup-talk] Bug: Xapian exception after having polled



Still the same..

(run without '-n' then P this time, thats the reason for the longer exception..)

[sup.git](next-nil) $ git log --oneline -4
7e99810 for your confirmation..
eafea2b more id assertions
0eee097 Merge branch 'master' into next
93b5552 more release-script updates

- gaute

On Sun, Oct 4, 2009 at 9:03 PM, Rich Lane <rlane@club.cc.cmu.edu> wrote:
> Oops, sorry, bad assertions. Please move the two in
> self.build_from_source to the end of load_from_source!.
>
> Excerpts from Gaute Hope's message of Sun Oct 04 14:56:31 -0400 2009:
>> Still having problems, but got a bit more output, see attached exception.log
>>
>> [sup.git](next) $ git log --oneline -6
>> a209178 more id assertions
>> 0eee097 Merge branch 'master' into next
>> 93b5552 more release-script updates
>> f56badb Merge branch 'master' into next
>> b9071e5 change date for 0.9 release
>> 9a5c0d1 Merge branch 'save-all-attachments' into next
>>
>> - gaute
>>
>> On Sun, Oct 4, 2009 at 8:45 PM, Rich Lane <rlane@club.cc.cmu.edu> wrote:
>> > Ok, I've attached a patch with more assertions. Also, can you try with a clean
>> > checkout of next and see if the problem still occurs?
>> >
>> --- RuntimeError from thread: main
>> @id nil
>> /home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/message.rb:342:in
>> `build_from_source'
>> /home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/poll.rb:160:in
>> `each_message_from'
>> /home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/maildir.rb:160:in `each'
>> /home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/maildir.rb:157:in `upto'
>> /home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/maildir.rb:157:in `each'
>> /home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/util.rb:560:in `send'
>> /home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/util.rb:560:in `__pass'
>> /home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/util.rb:547:in `method_missing'
>> /home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/poll.rb:154:in
>> `each_message_from'
>> /home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/poll.rb:108:in `do_poll'
>> /home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/poll.rb:96:in `each'
>> /home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/poll.rb:96:in `do_poll'
>> /home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/poll.rb:95:in `synchronize'
>> /home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/poll.rb:95:in `do_poll'
>> /home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/util.rb:520:in `send'
>> /home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/util.rb:520:in `method_missing'
>> /home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/modes/poll-mode.rb:15:in `poll'
>> /home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/poll.rb:47:in `poll_with_sources'
>> /home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/poll.rb:62:in `poll'
>> /home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/util.rb:520:in `send'
>> /home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/util.rb:520:in `method_missing'
>> /home/gaute/.gem/ruby/1.8/gems/sup-999/bin/sup:287
>> /home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup.rb:73:in `reporting_thread'
>> /home/gaute/.gem/ruby/1.8/gems/sup-999/bin/sup:287
>> /home/gaute/.gem/ruby/1.8/bin/sup:19:in `load'
>> /home/gaute/.gem/ruby/1.8/bin/sup:19
>
--- RuntimeError from thread: poll after loading inbox
@id nil
/home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/message.rb:254:in `load_from_source!'
/home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/message.rb:342:in `build_from_source'
/home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/poll.rb:160:in `each_message_from'
/home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/source.rb:104:in `each'
/home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/util.rb:560:in `send'
/home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/util.rb:560:in `__pass'
/home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/util.rb:547:in `method_missing'
/home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/poll.rb:154:in `each_message_from'
/home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/poll.rb:108:in `do_poll'
/home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/poll.rb:96:in `each'
/home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/poll.rb:96:in `do_poll'
/home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/poll.rb:95:in `synchronize'
/home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/poll.rb:95:in `do_poll'
/home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/util.rb:520:in `send'
/home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/util.rb:520:in `method_missing'
/home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/modes/poll-mode.rb:15:in `poll'
/home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/poll.rb:47:in `poll_with_sources'
/home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/poll.rb:62:in `poll'
/home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/util.rb:520:in `send'
/home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/util.rb:520:in `method_missing'
/home/gaute/.gem/ruby/1.8/gems/sup-999/bin/sup:197
/home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup.rb:77:in `reporting_thread'
/home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup.rb:75:in `initialize'
/home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup.rb:75:in `new'
/home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup.rb:75:in `reporting_thread'
/home/gaute/.gem/ruby/1.8/gems/sup-999/bin/sup:197
/home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/modes/thread-index-mode.rb:669:in `call'
/home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/modes/thread-index-mode.rb:669:in `__unprotected_load_threads'
/home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/modes/thread-index-mode.rb:610:in `call'
/home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/modes/thread-index-mode.rb:610:in `load_n_threads_background'
/home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup.rb:77:in `reporting_thread'
/home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup.rb:75:in `initialize'
/home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup.rb:75:in `new'
/home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup.rb:75:in `reporting_thread'
/home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/modes/thread-index-mode.rb:608:in `load_n_threads_background'
/home/gaute/.gem/ruby/1.8/gems/sup-999/lib/sup/modes/thread-index-mode.rb:679:in `__unprotected_load_threads'
(eval):12:in `load_threads'
/home/gaute/.gem/ruby/1.8/gems/sup-999/bin/sup:197
/home/gaute/.gem/ruby/1.8/bin/sup:19:in `load'
/home/gaute/.gem/ruby/1.8/bin/sup:19
From 7e99810ebf7d3f361c31236c3c131c1c794f8076 Mon Sep 17 00:00:00 2001
From: Gaute Hope <eg@gaute.vetsj.com>
Date: Sun, 4 Oct 2009 21:17:23 +0200
Subject: [PATCH] for your confirmation..

---
 lib/sup/message.rb |    5 ++---
 1 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/lib/sup/message.rb b/lib/sup/message.rb
index 7592b51..fad9866 100644
--- a/lib/sup/message.rb
+++ b/lib/sup/message.rb
@@ -242,7 +242,6 @@ class Message
           parse_header @source.load_header(@source_info)
           fail "id nil after parse_header" unless @id
           message_to_chunks @source.load_message(@source_info)
-          fail "id nil after message_to_chunks" unless @id
         rescue SourceError, SocketError => e
           warn "problem getting messages from #{@source}: #{e.message}"
           ## we need force_to_top here otherwise this window will cover
@@ -252,6 +251,8 @@ class Message
           [Chunk::Text.new(error_message(e.message).split("\n"))]
         end
       end
+    fail "@id nil" unless @id
+    fail "id nil" unless id
   end
 
   def error_message msg
@@ -339,8 +340,6 @@ EOS
   def self.build_from_source source, source_info
     m = Message.new :source => source, :source_info => source_info
     m.load_from_source!
-    fail "@id nil" unless @id
-    fail "id nil" unless id
     m
   end
 
-- 
1.6.4.4