On Thu, May 29, 2014 at 02:14:39PM -0700, Sebastian Kricner wrote:
> > @@ -56,7 +56,7 @@ def __init__(self, qtile, dgroups, key_binder=None, delay=1):
> >          self.rules_map = {}
> >          self.last_rule_id = 0
> >  
> > -        for rule in getattr(qtile.config, 'dgroups_app_rules', []):
> > +        for rule in getattr(qtile.config, 'app_rules', []):
> 
> I just had written, that this indeed breaks configs. Must be somehow gotten away in the commit message. This whole breaks some things. Like the new Match usage.But i think in the long term it is good to use a unique way to define matches.

Yep, I will try and post some more review comments later tonight, but
there is a lot of config breakage here. Overall I like it, though.
Even if it was possible already, it was cumbersome and this patch
makes Match in particular a lot easier to work with. Thanks!