Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 383245 - Willikins should support emailing contributors on a per-package basis
Summary: Willikins should support emailing contributors on a per-package basis
Status: RESOLVED WONTFIX
Alias: None
Product: Gentoo Infrastructure
Classification: Unclassified
Component: Other (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Robin Johnson
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-09-16 16:14 UTC by James Broadhead
Modified: 2021-06-19 04:53 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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 mike@something.oth
irc: Alan: !cmd msg app-foo/bar "Fix this please" 
irc: willikins: Emailing mike@something.oth ... 

Case 2:
irc: Alan: !cmd contributor app-foo/bar
irc: willikins: Last contributor: Mike mike@something.oth
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 mike@something.oth
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 mike@something.oth ...  

Case 4:
irc: Alan: !cmd contributors app-foo/bar
irc: willikins: Last contributor: Mike mike@something.oth
irc: willikins: Most frequent contributor: Bill bill@something.oth
irc: Alan: !cmd msgall app-foo/bar "Fix this please" 
irc: willikins: Emailing mike@something.oth, bill@something.oth

====

Reproducible: Always
Comment 1 Jeremy Olexa (darkside) (RETIRED) archtester gentoo-dev Security 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 archtester Gentoo Infrastructure gentoo-dev Security 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 mike@something.oth
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) archtester gentoo-dev Security 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 archtester Gentoo Infrastructure gentoo-dev Security 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 mike@something.oth
> 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 archtester Gentoo Infrastructure gentoo-dev Security 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 archtester Gentoo Infrastructure gentoo-dev Security 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.