[Development] Platform maintainers

Thiago Macieira thiago.macieira at intel.com
Fri Sep 26 16:34:54 CEST 2014


On Friday 26 September 2014 13:33:55 Giulio Camuffo wrote:
> Uhm, how did you get that number? A "git log -n 200 |grep Andy|wc -l"
> says 5 to me, four Reviewed-by and one Author, the last one being on
> February 11.
> I don't want to be dismissive of the work he has done on qtwayland,
> but he hasn't been working on it for many months, and on and off for
> even more. Jørgen or Laszlo would be a better fit, imho.

I ran this:

$ git log origin/5.4~200..origin/5.4 | grep Reviewed-by | sort | uniq -c
     32     Reviewed-by: Andrew Knight <andrew.knight at digia.com>
    140     Reviewed-by: Andy Nichols <andy.nichols at digia.com>
      1     Reviewed-by: Elvis Lee <kwangwoong.lee at lge.com>
      1     Reviewed-by: Frederik Gladhorn <frederik.gladhorn at digia.com>
      3     Reviewed-by: Giulio Camuffo <giuliocamuffo at gmail.com>
     50     Reviewed-by: Giulio Camuffo <giulio.camuffo at jollamobile.com>
      2     Reviewed-by: Gunnar Sletta <gunnar.sletta at digia.com>
     34     Reviewed-by: Gunnar Sletta <gunnar.sletta at jollamobile.com>
      1     Reviewed-by: Gunnar Sletta <gunnar at sletta.org>
     36     Reviewed-by: Jan Arne Petersen <jan.petersen at kdab.com>
      1     Reviewed-by: Jędrzej Nowacki <jedrzej.nowacki at digia.com>
      2     Reviewed-by: Joerg Bornemann <joerg.bornemann at digia.com>
    109     Reviewed-by: Jørgen Lind <jorgen.lind at digia.com>
      8     Reviewed-by: Jørgen Lind <jorgen.lind at gmail.com>
     89     Reviewed-by: Laszlo Agocs <laszlo.agocs at digia.com>
      1     Reviewed-by: Laszlo Papp <lpapp at kde.org>
      1     Reviewed-by: Lubomir Rintel <lkundrak at v3.sk>
      1     Reviewed-by: Michael Brasser <michael.brasser at live.com>
      2     Reviewed-by: Mikko Levonmaa <mikko.levonmaa at lge.com>
      6     Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen at digia.com>
      7     Reviewed-by: Paul Olav Tvete <paul.tvete at digia.com>
      2     Reviewed-by: Philippe Coval <rzr at gna.org>
     18     Reviewed-by: Pier Luigi Fiorini <pierluigi.fiorini at gmail.com>
     26     Reviewed-by: Robin Burchell <robin.burchell at viroteck.net>
     30     Reviewed-by: Robin Burchell <robin+qt at viroteck.net>
     17     Reviewed-by: Samuel Rødal <samuel.rodal at digia.com>
      3     Reviewed-by: Sergio Ahumada <sahumada at blackberry.com>
      5     Reviewed-by: Shawn Rutledge <shawn.rutledge at digia.com>
      1     Reviewed-by: Simo Fält <simo.falt at digia.com>
      2     Reviewed-by: Thiago Macieira <thiago.macieira at intel.com>
      1     Reviewed-by: Tor Arne Vestbø <tor.arne.vestbo at digia.com>
      1     Reviewed-by: Vesa Halttunen
      1     Reviewed-by: Yen-Chin Lee <coldnew.tw at gmail.com>

Problem: origin/5.4~200..origin/5.4 contains more than 200 commits due to 
merging. That's actually 491. I didn't notice this problem because the number 
of reviews per person were all less than 200. Bad coincidence.

If I restrict to actually 200 commits, here's the result:

$ git log -n200 origin/5.4 | grep Reviewed-by | sort | uniq -c          
      4     Reviewed-by: Andrew Knight <andrew.knight at digia.com>
      4     Reviewed-by: Andy Nichols <andy.nichols at digia.com>
      1     Reviewed-by: Frederik Gladhorn <frederik.gladhorn at digia.com>
      1     Reviewed-by: Giulio Camuffo <giuliocamuffo at gmail.com>
     50     Reviewed-by: Giulio Camuffo <giulio.camuffo at jollamobile.com>
     34     Reviewed-by: Gunnar Sletta <gunnar.sletta at jollamobile.com>
      1     Reviewed-by: Gunnar Sletta <gunnar at sletta.org>
      2     Reviewed-by: Jan Arne Petersen <jan.petersen at kdab.com>
     22     Reviewed-by: Jørgen Lind <jorgen.lind at digia.com>
     48     Reviewed-by: Laszlo Agocs <laszlo.agocs at digia.com>
      1     Reviewed-by: Michael Brasser <michael.brasser at live.com>
      2     Reviewed-by: Mikko Levonmaa <mikko.levonmaa at lge.com>
      1     Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen at digia.com>
      2     Reviewed-by: Philippe Coval <rzr at gna.org>
      6     Reviewed-by: Pier Luigi Fiorini <pierluigi.fiorini at gmail.com>
     26     Reviewed-by: Robin Burchell <robin.burchell at viroteck.net>
     25     Reviewed-by: Robin Burchell <robin+qt at viroteck.net>
      1     Reviewed-by: Sergio Ahumada <sahumada at blackberry.com>
      5     Reviewed-by: Shawn Rutledge <shawn.rutledge at digia.com>
      1     Reviewed-by: Thiago Macieira <thiago.macieira at intel.com>
      1     Reviewed-by: Vesa Halttunen
      1     Reviewed-by: Yen-Chin Lee <coldnew.tw at gmail.com>

And you're right, the last time Andy reviewed anything in qtwayland was 
February/2014.

So, Andy, are you coming back? Or do you want to step down for someone else?

-- 
Thiago Macieira - thiago.macieira (AT) intel.com
  Software Architect - Intel Open Source Technology Center




More information about the Development mailing list