Hi Otavio, On Wed, Aug 15, 2012 at 12:44 PM, Otavio Salvador <ota...@ossystems.com.br> wrote: > In case an address is listed in the To list, those will be skipped on > Cc list or user might end with a duplicated message. > > This fixes the case when a tag points to same address used as series > destination thus avoiding duplicated sending. > > Signed-off-by: Otavio Salvador <ota...@ossystems.com.br> > --- > tools/patman/series.py | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/tools/patman/series.py b/tools/patman/series.py > index eda1e9b..c338ccb 100644 > --- a/tools/patman/series.py > +++ b/tools/patman/series.py > @@ -114,6 +114,12 @@ class Series(dict): > cc_list += gitutil.BuildEmailList(commit.tags) > cc_list += gitutil.BuildEmailList(commit.cc_list) > > + # Remove items on To list > + try: > + map(cc_list.remove, gitutil.BuildEmailList(self.to)) > + except ValueError: > + pass > +
I'm ok with this as is. But could you do something like: if 'to' in self: map(cc_list.remove, gitutil.BuildEmailList(self.to)) > for email in cc_list: > if email == None: > email = col.Color(col.YELLOW, "<alias '%s' not found>" > -- > 1.7.10.4 > Regards, Simon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
