Great Circle Associates Majordomo-Workers
(May 1999)
 

Indexed By Date: [Previous] [Next] Indexed By Thread: [Previous] [Next]

Subject: Re: Minor address bug in resend's approval-bounce code
From: "Brian Kelly" <KellyB @ Mail . Montclair . edu>
Organization: Montclair State University (http://www.montclair.edu)
Date: Fri, 28 May 1999 12:00:51 -0400
To: Dave Wolfe <dwolfe @ risc . sps . mot . com>, Bill . Houle @ SanDiegoCA . NCR . COM
Cc: Majordomo-Workers @ GreatCircle . COM, Majordomo patches <majordomo-patches @ cloud . ccsf . cc . ca . us>
References: <374C74EE.8C9F3DB9@Montclair.edu> <19990528083333.C79324@risc.sps.mot.com>

Dave Wolfe wrote:

> [ Brian Kelly writes: ]
>
> > While trying out the moderation features of Majordomo, I came across
> > what is probably an obscure problem.  When "moderate (resend)" is
> > enabled and "moderator (resend)" is no defined, the default address to
> > send the approval messages to is "<list>-approval".  On my system, the
> > hostname is not the same as $whereami so these messages would not reach
> > their destination.
> [...]
> > (plus, I've modified my resend code so a diff might not be all
> > that useful).
>
> That seems to be the problem. I can't find any version of 1.94 that uses
> $opt_l-approval as the default when moderator is unset, they all use
> $sender, so your changes/other patches must've introduced this problem
> in the first place.

That's odd -- The changes I made to resend were earlier on in the code and
not around that section.  I did however install Majorcool's web interface (I
almost forgot about it).  Looking through the section there concerning
Majorcool's patches to allow it to handle the Approval queue, I see that it
patches resend and changes the section in question.

> Checking further, I see that this problem was introduced by Bill Houle's
> patch named resend_parse.0 in the unofficial 1.94.4 patches.

That's not my source for the patch, but he did write the Majorcool patch.

> Joe, can you see to it that Brian's fix is applied to the patch file?

Sorry about the confusion in this -- I should have remembered applying that
patch from the start.  My bad.

Brian Kelly (Brian.Kelly@Montclair.edu)
Information Technology
Montclair State University




References:
Indexed By Date Previous: Re: Minor address bug in resend's approval-bounce code
From: "Joe R. Jah" <jjah@sol.ccsf.cc.ca.us>
Next:
From: (nil)
Indexed By Thread Previous: Re: Minor address bug in resend's approval-bounce code
From: "Joe R. Jah" <jjah@sol.ccsf.cc.ca.us>
Next: Majordomo under NW4.11/5.0
From: "Andy Reisenhofer" <andy@reisenhofer.com>

Google
 
Search Internet Search www.greatcircle.com