From: Gaute Hope <eg@gaute.vetsj.com>
To: Rich Lane <rlane@club.cc.cmu.edu>
Cc: sup-talk <sup-talk@rubyforge.org>
Subject: Re: [sup-talk] Bug: Xapian exception after having polled
Date: Sun, 4 Oct 2009 21:20:50 +0200 [thread overview]
Message-ID: <5a5b14cf0910041220p26a4c830va172df0e33542480@mail.gmail.com> (raw)
In-Reply-To: <1254682966-sup-6629@zyrg.net>
[-- Attachment #1: Type: text/plain, Size: 3128 bytes --]
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
>
[-- Attachment #2: exception-log.txt --]
[-- Type: text/plain, Size: 3009 bytes --]
--- 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
[-- Attachment #3: 0001-for-your-confirmation.patch --]
[-- Type: text/x-patch, Size: 1347 bytes --]
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
next prev parent reply other threads:[~2009-10-04 19:20 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-10-03 10:36 gauteh
2009-10-03 18:22 ` Rich Lane
2009-10-04 10:15 ` Gaute Hope
2009-10-04 18:45 ` Rich Lane
2009-10-04 18:56 ` Gaute Hope
2009-10-04 19:03 ` Rich Lane
2009-10-04 19:20 ` Gaute Hope [this message]
2009-10-05 22:01 ` Gaute Hope
2009-10-06 10:14 ` Guillaume Quintard
2009-10-06 13:34 ` Gaute Hope
2009-10-06 14:59 ` William Morgan
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5a5b14cf0910041220p26a4c830va172df0e33542480@mail.gmail.com \
--to=eg@gaute.vetsj.com \
--cc=rlane@club.cc.cmu.edu \
--cc=sup-talk@rubyforge.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox