process: include deputies in README Owner entries #2350

Alternative possible version: the first version mirrors the Authors logic (full regex + email storage in %emails) for consistency. However, since Deputies are already in %EmailField, the check already validates the Name <email> format, so we could optionally only extract the name and append it.

+		} elsif ($field eq 'Deputies') {
+			$val =~ m/^(\S[^<@>]*\S) </ or die "Malformed Deputies line in $fn";
+			my $deputyname = $1;
+			if (defined $authors) {
+				$authors .= ", $deputyname";
+			} else {
+				$authors = $deputyname;
+			}

I’d prefer if both the checks are of the same shape and I don’t mind the belt and suspenders here of checking the same thing twice. Even though that might make it easier to introduce a discrepancy in the future, I don’t anticipate that a regex for Name + Email would be modified frequently.

So, if we decided to go this way, I would suggest that we apply the same modification to the Authors check. It would however make more sense to me to just replace this script with one in a language that’s not “write-only” such as Python.

Agree on same shape and keeping the check. LMK if you were thinking of a further change at this time.

Perhaps the $authors variable should now also be renamed to $owners to better express the shifted context.

Good idea, added a refactor commit for the renaming.

No action necessary: I was briefly wondering if $owners can even be empty here, given that the Authors header is required. I came to the conclusion that it could be, if Deputies were listed before Authors (which would then later fail on the order check). So, this should stay as is.