Bug 383245 – Willikins should support emailing contributors on a per-package basis

• Product: Gentoo Infrastructure
• Status: RESOLVED WONTFIX

[Description James Broadhead 2011-09-16 16:14:41 UTC]

It would be very helpful if willikins could use information from the tree to help contact overlay contributors when they are not available on irc. 

I propose that willikins be extended to parse the Changelog, and enable the sending of emails from irc with a new command: 
!cmd contributor <pkg>
!cmd contributors <pkg> 
!msg <pkg>
!msgall <pkg>

Since many overlay packages are usually managed by a single contributor, but have occasional commits from others, an ideal implementation would take this into account, for example by picking the best person to contact by the number of their commits (or otherwise)

eg. Dev Alan needs to contact the contributor who manages package app-foo/bar from an overlay

Current Practise: 
- Alan finds a problem with app-foo/bar, from an overlay. 
- Alan reads metadata.xml / Changelog, finds out that Bill is the usual contributor
- Alan emails Bill manually

Desired functionality:
Case 1:
irc: Alan: !cmd contributor app-foo/bar
irc: willikins: Last contributor: Mike [e-mail]
irc: Alan: !cmd msg app-foo/bar "Fix this please" 
irc: willikins: Emailing [e-mail] ... 

Case 2:
irc: Alan: !cmd contributor app-foo/bar
irc: willikins: Last contributor: Mike [e-mail]
irc: Alan: !cmd msg app-foo/bar "Fix this please" 
irc: willikins: Looks like Mike is in the chatroom! 
irc: Alan: Mike, are you looking after bar? 
irc: Mike: Sure etc. 

Case 3:
irc: Alan: !cmd contributor app-foo/bar
irc: willikins: Last contributor: Mike [e-mail]
irc: Alan: !cmd msg app-foo/bar "Fix this please" 
irc: willikins: Looks like Mike is in the chatroom! 
irc: Alan: Mike, are you looking after bar? 
irc: Mike: Nope, that is OtherMike
irc: Alan: !cmd msg app-foo/bar "Fix this please" 
irc: willikins: Emailing [e-mail] ...  

Case 4:
irc: Alan: !cmd contributors app-foo/bar
irc: willikins: Last contributor: Mike [e-mail]
irc: willikins: Most frequent contributor: Bill [e-mail]
irc: Alan: !cmd msgall app-foo/bar "Fix this please" 
irc: willikins: Emailing [e-mail], [e-mail]

====

Reproducible: Always

[Comment 1 Jeremy Olexa (darkside) (RETIRED) 2011-09-16 16:24:08 UTC]

Yay, spam via IRC. Also, not sure if you can even get a good metric from the ChangeLog. What happens when person1 has 20x more CL entries than the maintainer but no longer cares about the package for whatever reason.

Personally, not in favor, regardless assigning to maintainer.

[Comment 2 Michał Górny 2011-09-16 16:28:18 UTC]

(In reply to comment #1)
> Yay, spam via IRC. Also, not sure if you can even get a good metric from the
> ChangeLog. What happens when person1 has 20x more CL entries than the
> maintainer but no longer cares about the package for whatever reason.

Well, the actual use here was in Sunrise to remotely ping and bring back committers to fix the breakages they have committed.

[Comment 3 James Broadhead 2011-09-16 16:29:47 UTC]

In case it was unclear, this would be for overlay packages only, which are not
covered by bugzilla. 

Case 5:
irc: Alan: !cmd contributor app-foo/zar
irc: willikins: Last contributor: Mike [e-mail]
irc: Alan: !cmd msg app-foo/zar "Fix this please" 
irc: willikins: app-foo/zar is in the main tree, consider opening a bug?

[Comment 4 Jeremy Olexa (darkside) (RETIRED) 2011-09-16 16:37:10 UTC]

Willikins doesn't even support overlays now, right? I guess that is the first point that someone will need to implement.

[Comment 5 Michał Górny 2011-09-16 19:00:48 UTC]

(In reply to comment #3)
> In case it was unclear, this would be for overlay packages only, which are not
> covered by bugzilla. 
> 
> Case 5:
> irc: Alan: !cmd contributor app-foo/zar
> irc: willikins: Last contributor: Mike [e-mail]
> irc: Alan: !cmd msg app-foo/zar "Fix this please" 
> irc: willikins: app-foo/zar is in the main tree, consider opening a bug?

Er, we wanted simple ircname->e-mail mapping:

> !msg hangfire fix it now!
< message to foo@bar sent

[Comment 6 Robin Johnson 2011-09-16 20:15:27 UTC]

1. It will need to be (overlay, package) -> (contributor), because different overlays will not always have the same contributors for a given package.
2. Willikins has NO overlay support at all right.
2.1. Amongst such, exposing the owner email for an overly would be doable.
3. The general ircname -> email stuff, that's doable, but it would be opt-in.

The source for the rbot plugins is here:
http://git.overlays.gentoo.org/gitweb/?p=proj/rbot-gentoo.git;a=summary

Patches welcome.

[Comment 7 Robin Johnson 2021-06-19 04:53:06 UTC]

I agree more about the spam now than in the past. won't fix. If the person isn't on IRC, you should use your own mail client or bug system for that overlay to contact them.