[FFmpeg-devel] [PATCH 0/1] tools/general_assembly.pl - print names with emails

[FFmpeg-devel] [PATCH 0/1] tools/general_assembly.pl - print names with emails

[Cosmin Stejerean via ffmpeg-devel]

Per http://ffmpeg.org/pipermail/ffmpeg-devel/2023-November/316480.html
a list of emails should not be published as the GA membership. To
facilitate publishing the GA list this updates the GA script to also
print names along with the emails in the format

```
Name <email@example.com>
```

Cosmin Stejerean (1):
  update GA script to print names in addition to emails since emails
    should not be shared on the list, this makes it easier to publish
    the current GA membership

 tools/general_assembly.pl | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

-- 
2.31.1

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

[FFmpeg-devel] [PATCH 1/1] tools/general_assembly.pl - print names with emails

[Cosmin Stejerean via ffmpeg-devel]

Update GA script to print names in addition to emails since emails
should not be shared in all contexts, this makes it easier to publish
the current GA membership.

Signed-off-by: Cosmin Stejerean <cosmin@cosmin.at>
---
 tools/general_assembly.pl | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/tools/general_assembly.pl b/tools/general_assembly.pl
index 898a6262ef..aa467ba34a 100644
--- a/tools/general_assembly.pl
+++ b/tools/general_assembly.pl
@@ -7,6 +7,8 @@ use POSIX qw(strftime);
 use Encode qw(decode);
 use Data::Dumper;
 
