Archive of RubyForge sup-talk mailing list
 help / color / mirror / Atom feed
* [sup-talk] [PATCH] Unbreak "bounce-command" hook (was referencing an nonexistent variable)
@ 2009-07-11 11:57 Adeodato Simó
  2009-07-11 14:28 ` Ben Walton
  2009-07-27 16:30 ` William Morgan
  0 siblings, 2 replies; 8+ messages in thread
From: Adeodato Simó @ 2009-07-11 11:57 UTC (permalink / raw)


Signed-off-by: Adeodato Sim? <dato at net.com.org.es>
---
 lib/sup/modes/thread-view-mode.rb |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/lib/sup/modes/thread-view-mode.rb b/lib/sup/modes/thread-view-mode.rb
index 759191e..737f6f1 100644
--- a/lib/sup/modes/thread-view-mode.rb
+++ b/lib/sup/modes/thread-view-mode.rb
@@ -204,7 +204,7 @@ EOS
       end
     end
 
-    cmd = case HookManager.run "bounce-command", :from => m.from, :to => to
+    cmd = case (hookcmd = HookManager.run "bounce-command", :from => m.from, :to => to)
           when nil, /^$/ then defcmd
           else hookcmd
           end + ' ' + to.map { |t| t.email }.join(' ')
-- 
1.6.3.3



^ permalink raw reply	[flat|nested] 8+ messages in thread

* [sup-talk] [PATCH] Unbreak "bounce-command" hook (was referencing an nonexistent variable)
  2009-07-11 11:57 [sup-talk] [PATCH] Unbreak "bounce-command" hook (was referencing an nonexistent variable) Adeodato Simó
@ 2009-07-11 14:28 ` Ben Walton
  2009-07-27 16:22   ` William Morgan
  2009-07-27 16:30 ` William Morgan
  1 sibling, 1 reply; 8+ messages in thread
From: Ben Walton @ 2009-07-11 14:28 UTC (permalink / raw)


Excerpts from Adeodato Sim?'s message of Sat Jul 11 07:57:55 -0400 2009:

> Signed-off-by: Adeodato Sim? <dato at net.com.org.es>
> ---
>  lib/sup/modes/thread-view-mode.rb |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)

+1 for this.  Not sure how I managed to submit the original patch like
this, since I did test various forms of results returned from the
bounce-command hook...anyway, I've applied this and verified it.

-Ben
-- 
Ben Walton
Systems Programmer - CHASS
University of Toronto
C:416.407.5610 | W:416.978.4302

GPG Key Id: 8E89F6D2; Key Server: pgp.mit.edu
Contact me to arrange for a CAcert assurance meeting.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://rubyforge.org/pipermail/sup-talk/attachments/20090711/3b5d572f/attachment.bin>


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [sup-talk] [PATCH] Unbreak "bounce-command" hook (was referencing an nonexistent variable)
  2009-07-11 14:28 ` Ben Walton
@ 2009-07-27 16:22   ` William Morgan
  2009-07-27 16:25     ` Ben Walton
  0 siblings, 1 reply; 8+ messages in thread
From: William Morgan @ 2009-07-27 16:22 UTC (permalink / raw)


Reformatted excerpts from Ben Walton's message of 2009-07-11:
> +1 for this.  Not sure how I managed to submit the original patch like
> this, since I did test various forms of results returned from the
> bounce-command hook...anyway, I've applied this and verified it.

I don't see this in your bw/bounce_message branch. I'd like to remerge
that.
-- 
William <wmorgan-sup at masanjin.net>


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [sup-talk] [PATCH] Unbreak "bounce-command" hook (was referencing an nonexistent variable)
  2009-07-27 16:22   ` William Morgan
@ 2009-07-27 16:25     ` Ben Walton
  2009-07-27 16:29       ` William Morgan
  2009-07-27 16:30       ` Ben Walton
  0 siblings, 2 replies; 8+ messages in thread
From: Ben Walton @ 2009-07-27 16:25 UTC (permalink / raw)


Excerpts from William Morgan's message of Mon Jul 27 12:22:48 -0400 2009:

> I don't see this in your bw/bounce_message branch. I'd like to remerge
> that.

Sorry.  I didn't publish the change, I thought it would be `git am`'d
from the original mail.  I applied it to my local running copy of next
though, which is what I was referring to.  I'll try to remember to
push things like this in the future.

