[Mailman-Developers] patch to remove_member for mass unsubscribes

Marc MERLIN marc_news@valinux.com
Tue, 3 Apr 2001 14:46:08 -0700


On Tue, Apr 03, 2001 at 11:09:44AM +0100, Ralph Corderoy wrote:
> 
> Hi,
> 
> > Basically it adds a special list name: _alllists_ which does this:
> > 
> > # time ./remove_members -f /tmp/badmails _alllists_
> 
> Perhaps an -option would be better than special-casing a list name.

I  thought  about  it,  but  it  breaks  the  option  parsing  a  bit  since
remove_members kind of expects a list name.
Not  that it  can't be  fixed, mind  you, but  having a  list name  required
depending on whether a flag was there or not seemed a bit ackward.

That said, it's trivial to change, and as long as the functionality makes it
into  the main  distribution, how  the arguments  are parsed  doesn't really
matter to me,
Barry or however considers merging this  in is welcome to change the parsing
logic and I don't think they need a patch from me to do this :-)

Marc
-- 
Microsoft is to operating systems & security ....
                                      .... what McDonalds is to gourmet cooking
  
Home page: http://marc.merlins.org/   |   Finger marc_f@merlins.org for PGP key