+binmode(STDOUT, ":utf8");
+
 sub trim { my $s = shift; $s =~ s/^\s+|\s+$//g; return $s };
 
 my @shortlog = split /\n/, decode('UTF-8', `git log --pretty=format:"%aN <%aE>" --since="last 36 months" | sort | uniq -c | sort -r`, Encode::FB_CROAK);
@@ -35,6 +37,7 @@ foreach my $line (@shortlog) {
 }
 
 printf("# %s %s", strftime("%Y-%m-%d", localtime), decode('UTF-8', `git rev-parse HEAD`, Encode::FB_CROAK));
-foreach my $email (sort values %assembly) {
-    printf("%s\n", $email);
+foreach my $name (sort keys %assembly) {
+    my $email = $assembly{$name};
+    printf("%s <%s>\n", $name, $email);
 }
-- 
2.31.1


_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH 1/1] tools/general_assembly.pl - print names with emails

[Nicolas George]

[-- Attachment #1.1: Type: text/plain, Size: 1445 bytes --]

Cosmin Stejerean via ffmpeg-devel (12023-11-03):
> Update GA script to print names in addition to emails since emails
> should not be shared in all contexts, this makes it easier to publish
> the current GA membership.
> 
> Signed-off-by: Cosmin Stejerean <cosmin@cosmin.at>
> ---
>  tools/general_assembly.pl | 7 +++++--
>  1 file changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/tools/general_assembly.pl b/tools/general_assembly.pl
> index 898a6262ef..aa467ba34a 100644
> --- a/tools/general_assembly.pl
> +++ b/tools/general_assembly.pl
> @@ -7,6 +7,8 @@ use POSIX qw(strftime);
>  use Encode qw(decode);
>  use Data::Dumper;
>  
> +binmode(STDOUT, ":utf8");
> +
>  sub trim { my $s = shift; $s =~ s/^\s+|\s+$//g; return $s };
>  
>  my @shortlog = split /\n/, decode('UTF-8', `git log --pretty=format:"%aN <%aE>" --since="last 36 months" | sort | uniq -c | sort -r`, Encode::FB_CROAK);
> @@ -35,6 +37,7 @@ foreach my $line (@shortlog) {
>  }
>  
>  printf("# %s %s", strftime("%Y-%m-%d", localtime), decode('UTF-8', `git rev-parse HEAD`, Encode::FB_CROAK));

> -foreach my $email (sort values %assembly) {
> -    printf("%s\n", $email);
> +foreach my $name (sort keys %assembly) {

This is also changing the sort order. It might be acceptable but it
might also not be.

> +    my $email = $assembly{$name};
> +    printf("%s <%s>\n", $name, $email);
>  }

Regards,

-- 
  Nicolas George

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

[-- Attachment #2: Type: text/plain, Size: 251 bytes --]

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH 1/1] tools/general_assembly.pl - print names with emails

[Cosmin Stejerean via ffmpeg-devel]

> On Nov 3, 2023, at 11:40 AM, Nicolas George <george@nsup.org> wrote:
> 
> This is also changing the sort order. It might be acceptable but it
> might also not be.

It is and I probably should have called that out in the description. If the goal is to publish mostly names and not emails in various places like this list then it seems sorting by name would be more natural. But I'm happy to to update the patch with whatever sort order we want to use.

- Cosmin
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH 1/1] tools/general_assembly.pl - print names with emails

[Thilo Borgmann via ffmpeg-devel]

Hi,

sorry can't comment on the ML atm as people will wonder why I respond to this 
but not to other "more important" mails...

Set aside that I think its complete nonsense not to share the list of mail 
addresses directly for any legal consideration - all these mail addresses were 
put into a public repo and used on a public ML and there is no reason not to 
reshare public info. However if people (next to JB who I don't believe is 
actually having any legal considerations...) don't like it...

The script should output two files then, so that no manual editing is required.
The voting system expects a file with mail addresses one per line and the ML 
then shall be given the very same list but with only the names.
Therefore I think the script should just output two files, ga_names.txt and 
ga_mails.txt.

Thanks for the patch!
-Thilo

Am 03.11.23 um 20:18 schrieb Cosmin Stejerean via ffmpeg-devel:
> 
> 
>> On Nov 3, 2023, at 11:40 AM, Nicolas George <george@nsup.org> wrote:
>>
>> This is also changing the sort order. It might be acceptable but it
>> might also not be.
> 
> It is and I probably should have called that out in the description. If the goal is to publish mostly names and not emails in various places like this list then it seems sorting by name would be more natural. But I'm happy to to update the patch with whatever sort order we want to use.
> 
> - Cosmin
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
> 

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH 1/1] tools/general_assembly.pl - print names with emails

[Thilo Borgmann via ffmpeg-devel]

Am 05.11.23 um 14:41 schrieb Thilo Borgmann via ffmpeg-devel:
> Hi,
> 
> sorry can't comment on the ML atm as people will wonder why I respond to this 
> but not to other "more important" mails...

Having sent this private mail to the ML anyways just proves how much I should 
not post to the ML in my current state :-/

-Thilo
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH 1/1] tools/general_assembly.pl - print names with emails

[Cosmin Stejerean via ffmpeg-devel]

> On Nov 5, 2023, at 5:41 AM, Thilo Borgmann via ffmpeg-devel <ffmpeg-devel@ffmpeg.org> wrote:
> 
> The script should output two files then, so that no manual editing is required.
> The voting system expects a file with mail addresses one per line and the ML then shall be given the very same list but with only the names.

The editing in this case is fairly straight forward to extract the emails between the <>, but if we need a convenient "emails only" output option for the voting system I could also add some switches to the script to print only names, only emails or name and email for example. Outputting to a file (or more than one file) rather than to stdout is certainly an option as well but feels like a bigger change compared to some flags to control formatting. 

- Cosmin
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH 1/1] tools/general_assembly.pl - print names with emails

[Jean-Baptiste Kempf]

On Mon, 6 Nov 2023, at 03:06, Cosmin Stejerean via ffmpeg-devel wrote:
>> On Nov 5, 2023, at 5:41 AM, Thilo Borgmann via ffmpeg-devel <ffmpeg-devel@ffmpeg.org> wrote: 
>> The script should output two files then, so that no manual editing is required.
>> The voting system expects a file with mail addresses one per line and the ML then shall be given the very same list but with only the names.
>
> The editing in this case is fairly straight forward to extract the 
> emails between the <>, but if we need a convenient "emails only" output 
> option for the voting system I could also add some switches to the 
> script to print only names, only emails or name and email for example. 

That would be helpful, tbh.


-- 
Jean-Baptiste Kempf -  President
+33 672 704 734
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".