Bug 79234 – Add webkitpy.common.config.affiliations

• Status: RESOLVED
• Resolution: INVALID
• Component: Tools / Tests

[Andy Wingo]

The patch to be attached scrapes http://trac.webkit.org/wiki/WebKit%20Team for affiliation information, associating it with committers.Account objects.

[Andy Wingo]

Created attachment 128199 [details]
Patch

[Ryosuke Niwa]

Why are we adding this feature?

[Ryosuke Niwa]

FWIW, affiliation information on the wiki page is outdated for many people.

[Andy Wingo]

(In reply to comment #2)
> Why are we adding this feature?

It's useful when doing commit analysis, for example as Jon Corbet does for the Linux kernel.  Don't know if you've seen those, but they're always interesting.  http://lwn.net/Articles/472852/, for example.  It's not so useful for Apple and Google, but it's fun to see contributions from smaller organizations bob up and down.

(In reply to comment #3)
> FWIW, affiliation information on the wiki page is outdated for many people.

ACK.  If it is useful to have this information, people should update it themselves.

[Ryosuke Niwa]

(In reply to comment #4)
> (In reply to comment #3)
> > FWIW, affiliation information on the wiki page is outdated for many people.
> 
> ACK.  If it is useful to have this information, people should update it themselves.

How are you going to enforce that? wiki page is very unreliable source of information since anyone who registers can edit. They don't even need to be a committer.

If anything, you should be relying on email addresses as done in view-source:http://www.webkit.org/team.html.

[Andy Wingo]

(In reply to comment #5)
> (In reply to comment #4)
> > (In reply to comment #3)
> > > FWIW, affiliation information on the wiki page is outdated for many people.
> > 
> > ACK.  If it is useful to have this information, people should update it themselves.
> 
> How are you going to enforce that?

What do you think needs enforcing?

My point is simply that this wiki page indicates affiliations.  Regardless of the outcome of this bug, if that page has an incorrect entry for me, it is my responsibility to fix it, or otherwise remove myself from the page, or work for the deletion of that page, or whatever.

> If anything, you should be relying on email addresses as done in view-source:http://www.webkit.org/team.html.

That's an interesting page.  Have you viewed it as http://www.webkit.org/team.html?annotate ?  The wiki often has better current information.

Matching against domains is a useful fallback, but it's not great for @webkit.org and @chromium.org addresses.

[Ryosuke Niwa]

(In reply to comment #6)
> (In reply to comment #5)
> > (In reply to comment #4)
> > > (In reply to comment #3)
> > > > FWIW, affiliation information on the wiki page is outdated for many people.
> > > 
> > > ACK.  If it is useful to have this information, people should update it themselves.
> > 
> > How are you going to enforce that?
> 
> What do you think needs enforcing?
> 
> My point is simply that this wiki page indicates affiliations.  Regardless of the outcome of this bug, if that page has an incorrect entry for me, it is my responsibility to fix it, or otherwise remove myself from the page, or work for the deletion of that page, or whatever.

That's not your responsibility and nobody else's. We simply don't have such a policy.

[Andy Wingo]

We seem to have gotten off on the wrong foot.  Apologies if I've gone about this the wrong way.

I would like to map commits to affiliations.  How do you suggest that I do that for @webkit.org addresses?

Andy

[Andy Wingo]

Also, should we delete http://trac.webkit.org/wiki/WebKit%20Team, in favor of http://www.webkit.org/team.html ?

[Ryosuke Niwa]

(In reply to comment #8)
> We seem to have gotten off on the wrong foot.  Apologies if I've gone about this the wrong way.
> 
> I would like to map commits to affiliations.

I don't think that's possible nor is desirable since those kinds of statistics tend to invoke unnecessary political tensions between people. I'll be strongly against such statistics.

(In reply to comment #9)
> Also, should we delete http://trac.webkit.org/wiki/WebKit%20Team, in favor of http://www.webkit.org/team.html ?

That's the plan. We still need a way to figure out areas of expertise.

[Ryosuke Niwa]

strongly against *generating* such statistics.

[Adam Barth]

Comment on attachment 128199 [details]
Patch

I don't think it's good for the community to track this information.

[Andy Wingo]

Closing as invalid.  Thanks for the feedback, though!