Re: emacs-26 4387bb4 1/3: Update authors

Re: emacs-26 4387bb4 1/3: Update authors

[Glenn Morris]

Nicolas Petton wrote:

> branch: emacs-26
> commit 4387bb44ae59853feab94b72ec568b73ad1f827a
[...]
> --- a/admin/authors.el
> +++ b/admin/authors.el
> @@ -130,6 +130,7 @@ files.")
>      ("Agustín Martín" "Agustin Martin" "Agustín Martín Domingo")
>      ("Martin Lorentzon" "Martin Lorentzson")
>      ("Matt Swift" "Matthew Swift")
> +    (nil "Max")
>      ("Maxime Edouard Robert Froumentin" "Max Froumentin")
>      ("Michael R. Mauger" "Michael Mauger")
>      ("Michael D. Ernst" "Michael Ernst")

Hi - this is too broad. It excludes anyone whose name includes "Max", viz:

> -Doug Maxey: changed mouse.el
[...]
> -Maxime Edouard Robert Froumentin: changed gnus-art.el mml.el
> -
> -Max Mikhanosha: changed org-agenda.el org-habit.el org.el

Re: emacs-26 4387bb4 1/3: Update authors

[Glenn Morris]

PS similar thing here due to too-broad "Bastien" entry in authors-aliases:

> -Sébastien Delafond: changed org.el
> -
> -Sébastien Gross: changed hideshow.el
> -
> -Sebastien Kirche: changed mail-extr.el
> -
> -Sébastien Vauban: changed org.el org-agenda.el ox-latex.el ob-core.el
> -  org-clock.el ox-ascii.el ox-html.el
> -

Re: emacs-26 4387bb4 1/3: Update authors

[Glenn Morris]

PPS (Sorry for all the mails). I would suggest reviewing all those
entries, eg these are probably also problematic:
(nil "felix")
(nil "cg")

Re: emacs-26 4387bb4 1/3: Update authors

[Nicolas Petton]

[-- Attachment #1: Type: text/plain, Size: 225 bytes --]

Glenn Morris <rgm@gnu.org> writes:

Hi Glenn,

> Hi - this is too broad. It excludes anyone whose name includes "Max",
> viz:

Indeed, I'll change it (I contacted the author of the commit) to get his
full name.

Cheers,
Nico

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 487 bytes --]

Re: emacs-26 4387bb4 1/3: Update authors

[Nicolas Petton]

[-- Attachment #1: Type: text/plain, Size: 307 bytes --]

Glenn Morris <rgm@gnu.org> writes:

> PPS (Sorry for all the mails). I would suggest reviewing all those
> entries, eg these are probably also problematic:
> (nil "felix")
> (nil "cg")

Indeed, they are.  FWIW, I changed the new entry from "Max" to his email
address, which works as expected.

Cheers,
Nico

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 487 bytes --]