-Ben
-- 
Ben Walton
Systems Programmer - CHASS
University of Toronto
C:416.407.5610 | W:416.978.4302

GPG Key Id: 8E89F6D2; Key Server: pgp.mit.edu
Contact me to arrange for a CAcert assurance meeting.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://rubyforge.org/pipermail/sup-talk/attachments/20090727/85a7f7a7/attachment.bin>


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [sup-talk] [PATCH] Unbreak "bounce-command" hook (was referencing an nonexistent variable)
  2009-07-27 16:25     ` Ben Walton
@ 2009-07-27 16:29       ` William Morgan
  2009-07-27 16:30       ` Ben Walton
  1 sibling, 0 replies; 8+ messages in thread
From: William Morgan @ 2009-07-27 16:29 UTC (permalink / raw)


Reformatted excerpts from Ben Walton's message of 2009-07-27:
> Sorry. I didn't publish the change, I thought it would be `git am`'d
> from the original mail.

No problem. We can do it either way. I'll apply this patch directly.
-- 
William <wmorgan-sup at masanjin.net>


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [sup-talk] [PATCH] Unbreak "bounce-command" hook (was referencing an nonexistent variable)
  2009-07-27 16:25     ` Ben Walton
  2009-07-27 16:29       ` William Morgan
@ 2009-07-27 16:30       ` Ben Walton
  2009-07-27 16:45         ` William Morgan
  1 sibling, 1 reply; 8+ messages in thread
From: Ben Walton @ 2009-07-27 16:30 UTC (permalink / raw)


Excerpts from Ben Walton's message of Mon Jul 27 12:25:52 -0400 2009:
> Sorry.  I didn't publish the change, I thought it would be `git am`'d
> from the original mail.  I applied it to my local running copy of next
> though, which is what I was referring to.  I'll try to remember to
> push things like this in the future.

I've now `git am`'d and pushed on my development copy to
git://code.chass.utoronto.ca/bwalton-sup.git.  You should be able to
remerge the bw/bounce_message branch and get Adeodato's fix.

Thanks
-Ben

-- 
Ben Walton
Systems Programmer - CHASS
University of Toronto
C:416.407.5610 | W:416.978.4302

GPG Key Id: 8E89F6D2; Key Server: pgp.mit.edu
Contact me to arrange for a CAcert assurance meeting.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://rubyforge.org/pipermail/sup-talk/attachments/20090727/eb249faf/attachment.bin>


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [sup-talk] [PATCH] Unbreak "bounce-command" hook (was referencing an nonexistent variable)
  2009-07-11 11:57 [sup-talk] [PATCH] Unbreak "bounce-command" hook (was referencing an nonexistent variable) Adeodato Simó
  2009-07-11 14:28 ` Ben Walton
@ 2009-07-27 16:30 ` William Morgan
  1 sibling, 0 replies; 8+ messages in thread
From: William Morgan @ 2009-07-27 16:30 UTC (permalink / raw)


Applied, thanks!
-- 
William <wmorgan-sup at masanjin.net>


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [sup-talk] [PATCH] Unbreak "bounce-command" hook (was referencing an nonexistent variable)
  2009-07-27 16:30       ` Ben Walton
@ 2009-07-27 16:45         ` William Morgan
  0 siblings, 0 replies; 8+ messages in thread
From: William Morgan @ 2009-07-27 16:45 UTC (permalink / raw)


Reformatted excerpts from Ben Walton's message of 2009-07-27:
> I've now `git am`'d and pushed on my development copy to
> git://code.chass.utoronto.ca/bwalton-sup.git.  You should be able to
> remerge the bw/bounce_message branch and get Adeodato's fix.

Heh. :)
-- 
William <wmorgan-sup at masanjin.net>


^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2009-07-27 16:45 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-07-11 11:57 [sup-talk] [PATCH] Unbreak "bounce-command" hook (was referencing an nonexistent variable) Adeodato Simó
2009-07-11 14:28 ` Ben Walton
2009-07-27 16:22   ` William Morgan
2009-07-27 16:25     ` Ben Walton
2009-07-27 16:29       ` William Morgan
2009-07-27 16:30       ` Ben Walton
2009-07-27 16:45         ` William Morgan
2009-07-27 16:30 ` William Morgan

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox